[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH v3 3/5] qemu-char: fix incorrect state in erro
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-trivial] [PATCH v3 3/5] qemu-char: fix incorrect state in error message |
Date: |
Wed, 05 Nov 2014 10:08:44 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.2.0 |
04.11.2014 13:50, zhanghailiang wrote
> It is either "Failed _to_ do something", or "something failed",
> but not "failed something".
Um. Heh. I was afraid of seeing this ;)
My comment about bad grammar was in context of changing error
reporting in this area -- so while chaning error API in use,
it might be a good idea to reword the obviously wrong error
messages _too_, on the same line..
I'll fold this to the next patch, so not to resend whole thing
again. Oh well ;)
The error messages aren't nice still, but maybe it will be
more clean and easier to understand from the context provided
by the error api.
Thanks,
/mjt