From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38753) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bB5nu-0007NZ-HT for qemu-devel@nongnu.org; Thu, 09 Jun 2016 15:39:47 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bB5ns-0005py-5F for qemu-devel@nongnu.org; Thu, 09 Jun 2016 15:39:45 -0400 References: <1465492263-28472-1-git-send-email-imammedo@redhat.com> <1465492263-28472-2-git-send-email-imammedo@redhat.com> <20160609172900.GV18662@thinpad.lan.raisama.net> From: Eric Blake Message-ID: <5759C5F6.4020405@redhat.com> Date: Thu, 9 Jun 2016 13:39:34 -0600 MIME-Version: 1.0 In-Reply-To: <20160609172900.GV18662@thinpad.lan.raisama.net> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="8WsL3jq1GORPOmgBVc140PfKIEoMd0RVG" Subject: Re: [Qemu-devel] [PATCH v2 1/6] target-i386: cpu: consolidate calls of object_property_parse() in x86_cpu_parse_featurestr List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eduardo Habkost , Igor Mammedov Cc: peter.maydell@linaro.org, mark.cave-ayland@ilande.co.uk, qemu-devel@nongnu.org, blauwirbel@gmail.com, qemu-arm@nongnu.org, pbonzini@redhat.com, rth@twiddle.net This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --8WsL3jq1GORPOmgBVc140PfKIEoMd0RVG Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable 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); >> } >=20 > error_propagate() already ignores local_err=3D=3DNULL 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. --=20 Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org --8WsL3jq1GORPOmgBVc140PfKIEoMd0RVG Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Public key at http://people.redhat.com/eblake/eblake.gpg Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQEcBAEBCAAGBQJXWcX2AAoJEKeha0olJ0Nqr8sH+wUrsy+YgzZ8ImQxyKsGns1U GouVKlYvPi47st6PEbpKvuqlCY5ST8qdwqp+SBA1Kb5gXRoHF+jKVAgpa4ChgBRq qKNQ9LCdhmbmjE/JV9VuhHR5WHWBH9ADpY40kw0D3D3Bohjs8b1CDfGqyYrsaORw w99DC1JRqi35EQQKN0W+2CmVUgdet+BLusK9OoNi36vtrzPJ7wxlUmHzg9ryDoxZ SDH4a35Hhr/Myz6V8vXhJ5CN4gw92MLtSsaeo2Eb4j0Y2gsC3f39LVlkR7vdDgDk SwrLIwT+e+3F5Afdq1RhhgcB3huxGXfgQ292TLgzDLagUySUdY2HzsGGb/XcJAw= =RGCb -----END PGP SIGNATURE----- --8WsL3jq1GORPOmgBVc140PfKIEoMd0RVG--