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