diff options
author | Markus Armbruster <armbru@redhat.com> | 2020-07-07 18:06:05 +0200 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2020-07-10 15:18:08 +0200 |
commit | a5f9b9df252d0dfb407178ef4c3769f78a64b2ff (patch) | |
tree | 3332a93b91cc71e15b74b9cb566451397c1ff387 /hw/core/qdev.c | |
parent | 992861fb1e4cf410f30ec8f05bd2dc2a14a5a027 (diff) | |
download | qemu-a5f9b9df252d0dfb407178ef4c3769f78a64b2ff.zip qemu-a5f9b9df252d0dfb407178ef4c3769f78a64b2ff.tar.gz qemu-a5f9b9df252d0dfb407178ef4c3769f78a64b2ff.tar.bz2 |
error: Reduce unnecessary error propagation
When all we do with an Error we receive into a local variable is
propagating to somewhere else, we can just as well receive it there
right away, even when we need to keep error_propagate() for other
error paths.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20200707160613.848843-38-armbru@redhat.com>
Diffstat (limited to 'hw/core/qdev.c')
-rw-r--r-- | hw/core/qdev.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/core/qdev.c b/hw/core/qdev.c index 6894876..0179682 100644 --- a/hw/core/qdev.c +++ b/hw/core/qdev.c @@ -871,7 +871,7 @@ static void device_set_realized(Object *obj, bool value, Error **errp) } if (value && !dev->realized) { - if (!check_only_migratable(obj, &local_err)) { + if (!check_only_migratable(obj, errp)) { goto fail; } |