Attention is currently required from: laforge.
Hoernchen has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42784?usp=email )
Change subject: libosmo_emb: type-safe tearfree_u64_t wrapper for LDRD/STRD access ......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Looking at the patch, I don't really see what it gains us? There appears to be no code that we can […]
The whole point of c++ templates/c11 generics is that types are never automatically converted and types have to match so this patch ensures you get compile time errors if you try to call the functions with unaligned data that is not our struct. Things the compiler can check at compile time should be loaded off to the compiler, in particular unexpected requirements like alignment. Many parts of the firmware could be much safer and easier to understand by introducing generics + TS 25755/gcc >8 __cleanup__ defer.