Hey, > > It's definitely not used consistently, but removing a 'const' from > > external API is going to cause breakage in C++ code: > > Well that's true but IMHO this breakage is OK, given: > > * the unlikelyhood of an existing C++ project using libosinfo.
This patch would fix generation of rust bindings, which have 0 users for sure at the moment since it's broken. So there are at least as many C++ projects using libosinfo, maybe more :) If the breakage is fixed in the Rust binding generator soon, I'd rather we drop this patch. Christophe
signature.asc
Description: PGP signature
_______________________________________________ Libosinfo mailing list [email protected] https://www.redhat.com/mailman/listinfo/libosinfo
