Re: [PATCH 9/9] ui: Reduce nesting in hmp_change_vnc() slightly

[Daniel P . Berrangé]

On Thu, Dec 01, 2022 at 07:13:11AM +0100, Markus Armbruster wrote:
> Transform
> 
>     if (good) {
>         do stuff
>     } else {
>         handle error
>     }
> 
> to
> 
>     if (!good) {
>         handle error
>       return;
>     }
>     do stuff
> 
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  ui/ui-hmp-cmds.c | 17 +++++++----------
>  1 file changed, 7 insertions(+), 10 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|