[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 21/21] nbd: assert that Error** is not NULL in nbd_iter_ch
Re: [PATCH v7 21/21] nbd: assert that Error** is not NULL in nbd_iter_channel_error
Thu, 5 Dec 2019 13:56:54 -0600
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2
On 12/5/19 12:09 PM, Vladimir Sementsov-Ogievskiy wrote:
All callers of nbd_iter_channel_error() pass the address of a local_err
variable, and only call this function if an error has already occurred, using
to append details to that error.
Hmm, not to append details but to report the error to the magical receiving
loop, which doesn't yet know about the error
This is already implied by its name (local_err instead of the classic errp),
but it is worth additionally stressing this by adding an assertion to make it
part of the function contract.
So, how about simply s/to append details to that error/to report that error/ ?
That should be fine.
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
Re: [PATCH v7 00/21] error: prepare for auto propagated local_err, Cornelia Huck, 2019/12/05