[libvirt] [PATCH 3/3] docs: Update HACKING
Eric Blake
eblake at redhat.com
Wed Apr 17 20:25:20 UTC 2013
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130417/afd198fb/attachment-0001.sig>
More information about the libvir-list
mailing list