Follow-up from "New ODIN implementation"
The following discussion from !3088 (merged) should be addressed:
-
@graven started a discussion: (+2 comments) instead of asserting here, you could push the constraint 'up' by doing:
void ODIN::to_version<6>( LHCb::span<std::uint32_t,10> buffer ) const { using LHCb::ODINImplementation::details::set_bits;
which forces the caller to get it right...