On Thu, 9 Jun 2016 13:39:34 -0600
Eric Blake <ebl...@redhat.com> wrote:
> On 06/09/2016 11:29 AM, Eduardo Habkost wrote:
> > On Thu, Jun 09, 2016 at 07:10:58PM +0200, Igor Mammedov wrote:
> > [...]
> >> +
> >> + if (local_err) {
> >> + error_propagate(errp, local_err);
> >> }
> >
> > error_propagate() already ignores local_err==NULL so you don't
> > need to check it first.
>
> In fact, if the ONLY reason you are doing an 'if (local_err)'
> conditional is to decide if an error was set, then you don't care about
> the error locally, and could have passed errp instead of &local_err in
> the first place.
errp might be NULL, so we won't do parsing at all.
considering it's more or less generic API hook it's safer
to follow error handling pattern used in properties
i.e. use local_error internally and then propagate error
if caller cares about it.
