On Thu, 9 Jun 2016 14:29:00 -0300
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
On Thu, 9 Jun 2016 13:39:34 -0600
Eric Blake 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_propag
On Thu, Jun 09, 2016 at 01:39:34PM -0600, Eric Blake 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_prop
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.
From: Eduardo Habkost
Signed-off-by: Eduardo Habkost
Reviewed-by: Igor Mammedov
Signed-off-by: Igor Mammedov
Reviewed-by: Eduardo Habkost
---
v1:
- fix error handling in of +-feat, Igor Mammedov
- rebase on top of
"target-i386: Remove xlevel & hv-spinlocks option fixups"
v2:
- move er
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.
I can change this while applying the patch, if you're OK.