On 09/15/14 19:00, Eric Blake wrote: > On 09/15/2014 09:42 AM, Peter Krempa wrote: >> Add a note to make the users aware that some stats groups or fields may >> be missing in certain cases. >> --- >> src/libvirt.c | 5 +++++ >> 1 file changed, 5 insertions(+) >> > > ACK. > >> diff --git a/src/libvirt.c b/src/libvirt.c >> index 03f7e5b..b3ddf95 100644 >> --- a/src/libvirt.c >> +++ b/src/libvirt.c >> @@ -21657,6 +21657,11 @@ virConnectGetDomainCapabilities(virConnectPtr conn, >> * "block.<num>.allocation" - offset of the highest written sector >> * as unsigned long long. >> * >> + * Note that entire stats groups or individual stat fields may be missing from >> + * the output in case they are not supported by the given hypervisor, are not >> + * applicable for the current state of the guest domain or their retrieval >> + * was not successful. > > Depending on which grammar expert you ask, some people prefer "a, b, or > c" while others prefer "a, b or c" when listing three or more things > (the use of a comma before the conjunction that lists the last element > of a list). I'm in the former group, but not strongly enough to request > that others change their habits. So up to you if you want to add a > comma before that last "or". > Well, english is not my first neither second language so I'll gladly learn something new. I'll go with the first option as it makes sense to me in retrospect. (I should also get into the habit of re-reading things I write) Peter (No I didn't re-read this message :) )
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list