qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] qemu-char: Print strerror message on failure


From: Kevin Wolf
Subject: Re: [Qemu-devel] [PATCH] qemu-char: Print strerror message on failure
Date: Tue, 21 Jun 2011 18:18:57 +0200
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110428 Fedora/3.1.10-1.fc15 Thunderbird/3.1.10

Am 21.06.2011 18:09, schrieb Markus Armbruster:
> Kevin Wolf <address@hidden> writes:
> 
>> The only way for chardev drivers to communicate an error was to return a NULL
>> pointer, which resulted in an error message that said _that_ something went
>> wrong, but not _why_.
>>
>> This patch changes the interface to return 0/-errno and updates
>> qemu_chr_open_opts to use strerror to display a more helpful error message.
> 
> Returning the result through a pointer is awkward.  What about stuffing
> the error code into errno?

I generally like it better to return error codes explicitly instead of
storing them in some global variable where it tends to be trampled over
before you print the message. But if people prefer it that way, I can
redo the patch.

Kevin



reply via email to

[Prev in Thread] Current Thread [Next in Thread]