On 04/17/2013 04:19 AM, Osier Yang wrote:
> To tell libvirt-{qemu,lxc}.h shouldn't be included either.
> ---
> docs/hacking.html.in | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)You also need to run 'make HACKING' and check that in at the same time. (Yes, that's one of the few generated files we actually version control). > > diff --git a/docs/hacking.html.in b/docs/hacking.html.in > index 7ef826c..67af5c2 100644 > --- a/docs/hacking.html.in > +++ b/docs/hacking.html.in > @@ -929,10 +929,10 @@ > </pre> > > <p> > - Of particular note: <b>Do not</b> include libvirt/libvirt.h or > - libvirt/virterror.h. It is included by "internal.h" already and there > - are some special reasons why you cannot include these files > - explicitly. > + Of particular note: <b>Do not</b> include libvirt/libvirt.h, > + libvirt/virterror.h, libvirt/libvirt-qemu.h, and libvirt/libvirt-lxc.h. s/and/or/ > + It is included by "internal.h" already and there are some special > reasons s/It is/They are/ > + why you cannot include these files explicitly. Is it worth mentioning the visibility of *_LAST enums as one of these reasons? ACK with the grammar fix and generated file; up to you whether to further document the "why". -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
