Re: more missing #include's of "config.h"

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

 



On Thu, Dec 06, 2007 at 06:32:24PM +0100, Jim Meyering wrote:
> This is the last of the non-generated files requiring #include "config.h":
> 
> 	Include "config.h" in remaining non-generated files.
>         * docs/examples/info1.c: Likewise.
> 	* docs/examples/suspend.c: Likewise.

For those 2 files I don't agree with adding it, they are example files
showing how to use the API and config.h is an internal header, those 
user level examples should really not include it.
But for the other case this looks fine to me,

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard@xxxxxxxxxx  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/

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