[virt-tools-list] [virt-viewer 2/2] app, cosmetic: remove one unneeded level of identation

Pavel Grunt pgrunt at redhat.com
Mon Mar 7 10:53:54 UTC 2016


Ack,

Thanks,
Pavel

On Mon, 2016-03-07 at 10:37 +0100, Fabiano Fidêncio wrote:
> Signed-off-by: Fabiano Fidêncio <fidencio at redhat.com>
> ---
>  src/virt-viewer-app.c | 28 ++++++++++++++--------------
>  1 file changed, 14 insertions(+), 14 deletions(-)
> 
> diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
> index bbb4928..1cab1d9 100644
> --- a/src/virt-viewer-app.c
> +++ b/src/virt-viewer-app.c
> @@ -407,21 +407,21 @@ virt_viewer_app_parse_monitor_mappings(gchar
> **mappings, gsize nmappings)
>          if (monitor > nmonitors) {
>              g_warning("Initial monitor #%i for display #%i does not
> exist", monitor, display);
>              goto configerror;
> -        } else {
> -            /* config file format is 1-based, not 0-based */
> -            display--;
> -            monitor--;
> -
> -            if (g_hash_table_lookup_extended(displaymap,
> GINT_TO_POINTER(display), NULL, NULL) ||
> -                g_hash_table_lookup_extended(monitormap,
> GINT_TO_POINTER(monitor), NULL, NULL)) {
> -                g_warning("Invalid monitor-mapping configuration: a
> display or monitor id was specified twice");
> -                goto configerror;
> -            }
> -            g_debug("Fullscreen config: mapping guest display %i to
> monitor %i", display, monitor);
> -            g_hash_table_insert(displaymap,
> GINT_TO_POINTER(display), GINT_TO_POINTER(monitor));
> -            g_hash_table_insert(monitormap,
> GINT_TO_POINTER(monitor), GINT_TO_POINTER(display));
> -            max_display_id = MAX(display, max_display_id);
>          }
> +
> +        /* config file format is 1-based, not 0-based */
> +        display--;
> +        monitor--;
> +
> +        if (g_hash_table_lookup_extended(displaymap,
> GINT_TO_POINTER(display), NULL, NULL) ||
> +            g_hash_table_lookup_extended(monitormap,
> GINT_TO_POINTER(monitor), NULL, NULL)) {
> +            g_warning("Invalid monitor-mapping configuration: a
> display or monitor id was specified twice");
> +            goto configerror;
> +        }
> +        g_debug("Fullscreen config: mapping guest display %i to
> monitor %i", display, monitor);
> +        g_hash_table_insert(displaymap, GINT_TO_POINTER(display),
> GINT_TO_POINTER(monitor));
> +        g_hash_table_insert(monitormap, GINT_TO_POINTER(monitor),
> GINT_TO_POINTER(display));
> +        max_display_id = MAX(display, max_display_id);
>      }
>  
>      for (i = 0; i < max_display_id; i++) {




More information about the virt-tools-list mailing list