On 11/28/2013 07:03 AM, Nehal J Wani wrote: > On Thu, Nov 28, 2013 at 5:31 PM, Chen Hanxiao > <chenhanxiao@xxxxxxxxxxxxxx> wrote: >> From: Chen Hanxiao <chenhanxiao@xxxxxxxxxxxxxx> >> >> /** >> * virDomainEventDefinedDetailType: >> * >> - * Details on the caused of the 'defined' lifecycle event >> + * Details on the cause of the 'defined' lifecycle event >> */ >> 1.8.2.1 >> > > I am no English expert, but would like to ask, if a particular event > can take place due to multiple reasons, shouldn't it be : > "Details on the causes of the 'defined' lifecycle event" > I think the original author made a typo by typing 'd' instead of 's' > (very close on keyboard), i.e., s/caused/causes Causes would make sense if you could have a plurality of causes triggering a single event. But we don't. Each event has exactly one cause. The patch is correct as-is, so I'll apply it soon. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list