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. -- To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42784?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ccid-firmware Gerrit-Branch: master Gerrit-Change-Id: I458bfaf53e27c51882c7a8c1326e51d12943b0bb Gerrit-Change-Number: 42784 Gerrit-PatchSet: 1 Gerrit-Owner: Hoernchen <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <[email protected]> Gerrit-Attention: laforge <[email protected]> Gerrit-Comment-Date: Wed, 03 Jun 2026 16:17:07 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <[email protected]>
