On 11/28/2013 07:03 AM, Nehal J Wani wrote:
On Thu, Nov 28, 2013 at 5:31 PM, Chen Hanxiao <chenhanxiao@cn.fujitsu.com> wrote:
From: Chen Hanxiao <chenhanxiao@cn.fujitsu.com>
/** * 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