Re: [PATCH] Drop redundant 'const' keyword from object parameters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 17, 2017 at 12:26:48PM +0100, Zeeshan Ali (Khattak) wrote:
> On 17 November 2017 at 12:21, Daniel P. Berrange <berrange@xxxxxxxxxx> 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.
> 
> Moreover, it's super trivial fix for the build of C++ code. Does that
> fact help you cross the fence? :)

If you go the ugly way (add a cast from const to non-const), then yes
it's probably trivial to change. If you want to do things properly, then
I would not call it "super trivial"

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux