qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [RFC PATCH] qemu-socket: Use local error variable


From: Laszlo Ersek
Subject: Re: [Qemu-devel] [RFC PATCH] qemu-socket: Use local error variable
Date: Thu, 14 Mar 2013 16:52:04 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130216 Thunderbird/17.0.3

On 03/14/13 15:57, Kevin Wolf wrote:
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
> After rebasing this I saw that Anthony already committed a fix that is
> very close to my v1. I don't intend to actually change that code, but as
> I've already done this, just for comparison what it would look like with
> error propagation. Is this what you meant? I find the result more
> confusing, to be honest.

I think what I had in mind was:
- I was okay with the logic change you suggested in your v1, just
- turn *errp accesses into local_err accesses,
- when returning, propagate the latter to the former.

The logic seemed OK, I just suggested to keep the massage internal to
the function, only try to propagate it outwards at return time. IOW,
never read *errp.

Laszlo



reply via email to

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