Philippe Mathieu-Daudé <phi...@redhat.com> writes:
> Simplify by directly using visit_type_size() return value.
>
> Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
> ---
> hw/virtio/virtio-mem.c | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/hw/virtio/virtio-mem.c b/hw/virtio/virtio-mem.c
> index ddf09ed1bf..e407abc2d8 100644
> --- a/hw/virtio/virtio-mem.c
> +++ b/hw/virtio/virtio-mem.c
> @@ -687,12 +687,9 @@ static void virtio_mem_set_requested_size(Object *obj,
> Visitor *v,
> Error **errp)
> {
> VirtIOMEM *vmem = VIRTIO_MEM(obj);
> - Error *err = NULL;
> uint64_t value;
>
> - visit_type_size(v, name, &value, &err);
> - if (err) {
> - error_propagate(errp, err);
> + if (!visit_type_size(v, name, &value, errp)) {
> return;
> }
Crept in after I transformed the pattern in my "Less clumsy error
checking" series. There are a few more instances. I expected that to
happen, and intend to transform stragglers periodically until their
supply dries up.
