Skip to content

Commit

Permalink
chore(aeva): remove pointer hackery for cleaner code
Browse files Browse the repository at this point in the history
Signed-off-by: Max SCHMELLER <[email protected]>
  • Loading branch information
mojomex committed Nov 18, 2024
1 parent f5bd7fc commit a132496
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,8 @@ class HealthParser : public AevaParser<AevaStreamType::kHealth>
entries.reserve(n_entries);

Check warning on line 58 in nebula_hw_interfaces/include/nebula_hw_interfaces/nebula_hw_interfaces_aeva/connections/health.hpp

View check run for this annotation

Codecov / codecov/patch

nebula_hw_interfaces/include/nebula_hw_interfaces/nebula_hw_interfaces_aeva/connections/health.hpp#L58

Added line #L58 was not covered by tests

for (size_t i = 0; i < n_entries; ++i) {
auto pointer = &*payload_bytes.consume_unsafe(sizeof(uint32_t)).cbegin();
auto entry = boost::endian::load_little_u32(pointer);
auto health_code_raw = payload_bytes.consume_unsafe(sizeof(uint32_t)).cbegin();
auto entry = boost::endian::load_little_u32(health_code_raw.base());
entries.emplace_back(entry);

Check warning on line 63 in nebula_hw_interfaces/include/nebula_hw_interfaces/nebula_hw_interfaces_aeva/connections/health.hpp

View check run for this annotation

Codecov / codecov/patch

nebula_hw_interfaces/include/nebula_hw_interfaces/nebula_hw_interfaces_aeva/connections/health.hpp#L61-L63

Added lines #L61 - L63 were not covered by tests
}

Expand Down

0 comments on commit a132496

Please sign in to comment.