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

Fabiano Fidêncio fidencio at redhat.com
Mon Mar 7 15:07:58 UTC 2016


On Mon, Mar 7, 2016 at 11:53 AM, Pavel Grunt <pgrunt at redhat.com> wrote:
> Ack,
>
> Thanks,
> Pavel

Pushed, thanks!

>
> 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