[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 00/10] qemu-ga: revamp error messages (for 1.4)
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 00/10] qemu-ga: revamp error messages (for 1.4) |
Date: |
Tue, 11 Dec 2012 18:03:24 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 |
On 11/28/2012 01:04 PM, mdroth wrote:
> On Tue, Nov 27, 2012 at 11:01:54AM -0200, Luiz Capitulino wrote:
>> This series revamps qemu-ga error messages by:
>>
>> - Propagating errors down to functions that generate errors
>> - Fixing UndefinedErrors
>> - Getting rid of sprint() + error_set()
>> - Other small fixes
>
> Hi Luiz,
>
> Patches look good other than a few comments on usage of possibly-unset errno.
>
> CC'ing Eric as a heads up, but based on previous comments (libvirt
> always using 'desc' field over 'class' field and reporting it directly
> to users) I don't think it should cause any compatibility issues on that
> end.
And finally replying. You are correct that libvirt favors the contents
of 'desc' and only if that is missing falls back to 'class' when passing
an error message through to the user. I didn't spot any issues with the
changes in this series, although I admit my testing was rather light.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH 00/10] qemu-ga: revamp error messages (for 1.4),
Eric Blake <=