diff mbox

vl.c: fix some alignment issues

Message ID 1420969037-16603-1-git-send-email-marcel@redhat.com
State New
Headers show

Commit Message

Marcel Apfelbaum Jan. 11, 2015, 9:37 a.m. UTC
The main's switch had a few cases misaligned.

Signed-off-by: Marcel Apfelbaum <marcel@redhat.com>
---
 vl.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

Comments

Stefan Weil Jan. 11, 2015, 10:06 a.m. UTC | #1
Am 11.01.2015 um 10:37 schrieb Marcel Apfelbaum:
> The main's switch had a few cases misaligned.
>
> Signed-off-by: Marcel Apfelbaum <marcel@redhat.com>
> ---
>   vl.c | 12 ++++++------
>   1 file changed, 6 insertions(+), 6 deletions(-)
>

Reviewed-by: Stefan Weil <sw@weilnetz.de>

The misalignment was caused by tabs which were used instead of spaces.

A short look on vl.c shows lots of more lines with tabs (also in other 
switch
statements), so this patch could be extended to fix more lines.

Stefan
Marcel Apfelbaum Jan. 11, 2015, 10:12 a.m. UTC | #2
On 01/11/2015 12:06 PM, Stefan Weil wrote:
> Am 11.01.2015 um 10:37 schrieb Marcel Apfelbaum:
>> The main's switch had a few cases misaligned.
>>
>> Signed-off-by: Marcel Apfelbaum <marcel@redhat.com>
>> ---
>>   vl.c | 12 ++++++------
>>   1 file changed, 6 insertions(+), 6 deletions(-)
>>
>
> Reviewed-by: Stefan Weil <sw@weilnetz.de>
>
> The misalignment was caused by tabs which were used instead of spaces.
>
> A short look on vl.c shows lots of more lines with tabs (also in other switch
> statements), so this patch could be extended to fix more lines.
This is what I saw while fixing other stuff. :)
I'll try to hunt others and send another version.

Thanks,
Marcel

>
> Stefan
>
diff mbox

Patch

diff --git a/vl.c b/vl.c
index 7786b2f..adfdbfd 100644
--- a/vl.c
+++ b/vl.c
@@ -3147,9 +3147,9 @@  int main(int argc, char **argv, char **envp)
             case QEMU_OPTION_S:
                 autostart = 0;
                 break;
-	    case QEMU_OPTION_k:
-		keyboard_layout = optarg;
-		break;
+            case QEMU_OPTION_k:
+                keyboard_layout = optarg;
+                break;
             case QEMU_OPTION_localtime:
                 rtc_utc = 0;
                 break;
@@ -3357,9 +3357,9 @@  int main(int argc, char **argv, char **envp)
             case QEMU_OPTION_debugcon:
                 add_device_config(DEV_DEBUGCON, optarg);
                 break;
-	    case QEMU_OPTION_loadvm:
-		loadvm = optarg;
-		break;
+            case QEMU_OPTION_loadvm:
+                loadvm = optarg;
+                break;
             case QEMU_OPTION_full_screen:
                 full_screen = 1;
                 break;