Re: [PATCH libvirt-glib 1/3] Add GVirDomainDevice abstract class

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

 



Hey,

On Thu, Nov 10, 2011 at 04:10:32PM -0500, Marc-André Lureau wrote:
> ----- Mensaje original -----
> > > +G_GNUC_INTERNAL
> > > +virDomainPtr gvir_domain_device_get_domain_handle(GVirDomainDevice
> > > *self)
> > 
> > G_GNUC_INTERNAL isn't needed here since it's the default setting for
> > symbols not listed in the .sym file.
> > 
> 
> I think it's better to be explicit about it. It's a way to declare in the
> code that this function is private, without having to look into the sym
> file. I wish we could do the same for the rest of the prviate functions.

Given that having it or not will have exactly the same result, I have
doubts we'll consistently mark all private functions with G_GNUC_INTERNAL.
Only having some of the private functions marked with it, and the others
without the marking will only create more confusion.
If we are careful about marking all private functions with G_GNUC_INTERNAL,
then why not.

Christophe

Attachment: pgp8auF0TqK3K.pgp
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

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