[Libosinfo] [PATCH] Drop redundant 'const' keyword from object parameters

Christophe Fergeau cfergeau at redhat.com
Fri Nov 17 13:09:02 UTC 2017


On Fri, Nov 17, 2017 at 11:21:21AM +0000, Daniel P. Berrange wrote:
> On Fri, Nov 17, 2017 at 11:57:29AM +0100, Christophe Fergeau wrote:
> > 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.
> 
> I'm on the fence, but I think on balance i'm slightly in favour of  taking
> in the patch. It doesn't change ABI, only API so it would only be source
> level incompatible. Even then its only incompatible if the dev actually
> declared the variable const, which is about as likely as someone using
> C++ in the first place.

Yes, your point about the actual object most likely being declared
without the "const" modifier also makes me lean towards taking it.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libosinfo/attachments/20171117/a758c2fb/attachment.sig>


More information about the Libosinfo mailing list