[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/11] migration: return real error code
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH 04/11] migration: return real error code |
Date: |
Wed, 05 Oct 2011 22:07:18 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux) |
Anthony Liguori <address@hidden> wrote:
> On 10/04/2011 01:35 PM, Juan Quintela wrote:
>> Anthony Liguori<address@hidden> wrote:
>>> On 09/23/2011 07:50 AM, Juan Quintela wrote:
>>>> make functions propaget errno, instead of just using -EIO.
>>>>
>>>> Signed-off-by: Juan Quintela<address@hidden>
>>>
>>> qemu_file_has_error() implies a boolean response. Wouldn't
>>> qemu_file_get_error() make more sense if you're going to rely on the
>>> return value?
>>
>> I just didn't want to change more things on the same patch.
>> I can add a patch on top of this series?
>
> It's terribly odd to make a function that looks like a bool return a
> non-boolean value. It can't be that much of a change to do it in this
> patch, it's just a matter of running sed.
That is already fixed. Problem was not doing that change, was to fix
the rejects after that on the following patches.
Later, Juan.