Re: [PATCHv2 5/9] network: reorganize dnsmasq and radvd config file / startup

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

 



On 09/18/2012 07:45 PM, Eric Blake wrote:
> On 09/18/2012 01:39 AM, Laine Stump wrote:
>> This patch splits the starting of dnsmasq and radvd into multiple
>> files, and adds new networkRefreshXX() and networkRestartXX()
>> functions for each. These new functions are currently commented out
>> because they won't be used until the next commit, and the compile options
>> require all static functions to be used.
> The other solution to using #if 0 is to instead mark the function
> ATTRIBUTE_UNUSED, so that it will at least compile; then remove the
> attribute in the later patch where you finally use it.

Ah, I didn't realize ATTRIBUTE_UNUSED worked for functions as well (or
maybe I used to know that, and forgot). Much better solution (since you
can then compile the function to make sure it has no errors).

--
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]