You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The generated header has the intended type. Example: std::vectorstd::uint32_t value{}; . But it's trying to push_back() with no argument in the deserialize code.
std::vector<std::uint32_t> value{};
...
nunavut::support::SerializeResult
deserialize(nunavut::support::const_bitspan in_buffer)
{
...
for (size_t _index5_ = 0U; _index5_ < _size0_; ++_index5_)
{
// TODO This is terribly inefficient. We need to completely refactor this template to use C++ emplace and
// move semantics instead of assuming C-style containers
value.push_back(); // <--- This causes compiler errors
value[_index5_] = in_buffer.getU32(32U);
in_buffer.add_offset(32U);
}
...
}
The text was updated successfully, but these errors were encountered:
Confirmed this is a bug (sorry) and have a fast fix in my fork (thirtytwobits@ed93979) but I'd like to massage out that TODO This is terribly inefficient... comment as part of this fix.
When following https://nunavut.readthedocs.io/en/latest/docs/languages.html to use
std::vector
instead ofvariable_length_array
, there is a bug in the generated code.The generated header has the intended type. Example: std::vectorstd::uint32_t value{}; . But it's trying to push_back() with no argument in the deserialize code.
The text was updated successfully, but these errors were encountered: