VirtualBox

Changeset 18780 in vbox for trunk/src/VBox


Ignore:
Timestamp:
Apr 6, 2009 3:47:00 PM (16 years ago)
Author:
vboxsync
Message:

VBoxManage/modifyvm: double-dash command line options

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/VBox/Frontends/VBoxManage/VBoxManageModifyVM.cpp

    r18591 r18780  
    11/* $Id$ */
    22/** @file
    3  * VBoxManage - Implementation of -modifyvm command.
     3 * VBoxManage - Implementation of modifyvm command.
    44 */
    55
     
    141141    for (int i = 1; i < a->argc; i++)
    142142    {
    143         if (strcmp(a->argv[i], "-name") == 0)
     143        if (   !strcmp(a->argv[i], "--name")
     144            || !strcmp(a->argv[i], "-name"))
    144145        {
    145146            if (a->argc <= i + 1)
     
    148149            name = a->argv[i];
    149150        }
    150         else if (strcmp(a->argv[i], "-ostype") == 0)
     151        else if (   !strcmp(a->argv[i], "--ostype")
     152                 || !strcmp(a->argv[i], "-ostype"))
    151153        {
    152154            if (a->argc <= i + 1)
     
    155157            ostype = a->argv[i];
    156158        }
    157         else if (strcmp(a->argv[i], "-memory") == 0)
     159        else if (   !strcmp(a->argv[i], "--memory")
     160                 || !strcmp(a->argv[i], "-memory"))
    158161        {
    159162            if (a->argc <= i + 1)
     
    162165            memorySize = RTStrToUInt32(a->argv[i]);
    163166        }
    164         else if (strcmp(a->argv[i], "-vram") == 0)
     167        else if (   !strcmp(a->argv[i], "--vram")
     168                 || !strcmp(a->argv[i], "-vram"))
    165169        {
    166170            if (a->argc <= i + 1)
     
    169173            vramSize = RTStrToUInt32(a->argv[i]);
    170174        }
    171         else if (strcmp(a->argv[i], "-acpi") == 0)
     175        else if (   !strcmp(a->argv[i], "--acpi")
     176                 || !strcmp(a->argv[i], "-acpi"))
    172177        {
    173178            if (a->argc <= i + 1)
     
    176181            acpi = a->argv[i];
    177182        }
    178         else if (strcmp(a->argv[i], "-ioapic") == 0)
     183        else if (   !strcmp(a->argv[i], "--ioapic")
     184                 || !strcmp(a->argv[i], "-ioapic"))
    179185        {
    180186            if (a->argc <= i + 1)
     
    183189            ioapic = a->argv[i];
    184190        }
    185         else if (strcmp(a->argv[i], "-hwvirtex") == 0)
     191        else if (   !strcmp(a->argv[i], "--hwvirtex")
     192                 || !strcmp(a->argv[i], "-hwvirtex"))
    186193        {
    187194            if (a->argc <= i + 1)
     
    190197            hwvirtex = a->argv[i];
    191198        }
    192         else if (strcmp(a->argv[i], "-nestedpaging") == 0)
     199        else if (   !strcmp(a->argv[i], "--nestedpaging")
     200                 || !strcmp(a->argv[i], "-nestedpaging"))
    193201        {
    194202            if (a->argc <= i + 1)
     
    197205            nestedpaging = a->argv[i];
    198206        }
    199         else if (strcmp(a->argv[i], "-vtxvpid") == 0)
     207        else if (   !strcmp(a->argv[i], "--vtxvpid")
     208                 || !strcmp(a->argv[i], "-vtxvpid"))
    200209        {
    201210            if (a->argc <= i + 1)
     
    204213            vtxvpid = a->argv[i];
    205214        }
    206         else if (strcmp(a->argv[i], "-pae") == 0)
     215        else if (   !strcmp(a->argv[i], "--pae")
     216                 || !strcmp(a->argv[i], "-pae"))
    207217        {
    208218            if (a->argc <= i + 1)
     
    211221            pae = a->argv[i];
    212222        }
    213         else if (strcmp(a->argv[i], "-monitorcount") == 0)
     223        else if (   !strcmp(a->argv[i], "--monitorcount")
     224                 || !strcmp(a->argv[i], "-monitorcount"))
    214225        {
    215226            if (a->argc <= i + 1)
     
    218229            monitorcount = RTStrToUInt32(a->argv[i]);
    219230        }
    220         else if (strcmp(a->argv[i], "-accelerate3d") == 0)
     231        else if (   !strcmp(a->argv[i], "--accelerate3d")
     232                 || !strcmp(a->argv[i], "-accelerate3d"))
    221233        {
    222234            if (a->argc <= i + 1)
     
    225237            accelerate3d = a->argv[i];
    226238        }
    227         else if (strcmp(a->argv[i], "-bioslogofadein") == 0)
     239        else if (   !strcmp(a->argv[i], "--bioslogofadein")
     240                 || !strcmp(a->argv[i], "-bioslogofadein"))
    228241        {
    229242            if (a->argc <= i + 1)
     
    232245            bioslogofadein = a->argv[i];
    233246        }
    234         else if (strcmp(a->argv[i], "-bioslogofadeout") == 0)
     247        else if (   !strcmp(a->argv[i], "--bioslogofadeout")
     248                 || !strcmp(a->argv[i], "-bioslogofadeout"))
    235249        {
    236250            if (a->argc <= i + 1)
     
    239253            bioslogofadeout = a->argv[i];
    240254        }
    241         else if (strcmp(a->argv[i], "-bioslogodisplaytime") == 0)
     255        else if (   !strcmp(a->argv[i], "--bioslogodisplaytime")
     256                 || !strcmp(a->argv[i], "-bioslogodisplaytime"))
    242257        {
    243258            if (a->argc <= i + 1)
     
    246261            bioslogodisplaytime = RTStrToUInt32(a->argv[i]);
    247262        }
    248         else if (strcmp(a->argv[i], "-bioslogoimagepath") == 0)
     263        else if (   !strcmp(a->argv[i], "--bioslogoimagepath")
     264                 || !strcmp(a->argv[i], "-bioslogoimagepath"))
    249265        {
    250266            if (a->argc <= i + 1)
     
    253269            bioslogoimagepath = a->argv[i];
    254270        }
    255         else if (strcmp(a->argv[i], "-biosbootmenu") == 0)
     271        else if (   !strcmp(a->argv[i], "--biosbootmenu")
     272                 || !strcmp(a->argv[i], "-biosbootmenu"))
    256273        {
    257274            if (a->argc <= i + 1)
     
    260277            biosbootmenumode = a->argv[i];
    261278        }
    262         else if (strcmp(a->argv[i], "-biossystemtimeoffset") == 0)
     279        else if (   !strcmp(a->argv[i], "--biossystemtimeoffset")
     280                 || !strcmp(a->argv[i], "-biossystemtimeoffset"))
    263281        {
    264282            if (a->argc <= i + 1)
     
    267285            biossystemtimeoffset = a->argv[i];
    268286        }
    269         else if (strcmp(a->argv[i], "-biospxedebug") == 0)
     287        else if (   !strcmp(a->argv[i], "--biospxedebug")
     288                 || !strcmp(a->argv[i], "-biospxedebug"))
    270289        {
    271290            if (a->argc <= i + 1)
     
    274293            biospxedebug = a->argv[i];
    275294        }
    276         else if (strncmp(a->argv[i], "-boot", 5) == 0)
     295        else if (   !strncmp(a->argv[i], "--boot", 6)
     296                 || !strncmp(a->argv[i], "-boot", 5))
    277297        {
    278298            uint32_t n = 0;
    279             if (!a->argv[i][5])
     299            if (!a->argv[i][5 + (a->argv[i][1] == '-')])
    280300                return errorSyntax(USAGE_MODIFYVM, "Missing boot slot number in '%s'", a->argv[i]);
    281             if (VINF_SUCCESS != RTStrToUInt32Full(&a->argv[i][5], 10, &n))
     301            if (VINF_SUCCESS != RTStrToUInt32Full(&a->argv[i][5 + (a->argv[i][1] == '-')], 10, &n))
    282302                return errorSyntax(USAGE_MODIFYVM, "Invalid boot slot number in '%s'", a->argv[i]);
    283303            if (a->argc <= i + 1)
    284304                return errorArgument("Missing argument to '%s'", a->argv[i]);
    285305            i++;
    286             if (strcmp(a->argv[i], "none") == 0)
     306            if (!strcmp(a->argv[i], "none"))
    287307            {
    288308                bootDevice[n - 1] = DeviceType_Null;
    289309            }
    290             else if (strcmp(a->argv[i], "floppy") == 0)
     310            else if (!strcmp(a->argv[i], "floppy"))
    291311            {
    292312                bootDevice[n - 1] = DeviceType_Floppy;
    293313            }
    294             else if (strcmp(a->argv[i], "dvd") == 0)
     314            else if (!strcmp(a->argv[i], "dvd"))
    295315            {
    296316                bootDevice[n - 1] = DeviceType_DVD;
    297317            }
    298             else if (strcmp(a->argv[i], "disk") == 0)
     318            else if (!strcmp(a->argv[i], "disk"))
    299319            {
    300320                bootDevice[n - 1] = DeviceType_HardDisk;
    301321            }
    302             else if (strcmp(a->argv[i], "net") == 0)
     322            else if (!strcmp(a->argv[i], "net"))
    303323            {
    304324                bootDevice[n - 1] = DeviceType_Network;
     
    309329            bootDeviceChanged[n - 1] = true;
    310330        }
    311         else if (strcmp(a->argv[i], "-hda") == 0)
     331        else if (   !strcmp(a->argv[i], "--hda")
     332                 || !strcmp(a->argv[i], "-hda"))
    312333        {
    313334            if (a->argc <= i + 1)
     
    316337            hdds[0] = a->argv[i];
    317338        }
    318         else if (strcmp(a->argv[i], "-hdb") == 0)
     339        else if (   !strcmp(a->argv[i], "--hdb")
     340                 || !strcmp(a->argv[i], "-hdb"))
    319341        {
    320342            if (a->argc <= i + 1)
     
    323345            hdds[1] = a->argv[i];
    324346        }
    325         else if (strcmp(a->argv[i], "-hdd") == 0)
     347        else if (   !strcmp(a->argv[i], "--hdd")
     348                 || !strcmp(a->argv[i], "-hdd"))
    326349        {
    327350            if (a->argc <= i + 1)
     
    330353            hdds[2] = a->argv[i];
    331354        }
    332         else if (strcmp(a->argv[i], "-dvd") == 0)
     355        else if (   !strcmp(a->argv[i], "--dvd")
     356                 || !strcmp(a->argv[i], "-dvd"))
    333357        {
    334358            if (a->argc <= i + 1)
     
    337361            dvd = a->argv[i];
    338362        }
    339         else if (strcmp(a->argv[i], "-dvdpassthrough") == 0)
     363        else if (   !strcmp(a->argv[i], "--dvdpassthrough")
     364                 || !strcmp(a->argv[i], "-dvdpassthrough"))
    340365        {
    341366            if (a->argc <= i + 1)
     
    344369            dvdpassthrough = a->argv[i];
    345370        }
    346         else if (strcmp(a->argv[i], "-idecontroller") == 0)
     371        else if (   !strcmp(a->argv[i], "--idecontroller")
     372                 || !strcmp(a->argv[i], "-idecontroller"))
    347373        {
    348374            if (a->argc <= i + 1)
     
    351377            idecontroller = a->argv[i];
    352378        }
    353         else if (strcmp(a->argv[i], "-floppy") == 0)
     379        else if (   !strcmp(a->argv[i], "--floppy")
     380                 || !strcmp(a->argv[i], "-floppy"))
    354381        {
    355382            if (a->argc <= i + 1)
     
    358385            floppy = a->argv[i];
    359386        }
    360         else if (strcmp(a->argv[i], "-audio") == 0)
     387        else if (   !strcmp(a->argv[i], "--audio")
     388                 || !strcmp(a->argv[i], "-audio"))
    361389        {
    362390            if (a->argc <= i + 1)
     
    365393            audio = a->argv[i];
    366394        }
    367         else if (strcmp(a->argv[i], "-audiocontroller") == 0)
     395        else if (   !strcmp(a->argv[i], "--audiocontroller")
     396                 || !strcmp(a->argv[i], "-audiocontroller"))
    368397        {
    369398            if (a->argc <= i + 1)
     
    372401            audiocontroller = a->argv[i];
    373402        }
    374         else if (strcmp(a->argv[i], "-clipboard") == 0)
     403        else if (   !strcmp(a->argv[i], "--clipboard")
     404                 || !strcmp(a->argv[i], "-clipboard"))
    375405        {
    376406            if (a->argc <= i + 1)
     
    379409            clipboard = a->argv[i];
    380410        }
    381         else if (strncmp(a->argv[i], "-cableconnected", 15) == 0)
    382         {
    383             unsigned n = parseNum(&a->argv[i][15], NetworkAdapterCount, "NIC");
     411        else if (   !strncmp(a->argv[i], "--cableconnected", 16)
     412                 || !strncmp(a->argv[i], "-cableconnected", 15))
     413        {
     414            unsigned n = parseNum(&a->argv[i][15 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    384415            if (!n)
    385416                return 1;
     
    391422            i++;
    392423        }
    393         /* watch for the right order of these -nic* comparisons! */
    394         else if (strncmp(a->argv[i], "-nictracefile", 13) == 0)
    395         {
    396             unsigned n = parseNum(&a->argv[i][13], NetworkAdapterCount, "NIC");
     424        /* watch for the right order of these --nic* comparisons! */
     425        else if (   !strncmp(a->argv[i], "--nictracefile", 14)
     426                 || !strncmp(a->argv[i], "-nictracefile", 13))
     427        {
     428            unsigned n = parseNum(&a->argv[i][13 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    397429            if (!n)
    398430                return 1;
     
    404436            i++;
    405437        }
    406         else if (strncmp(a->argv[i], "-nictrace", 9) == 0)
    407         {
    408             unsigned n = parseNum(&a->argv[i][9], NetworkAdapterCount, "NIC");
     438        else if (   !strncmp(a->argv[i], "--nictrace", 10)
     439                 || !strncmp(a->argv[i], "-nictrace", 9))
     440        {
     441            unsigned n = parseNum(&a->argv[i][9 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    409442            if (!n)
    410443                return 1;
     
    414447            i++;
    415448        }
    416         else if (strncmp(a->argv[i], "-nictype", 8) == 0)
    417         {
    418             unsigned n = parseNum(&a->argv[i][8], NetworkAdapterCount, "NIC");
     449        else if (   !strncmp(a->argv[i], "--nictype", 9)
     450                 || !strncmp(a->argv[i], "-nictype", 8))
     451        {
     452            unsigned n = parseNum(&a->argv[i][8 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    419453            if (!n)
    420454                return 1;
     
    424458            i++;
    425459        }
    426         else if (strncmp(a->argv[i], "-nicspeed", 9) == 0)
    427         {
    428             unsigned n = parseNum(&a->argv[i][9], NetworkAdapterCount, "NIC");
     460        else if (   !strncmp(a->argv[i], "--nicspeed", 10)
     461                 || !strncmp(a->argv[i], "-nicspeed", 9))
     462        {
     463            unsigned n = parseNum(&a->argv[i][9 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    429464            if (!n)
    430465                return 1;
     
    434469            i++;
    435470        }
    436         else if (strncmp(a->argv[i], "-nic", 4) == 0)
    437         {
    438             unsigned n = parseNum(&a->argv[i][4], NetworkAdapterCount, "NIC");
     471        else if (   !strncmp(a->argv[i], "--nic", 5)
     472                 || !strncmp(a->argv[i], "-nic", 4))
     473        {
     474            unsigned n = parseNum(&a->argv[i][4 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    439475            if (!n)
    440476                return 1;
     
    444480            i++;
    445481        }
    446         else if (strncmp(a->argv[i], "-hostifdev", 10) == 0) /* backward compatibility */
    447         {
    448             unsigned n = parseNum(&a->argv[i][10], NetworkAdapterCount, "NIC");
     482        else if (   !strncmp(a->argv[i], "--hostifdev", 11)
     483                 || !strncmp(a->argv[i], "-hostifdev", 10)) /* backward compatibility */
     484        {
     485            unsigned n = parseNum(&a->argv[i][10 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    449486            if (!n)
    450487                return 1;
     
    454491            i++;
    455492        }
    456         else if (  strncmp(a->argv[i], "-bridgeadapter", 14) == 0)
    457         {
    458             unsigned n = parseNum(&a->argv[i][14], NetworkAdapterCount, "NIC");
     493        else if (   !strncmp(a->argv[i], "--bridgeadapter", 15)
     494                 || !strncmp(a->argv[i], "-bridgeadapter", 14))
     495        {
     496            unsigned n = parseNum(&a->argv[i][14 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    459497            if (!n)
    460498                return 1;
     
    465503        }
    466504#if defined(VBOX_WITH_NETFLT)
    467         else if (strncmp(a->argv[i], "-hostonlyadapter", 16) == 0)
    468         {
    469             unsigned n = parseNum(&a->argv[i][16], NetworkAdapterCount, "NIC");
     505        else if (   !strncmp(a->argv[i], "--hostonlyadapter", 17)
     506                 || !strncmp(a->argv[i], "-hostonlyadapter", 16))
     507        {
     508            unsigned n = parseNum(&a->argv[i][16 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    470509            if (!n)
    471510                return 1;
     
    476515        }
    477516#endif
    478         else if (strncmp(a->argv[i], "-intnet", 7) == 0)
    479         {
    480             unsigned n = parseNum(&a->argv[i][7], NetworkAdapterCount, "NIC");
     517        else if (   !strncmp(a->argv[i], "--intnet", 8)
     518                 || !strncmp(a->argv[i], "-intnet", 7))
     519        {
     520            unsigned n = parseNum(&a->argv[i][7 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    481521            if (!n)
    482522                return 1;
     
    486526            i++;
    487527        }
    488         else if (strncmp(a->argv[i], "-natnet", 7) == 0)
    489         {
    490             unsigned n = parseNum(&a->argv[i][7], NetworkAdapterCount, "NIC");
     528        else if (   !strncmp(a->argv[i], "--natnet", 8)
     529                 || !strncmp(a->argv[i], "-natnet", 7))
     530        {
     531            unsigned n = parseNum(&a->argv[i][7 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    491532            if (!n)
    492533                return 1;
     
    509550            i++;
    510551        }
    511         else if (strncmp(a->argv[i], "-macaddress", 11) == 0)
    512         {
    513             unsigned n = parseNum(&a->argv[i][11], NetworkAdapterCount, "NIC");
     552        else if (   !strncmp(a->argv[i], "--macaddress", 12)
     553                 || !strncmp(a->argv[i], "-macaddress", 11))
     554        {
     555            unsigned n = parseNum(&a->argv[i][11 + (a->argv[i][1] == '-')], NetworkAdapterCount, "NIC");
    514556            if (!n)
    515557                return 1;
     
    520562        }
    521563#ifdef VBOX_WITH_VRDP
    522         else if (strcmp(a->argv[i], "-vrdp") == 0)
     564        else if (   !strcmp(a->argv[i], "--vrdp")
     565                 || !strcmp(a->argv[i], "-vrdp"))
    523566        {
    524567            if (a->argc <= i + 1)
     
    527570            vrdp = a->argv[i];
    528571        }
    529         else if (strcmp(a->argv[i], "-vrdpport") == 0)
    530         {
    531             if (a->argc <= i + 1)
    532                 return errorArgument("Missing argument to '%s'", a->argv[i]);
    533             i++;
    534             if (strcmp(a->argv[i], "default") == 0)
     572        else if (   !strcmp(a->argv[i], "--vrdpport")
     573                 || !strcmp(a->argv[i], "-vrdpport"))
     574        {
     575            if (a->argc <= i + 1)
     576                return errorArgument("Missing argument to '%s'", a->argv[i]);
     577            i++;
     578            if (!strcmp(a->argv[i], "default"))
    535579                vrdpport = 0;
    536580            else
    537581                vrdpport = RTStrToUInt16(a->argv[i]);
    538582        }
    539         else if (strcmp(a->argv[i], "-vrdpaddress") == 0)
     583        else if (   !strcmp(a->argv[i], "--vrdpaddress")
     584                 || !strcmp(a->argv[i], "-vrdpaddress"))
    540585        {
    541586            if (a->argc <= i + 1)
     
    544589            vrdpaddress = a->argv[i];
    545590        }
    546         else if (strcmp(a->argv[i], "-vrdpauthtype") == 0)
     591        else if (   !strcmp(a->argv[i], "--vrdpauthtype")
     592                 || !strcmp(a->argv[i], "-vrdpauthtype"))
    547593        {
    548594            if (a->argc <= i + 1)
     
    551597            vrdpauthtype = a->argv[i];
    552598        }
    553         else if (strcmp(a->argv[i], "-vrdpmulticon") == 0)
     599        else if (   !strcmp(a->argv[i], "--vrdpmulticon")
     600                 || !strcmp(a->argv[i], "-vrdpmulticon"))
    554601        {
    555602            if (a->argc <= i + 1)
     
    558605            vrdpmulticon = a->argv[i];
    559606        }
    560         else if (strcmp(a->argv[i], "-vrdpreusecon") == 0)
     607        else if (   !strcmp(a->argv[i], "--vrdpreusecon")
     608                 || !strcmp(a->argv[i], "-vrdpreusecon"))
    561609        {
    562610            if (a->argc <= i + 1)
     
    566614        }
    567615#endif /* VBOX_WITH_VRDP */
    568         else if (strcmp(a->argv[i], "-usb") == 0)
    569         {
    570             if (a->argc <= i + 1)
    571                 return errorArgument("Missing argument to '%s'", a->argv[i]);
    572             i++;
    573             if (strcmp(a->argv[i], "on") == 0 || strcmp(a->argv[i], "enable") == 0)
     616        else if (   !strcmp(a->argv[i], "--usb")
     617                 || !strcmp(a->argv[i], "-usb"))
     618        {
     619            if (a->argc <= i + 1)
     620                return errorArgument("Missing argument to '%s'", a->argv[i]);
     621            i++;
     622            if (!strcmp(a->argv[i], "on") || !strcmp(a->argv[i], "enable"))
    574623                fUsbEnabled = 1;
    575             else if (strcmp(a->argv[i], "off") == 0 || strcmp(a->argv[i], "disable") == 0)
     624            else if (!strcmp(a->argv[i], "off") || !strcmp(a->argv[i], "disable"))
    576625                fUsbEnabled = 0;
    577626            else
    578                 return errorArgument("Invalid -usb argument '%s'", a->argv[i]);
    579         }
    580         else if (strcmp(a->argv[i], "-usbehci") == 0)
    581         {
    582             if (a->argc <= i + 1)
    583                 return errorArgument("Missing argument to '%s'", a->argv[i]);
    584             i++;
    585             if (strcmp(a->argv[i], "on") == 0 || strcmp(a->argv[i], "enable") == 0)
     627                return errorArgument("Invalid --usb argument '%s'", a->argv[i]);
     628        }
     629        else if (   !strcmp(a->argv[i], "--usbehci")
     630                 || !strcmp(a->argv[i], "-usbehci"))
     631        {
     632            if (a->argc <= i + 1)
     633                return errorArgument("Missing argument to '%s'", a->argv[i]);
     634            i++;
     635            if (!strcmp(a->argv[i], "on") || !strcmp(a->argv[i], "enable"))
    586636                fUsbEhciEnabled = 1;
    587             else if (strcmp(a->argv[i], "off") == 0 || strcmp(a->argv[i], "disable") == 0)
     637            else if (!strcmp(a->argv[i], "off") || !strcmp(a->argv[i], "disable"))
    588638                fUsbEhciEnabled = 0;
    589639            else
    590                 return errorArgument("Invalid -usbehci argument '%s'", a->argv[i]);
    591         }
    592         else if (strcmp(a->argv[i], "-snapshotfolder") == 0)
     640                return errorArgument("Invalid --usbehci argument '%s'", a->argv[i]);
     641        }
     642        else if (   !strcmp(a->argv[i], "--snapshotfolder")
     643                 || !strcmp(a->argv[i], "-snapshotfolder"))
    593644        {
    594645            if (a->argc <= i + 1)
     
    597648            snapshotFolder = a->argv[i];
    598649        }
    599         else if (strncmp(a->argv[i], "-uartmode", 9) == 0)
    600         {
    601             unsigned n = parseNum(&a->argv[i][9], SerialPortCount, "UART");
     650        else if (   !strncmp(a->argv[i], "--uartmode", 10)
     651                 || !strncmp(a->argv[i], "-uartmode", 9))
     652        {
     653            unsigned n = parseNum(&a->argv[i][9 + (a->argv[i][1] == '-')], SerialPortCount, "UART");
    602654            if (!n)
    603655                return 1;
    604656            i++;
    605             if (strcmp(a->argv[i], "disconnected") == 0)
     657            if (!strcmp(a->argv[i], "disconnected"))
    606658            {
    607659                uarts_mode[n - 1] = a->argv[i];
     
    609661            else
    610662            {
    611                 if (strcmp(a->argv[i], "server") == 0 || strcmp(a->argv[i], "client") == 0)
     663                if (!strcmp(a->argv[i], "server") || !strcmp(a->argv[i], "client"))
    612664                {
    613665                    uarts_mode[n - 1] = a->argv[i];
    614666                    i++;
    615667#ifdef RT_OS_WINDOWS
    616                     if (strncmp(a->argv[i], "\\\\.\\pipe\\", 9))
     668                    if (!strncmp(a->argv[i], "\\\\.\\pipe\\", 9))
    617669                        return errorArgument("Uart pipe must start with \\\\.\\pipe\\");
    618670#endif
     
    623675                }
    624676                if (a->argc <= i)
    625                     return errorArgument("Missing argument to -uartmode");
     677                    return errorArgument("Missing argument to --uartmode");
    626678                uarts_path[n - 1] = a->argv[i];
    627679            }
    628680        }
    629         else if (strncmp(a->argv[i], "-uart", 5) == 0)
    630         {
    631             unsigned n = parseNum(&a->argv[i][5], SerialPortCount, "UART");
     681        else if (   !strncmp(a->argv[i], "--uart", 6)
     682                 || !strncmp(a->argv[i], "-uart", 5))
     683        {
     684            unsigned n = parseNum(&a->argv[i][5 + (a->argv[i][1] == '-')], SerialPortCount, "UART");
    632685            if (!n)
    633686                return 1;
     
    635688                return errorArgument("Missing argument to '%s'", a->argv[i]);
    636689            i++;
    637             if (strcmp(a->argv[i], "off") == 0 || strcmp(a->argv[i], "disable") == 0)
     690            if (!strcmp(a->argv[i], "off") || !strcmp(a->argv[i], "disable"))
    638691            {
    639692                uarts_base[n - 1] = (ULONG)-1;
     
    657710        }
    658711#ifdef VBOX_WITH_MEM_BALLOONING
    659         else if (strncmp(a->argv[i], "-guestmemoryballoon", 19) == 0)
     712        else if (   !strcmp(a->argv[i], "--guestmemoryballoon")
     713                 || !strcmp(a->argv[i], "-guestmemoryballoon"))
    660714        {
    661715            if (a->argc <= i + 1)
     
    670724        }
    671725#endif
    672         else if (strncmp(a->argv[i], "-gueststatisticsinterval", 24) == 0)
     726        else if (   !strcmp(a->argv[i], "--gueststatisticsinterval")
     727                 || !strcmp(a->argv[i], "-gueststatisticsinterval"))
    673728        {
    674729            if (a->argc <= i + 1)
     
    682737            guestStatInterval = uVal;
    683738        }
    684         else if (strcmp(a->argv[i], "-sata") == 0)
    685         {
    686             if (a->argc <= i + 1)
    687                 return errorArgument("Missing argument to '%s'", a->argv[i]);
    688             i++;
    689             if (strcmp(a->argv[i], "on") == 0 || strcmp(a->argv[i], "enable") == 0)
     739        else if (   !strcmp(a->argv[i], "--sata")
     740                 || !strcmp(a->argv[i], "-sata"))
     741        {
     742            if (a->argc <= i + 1)
     743                return errorArgument("Missing argument to '%s'", a->argv[i]);
     744            i++;
     745            if (!strcmp(a->argv[i], "on") || !strcmp(a->argv[i], "enable"))
    690746                fSataEnabled = 1;
    691             else if (strcmp(a->argv[i], "off") == 0 || strcmp(a->argv[i], "disable") == 0)
     747            else if (!strcmp(a->argv[i], "off") || !strcmp(a->argv[i], "disable"))
    692748                fSataEnabled = 0;
    693749            else
    694                 return errorArgument("Invalid -usb argument '%s'", a->argv[i]);
    695         }
    696         else if (strcmp(a->argv[i], "-sataportcount") == 0)
     750                return errorArgument("Invalid --usb argument '%s'", a->argv[i]);
     751        }
     752        else if (   !strcmp(a->argv[i], "--sataportcount")
     753                 || !strcmp(a->argv[i], "-sataportcount"))
    697754        {
    698755            unsigned n;
     
    707764            sataPortCount = n;
    708765        }
    709         else if (strncmp(a->argv[i], "-sataport", 9) == 0)
    710         {
    711             unsigned n = parseNum(&a->argv[i][9], 30, "SATA");
     766        else if (   !strncmp(a->argv[i], "--sataport", 10)
     767                 || !strncmp(a->argv[i], "-sataport", 9))
     768        {
     769            unsigned n = parseNum(&a->argv[i][9 + (a->argv[i][1] == '-')], 30, "SATA");
    712770            if (!n)
    713771                return 1;
     
    717775            hdds[n-1+4] = a->argv[i];
    718776        }
    719         else if (strncmp(a->argv[i], "-sataideemulation", 17) == 0)
     777        else if (   !strncmp(a->argv[i], "--sataideemulation", 18)
     778                 || !strncmp(a->argv[i], "-sataideemulation", 17))
    720779        {
    721780            unsigned bootDevicePos = 0;
    722781            unsigned n;
    723782
    724             bootDevicePos = parseNum(&a->argv[i][17], 4, "SATA");
     783            bootDevicePos = parseNum(&a->argv[i][17 + (a->argv[i][1] == '-')], 4, "SATA");
    725784            if (!bootDevicePos)
    726785                return 1;
     
    737796            sataBootDevices[bootDevicePos] = n-1;
    738797        }
    739         else if (strcmp(a->argv[i], "-scsi") == 0)
    740         {
    741             if (a->argc <= i + 1)
    742                 return errorArgument("Missing argument to '%s'", a->argv[i]);
    743             i++;
    744             if (strcmp(a->argv[i], "on") == 0 || strcmp(a->argv[i], "enable") == 0)
     798        else if (   !strcmp(a->argv[i], "--scsi")
     799                 || !strcmp(a->argv[i], "-scsi"))
     800        {
     801            if (a->argc <= i + 1)
     802                return errorArgument("Missing argument to '%s'", a->argv[i]);
     803            i++;
     804            if (!strcmp(a->argv[i], "on") || !strcmp(a->argv[i], "enable"))
    745805                fScsiEnabled = 1;
    746             else if (strcmp(a->argv[i], "off") == 0 || strcmp(a->argv[i], "disable") == 0)
     806            else if (!strcmp(a->argv[i], "off") || !strcmp(a->argv[i], "disable"))
    747807                fScsiEnabled = 0;
    748808            else
    749                 return errorArgument("Invalid -scsi argument '%s'", a->argv[i]);
    750         }
    751         else if (strncmp(a->argv[i], "-scsiport", 9) == 0)
    752         {
    753             unsigned n = parseNum(&a->argv[i][9], 16, "SCSI");
     809                return errorArgument("Invalid --scsi argument '%s'", a->argv[i]);
     810        }
     811        else if (   !strncmp(a->argv[i], "--scsiport", 10)
     812                 || !strncmp(a->argv[i], "-scsiport", 9))
     813        {
     814            unsigned n = parseNum(&a->argv[i][9 + (a->argv[i][1] == '-')], 16, "SCSI");
    754815            if (!n)
    755816                return 1;
     
    759820            hdds[n-1+34] = a->argv[i];
    760821        }
    761         else if (strcmp(a->argv[i], "-scsitype") == 0)
    762         {
    763             if (a->argc <= i + 1)
    764                 return errorArgument("Missing argument to '%s'", a->argv[i]);
    765             i++;
    766             if (strcmp(a->argv[i], "LsiLogic") == 0 )
     822        else if (   !strcmp(a->argv[i], "--scsitype")
     823                 || !strcmp(a->argv[i], "-scsitype"))
     824        {
     825            if (a->argc <= i + 1)
     826                return errorArgument("Missing argument to '%s'", a->argv[i]);
     827            i++;
     828            if (!RTStrICmp(a->argv[i], "LsiLogic"))
    767829                fScsiLsiLogic = 1;
    768             else if (strcmp(a->argv[i], "BusLogic") == 0)
     830            else if (!RTStrICmp(a->argv[i], "BusLogic"))
    769831                fScsiLsiLogic = 0;
    770832            else
    771                 return errorArgument("Invalid -scsitype argument '%s'", a->argv[i]);
     833                return errorArgument("Invalid --scsitype argument '%s'", a->argv[i]);
    772834        }
    773835        else
     
    825887        if (acpi)
    826888        {
    827             if (strcmp(acpi, "on") == 0)
     889            if (!strcmp(acpi, "on"))
    828890            {
    829891                CHECK_ERROR(biosSettings, COMSETTER(ACPIEnabled)(true));
    830892            }
    831             else if (strcmp(acpi, "off") == 0)
     893            else if (!strcmp(acpi, "off"))
    832894            {
    833895                CHECK_ERROR(biosSettings, COMSETTER(ACPIEnabled)(false));
     
    835897            else
    836898            {
    837                 errorArgument("Invalid -acpi argument '%s'", acpi);
     899                errorArgument("Invalid --acpi argument '%s'", acpi);
    838900                rc = E_FAIL;
    839901                break;
     
    842904        if (ioapic)
    843905        {
    844             if (strcmp(ioapic, "on") == 0)
     906            if (!strcmp(ioapic, "on"))
    845907            {
    846908                CHECK_ERROR(biosSettings, COMSETTER(IOAPICEnabled)(true));
    847909            }
    848             else if (strcmp(ioapic, "off") == 0)
     910            else if (!strcmp(ioapic, "off"))
    849911            {
    850912                CHECK_ERROR(biosSettings, COMSETTER(IOAPICEnabled)(false));
     
    852914            else
    853915            {
    854                 errorArgument("Invalid -ioapic argument '%s'", ioapic);
     916                errorArgument("Invalid --ioapic argument '%s'", ioapic);
    855917                rc = E_FAIL;
    856918                break;
     
    859921        if (hwvirtex)
    860922        {
    861             if (strcmp(hwvirtex, "on") == 0)
     923            if (!strcmp(hwvirtex, "on"))
    862924            {
    863925                CHECK_ERROR(machine, COMSETTER(HWVirtExEnabled)(TSBool_True));
    864926            }
    865             else if (strcmp(hwvirtex, "off") == 0)
     927            else if (!strcmp(hwvirtex, "off"))
    866928            {
    867929                CHECK_ERROR(machine, COMSETTER(HWVirtExEnabled)(TSBool_False));
    868930            }
    869             else if (strcmp(hwvirtex, "default") == 0)
     931            else if (!strcmp(hwvirtex, "default"))
    870932            {
    871933                CHECK_ERROR(machine, COMSETTER(HWVirtExEnabled)(TSBool_Default));
     
    873935            else
    874936            {
    875                 errorArgument("Invalid -hwvirtex argument '%s'", hwvirtex);
     937                errorArgument("Invalid --hwvirtex argument '%s'", hwvirtex);
    876938                rc = E_FAIL;
    877939                break;
     
    880942        if (nestedpaging)
    881943        {
    882             if (strcmp(nestedpaging, "on") == 0)
     944            if (!strcmp(nestedpaging, "on"))
    883945            {
    884946                CHECK_ERROR(machine, COMSETTER(HWVirtExNestedPagingEnabled)(true));
    885947            }
    886             else if (strcmp(nestedpaging, "off") == 0)
     948            else if (!strcmp(nestedpaging, "off"))
    887949            {
    888950                CHECK_ERROR(machine, COMSETTER(HWVirtExNestedPagingEnabled)(false));
     
    890952            else
    891953            {
    892                 errorArgument("Invalid -nestedpaging argument '%s'", ioapic);
     954                errorArgument("Invalid --nestedpaging argument '%s'", ioapic);
    893955                rc = E_FAIL;
    894956                break;
     
    897959        if (vtxvpid)
    898960        {
    899             if (strcmp(vtxvpid, "on") == 0)
     961            if (!strcmp(vtxvpid, "on"))
    900962            {
    901963                CHECK_ERROR(machine, COMSETTER(HWVirtExVPIDEnabled)(true));
    902964            }
    903             else if (strcmp(vtxvpid, "off") == 0)
     965            else if (!strcmp(vtxvpid, "off"))
    904966            {
    905967                CHECK_ERROR(machine, COMSETTER(HWVirtExVPIDEnabled)(false));
     
    907969            else
    908970            {
    909                 errorArgument("Invalid -vtxvpid argument '%s'", ioapic);
     971                errorArgument("Invalid --vtxvpid argument '%s'", ioapic);
    910972                rc = E_FAIL;
    911973                break;
     
    914976        if (pae)
    915977        {
    916             if (strcmp(pae, "on") == 0)
     978            if (!strcmp(pae, "on"))
    917979            {
    918980                CHECK_ERROR(machine, COMSETTER(PAEEnabled)(true));
    919981            }
    920             else if (strcmp(pae, "off") == 0)
     982            else if (!strcmp(pae, "off"))
    921983            {
    922984                CHECK_ERROR(machine, COMSETTER(PAEEnabled)(false));
     
    924986            else
    925987            {
    926                 errorArgument("Invalid -pae argument '%s'", ioapic);
     988                errorArgument("Invalid --pae argument '%s'", ioapic);
    927989                rc = E_FAIL;
    928990                break;
     
    935997        if (accelerate3d)
    936998        {
    937             if (strcmp(accelerate3d, "on") == 0)
     999            if (!strcmp(accelerate3d, "on"))
    9381000            {
    9391001                CHECK_ERROR(machine, COMSETTER(Accelerate3DEnabled)(true));
    9401002            }
    941             else if (strcmp(accelerate3d, "off") == 0)
     1003            else if (!strcmp(accelerate3d, "off"))
    9421004            {
    9431005                CHECK_ERROR(machine, COMSETTER(Accelerate3DEnabled)(false));
     
    9451007            else
    9461008            {
    947                 errorArgument("Invalid -accelerate3d argument '%s'", ioapic);
     1009                errorArgument("Invalid --accelerate3d argument '%s'", ioapic);
    9481010                rc = E_FAIL;
    9491011                break;
     
    9521014        if (bioslogofadein)
    9531015        {
    954             if (strcmp(bioslogofadein, "on") == 0)
     1016            if (!strcmp(bioslogofadein, "on"))
    9551017            {
    9561018                CHECK_ERROR(biosSettings, COMSETTER(LogoFadeIn)(true));
    9571019            }
    958             else if (strcmp(bioslogofadein, "off") == 0)
     1020            else if (!strcmp(bioslogofadein, "off"))
    9591021            {
    9601022                CHECK_ERROR(biosSettings, COMSETTER(LogoFadeIn)(false));
     
    9621024            else
    9631025            {
    964                 errorArgument("Invalid -bioslogofadein argument '%s'", bioslogofadein);
     1026                errorArgument("Invalid --bioslogofadein argument '%s'", bioslogofadein);
    9651027                rc = E_FAIL;
    9661028                break;
     
    9691031        if (bioslogofadeout)
    9701032        {
    971             if (strcmp(bioslogofadeout, "on") == 0)
     1033            if (!strcmp(bioslogofadeout, "on"))
    9721034            {
    9731035                CHECK_ERROR(biosSettings, COMSETTER(LogoFadeOut)(true));
    9741036            }
    975             else if (strcmp(bioslogofadeout, "off") == 0)
     1037            else if (!strcmp(bioslogofadeout, "off"))
    9761038            {
    9771039                CHECK_ERROR(biosSettings, COMSETTER(LogoFadeOut)(false));
     
    9791041            else
    9801042            {
    981                 errorArgument("Invalid -bioslogofadeout argument '%s'", bioslogofadeout);
     1043                errorArgument("Invalid --bioslogofadeout argument '%s'", bioslogofadeout);
    9821044                rc = E_FAIL;
    9831045                break;
     
    9941056        if (biosbootmenumode)
    9951057        {
    996             if (strcmp(biosbootmenumode, "disabled") == 0)
     1058            if (!strcmp(biosbootmenumode, "disabled"))
    9971059                CHECK_ERROR(biosSettings, COMSETTER(BootMenuMode)(BIOSBootMenuMode_Disabled));
    998             else if (strcmp(biosbootmenumode, "menuonly") == 0)
     1060            else if (!strcmp(biosbootmenumode, "menuonly"))
    9991061                CHECK_ERROR(biosSettings, COMSETTER(BootMenuMode)(BIOSBootMenuMode_MenuOnly));
    1000             else if (strcmp(biosbootmenumode, "messageandmenu") == 0)
     1062            else if (!strcmp(biosbootmenumode, "messageandmenu"))
    10011063                CHECK_ERROR(biosSettings, COMSETTER(BootMenuMode)(BIOSBootMenuMode_MessageAndMenu));
    10021064            else
    10031065            {
    1004                 errorArgument("Invalid -biosbootmenu argument '%s'", biosbootmenumode);
     1066                errorArgument("Invalid --biosbootmenu argument '%s'", biosbootmenumode);
    10051067                rc = E_FAIL;
    10061068                break;
     
    10151077        if (biospxedebug)
    10161078        {
    1017             if (strcmp(biospxedebug, "on") == 0)
     1079            if (!strcmp(biospxedebug, "on"))
    10181080            {
    10191081                CHECK_ERROR(biosSettings, COMSETTER(PXEDebugEnabled)(true));
    10201082            }
    1021             else if (strcmp(biospxedebug, "off") == 0)
     1083            else if (!strcmp(biospxedebug, "off"))
    10221084            {
    10231085                CHECK_ERROR(biosSettings, COMSETTER(PXEDebugEnabled)(false));
     
    10251087            else
    10261088            {
    1027                 errorArgument("Invalid -biospxedebug argument '%s'", biospxedebug);
     1089                errorArgument("Invalid --biospxedebug argument '%s'", biospxedebug);
    10281090                rc = E_FAIL;
    10291091                break;
     
    10371099        if (hdds[0])
    10381100        {
    1039             if (strcmp(hdds[0], "none") == 0)
     1101            if (!strcmp(hdds[0], "none"))
    10401102            {
    10411103                machine->DetachHardDisk(Bstr("IDE"), 0, 0);
     
    10701132        if (hdds[1])
    10711133        {
    1072             if (strcmp(hdds[1], "none") == 0)
     1134            if (!strcmp(hdds[1], "none"))
    10731135            {
    10741136                machine->DetachHardDisk(Bstr("IDE"), 0, 1);
     
    11031165        if (hdds[2])
    11041166        {
    1105             if (strcmp(hdds[2], "none") == 0)
     1167            if (!strcmp(hdds[2], "none"))
    11061168            {
    11071169                machine->DetachHardDisk(Bstr("IDE"), 1, 1);
     
    11411203
    11421204            /* unmount? */
    1143             if (strcmp(dvd, "none") == 0)
     1205            if (!strcmp(dvd, "none"))
    11441206            {
    11451207                CHECK_ERROR(dvdDrive, Unmount());
    11461208            }
    11471209            /* host drive? */
    1148             else if (strncmp(dvd, "host:", 5) == 0)
     1210            else if (!strncmp(dvd, "host:", 5))
    11491211            {
    11501212                ComPtr<IHost> host;
     
    12081270            ASSERT(dvdDrive);
    12091271
    1210             CHECK_ERROR(dvdDrive, COMSETTER(Passthrough)(strcmp(dvdpassthrough, "on") == 0));
     1272            CHECK_ERROR(dvdDrive, COMSETTER(Passthrough)(!strcmp(dvdpassthrough, "on")));
    12111273        }
    12121274        if (idecontroller)
     
    12151277            CHECK_ERROR(machine, GetStorageControllerByName(Bstr("IDE"), storageController.asOutParam()));
    12161278
    1217             if (RTStrICmp(idecontroller, "PIIX3") == 0)
     1279            if (RTStrICmp(idecontroller, "PIIX3"))
    12181280            {
    12191281                CHECK_ERROR(storageController, COMSETTER(ControllerType)(StorageControllerType_PIIX3));
    12201282            }
    1221             else if (RTStrICmp(idecontroller, "PIIX4") == 0)
     1283            else if (RTStrICmp(idecontroller, "PIIX4"))
    12221284            {
    12231285                CHECK_ERROR(storageController, COMSETTER(ControllerType)(StorageControllerType_PIIX4));
    12241286            }
    1225             else if (RTStrICmp(idecontroller, "ICH6") == 0)
     1287            else if (RTStrICmp(idecontroller, "ICH6"))
    12261288            {
    12271289                CHECK_ERROR(storageController, COMSETTER(ControllerType)(StorageControllerType_ICH6));
     
    12291291            else
    12301292            {
    1231                 errorArgument("Invalid -idecontroller argument '%s'", idecontroller);
     1293                errorArgument("Invalid --idecontroller argument '%s'", idecontroller);
    12321294                rc = E_FAIL;
    12331295                break;
     
    12411303
    12421304            /* disable? */
    1243             if (strcmp(floppy, "disabled") == 0)
     1305            if (!strcmp(floppy, "disabled"))
    12441306            {
    12451307                /* disable the controller */
     
    12521314
    12531315                /* unmount? */
    1254                 if (strcmp(floppy, "empty") == 0)
     1316                if (!strcmp(floppy, "empty"))
    12551317                {
    12561318                    CHECK_ERROR(floppyDrive, Unmount());
    12571319                }
    12581320                /* host drive? */
    1259                 else if (strncmp(floppy, "host:", 5) == 0)
     1321                else if (!strncmp(floppy, "host:", 5))
    12601322                {
    12611323                    ComPtr<IHost> host;
     
    13101372            {
    13111373                /* disable? */
    1312                 if (strcmp(audio, "none") == 0)
     1374                if (!strcmp(audio, "none"))
    13131375                {
    13141376                    CHECK_ERROR(audioAdapter, COMSETTER(Enabled)(false));
    13151377                }
    1316                 else if (strcmp(audio, "null") == 0)
     1378                else if (!strcmp(audio, "null"))
    13171379                {
    13181380                    CHECK_ERROR(audioAdapter, COMSETTER(AudioDriver)(AudioDriverType_Null));
     
    13211383#ifdef RT_OS_WINDOWS
    13221384#ifdef VBOX_WITH_WINMM
    1323                 else if (strcmp(audio, "winmm") == 0)
     1385                else if (!strcmp(audio, "winmm"))
    13241386                {
    13251387                    CHECK_ERROR(audioAdapter, COMSETTER(AudioDriver)(AudioDriverType_WinMM));
     
    13271389                }
    13281390#endif
    1329                 else if (strcmp(audio, "dsound") == 0)
     1391                else if (!strcmp(audio, "dsound"))
    13301392                {
    13311393                    CHECK_ERROR(audioAdapter, COMSETTER(AudioDriver)(AudioDriverType_DirectSound));
     
    13341396#endif /* RT_OS_WINDOWS */
    13351397#ifdef RT_OS_LINUX
    1336                 else if (strcmp(audio, "oss") == 0)
     1398                else if (!strcmp(audio, "oss"))
    13371399                {
    13381400                    CHECK_ERROR(audioAdapter, COMSETTER(AudioDriver)(AudioDriverType_OSS));
     
    13401402                }
    13411403# ifdef VBOX_WITH_ALSA
    1342                 else if (strcmp(audio, "alsa") == 0)
     1404                else if (!strcmp(audio, "alsa"))
    13431405                {
    13441406                    CHECK_ERROR(audioAdapter, COMSETTER(AudioDriver)(AudioDriverType_ALSA));
     
    13471409# endif
    13481410# ifdef VBOX_WITH_PULSE
    1349                 else if (strcmp(audio, "pulse") == 0)
     1411                else if (!strcmp(audio, "pulse"))
    13501412                {
    13511413                    CHECK_ERROR(audioAdapter, COMSETTER(AudioDriver)(AudioDriverType_Pulse));
     
    13551417#endif /* !RT_OS_LINUX */
    13561418#ifdef RT_OS_SOLARIS
    1357                 else if (strcmp(audio, "solaudio") == 0)
     1419                else if (!strcmp(audio, "solaudio"))
    13581420                {
    13591421                    CHECK_ERROR(audioAdapter, COMSETTER(AudioDriver)(AudioDriverType_SolAudio));
     
    13631425#endif /* !RT_OS_SOLARIS */
    13641426#ifdef RT_OS_DARWIN
    1365                 else if (strcmp(audio, "coreaudio") == 0)
     1427                else if (!strcmp(audio, "coreaudio"))
    13661428                {
    13671429                    CHECK_ERROR(audioAdapter, COMSETTER(AudioDriver)(AudioDriverType_CoreAudio));
     
    13721434                else
    13731435                {
    1374                     errorArgument("Invalid -audio argument '%s'", audio);
     1436                    errorArgument("Invalid --audio argument '%s'", audio);
    13751437                    rc = E_FAIL;
    13761438                    break;
     
    13791441            if (audiocontroller)
    13801442            {
    1381                 if (strcmp(audiocontroller, "sb16") == 0)
     1443                if (!strcmp(audiocontroller, "sb16"))
    13821444                    CHECK_ERROR(audioAdapter, COMSETTER(AudioController)(AudioControllerType_SB16));
    1383                 else if (strcmp(audiocontroller, "ac97") == 0)
     1445                else if (!strcmp(audiocontroller, "ac97"))
    13841446                    CHECK_ERROR(audioAdapter, COMSETTER(AudioController)(AudioControllerType_AC97));
    13851447                else
    13861448                {
    1387                     errorArgument("Invalid -audiocontroller argument '%s'", audiocontroller);
     1449                    errorArgument("Invalid --audiocontroller argument '%s'", audiocontroller);
    13881450                    rc = E_FAIL;
    13891451                    break;
     
    13981460            ASSERT(clipboardMode);
    13991461*/
    1400             if (strcmp(clipboard, "disabled") == 0)
     1462            if (!strcmp(clipboard, "disabled"))
    14011463            {
    14021464                CHECK_ERROR(machine, COMSETTER(ClipboardMode)(ClipboardMode_Disabled));
    14031465            }
    1404             else if (strcmp(clipboard, "hosttoguest") == 0)
     1466            else if (!strcmp(clipboard, "hosttoguest"))
    14051467            {
    14061468                CHECK_ERROR(machine, COMSETTER(ClipboardMode)(ClipboardMode_HostToGuest));
    14071469            }
    1408             else if (strcmp(clipboard, "guesttohost") == 0)
     1470            else if (!strcmp(clipboard, "guesttohost"))
    14091471            {
    14101472                CHECK_ERROR(machine, COMSETTER(ClipboardMode)(ClipboardMode_GuestToHost));
    14111473            }
    1412             else if (strcmp(clipboard, "bidirectional") == 0)
     1474            else if (!strcmp(clipboard, "bidirectional"))
    14131475            {
    14141476                CHECK_ERROR(machine, COMSETTER(ClipboardMode)(ClipboardMode_Bidirectional));
     
    14161478            else
    14171479            {
    1418                 errorArgument("Invalid -clipboard argument '%s'", clipboard);
     1480                errorArgument("Invalid --clipboard argument '%s'", clipboard);
    14191481                rc = E_FAIL;
    14201482                break;
     
    14321494            if (nics[n])
    14331495            {
    1434                 if (strcmp(nics[n], "none") == 0)
     1496                if (!strcmp(nics[n], "none"))
    14351497                {
    14361498                    CHECK_ERROR_RET(nic, COMSETTER(Enabled) (FALSE), 1);
    14371499                }
    1438                 else if (strcmp(nics[n], "null") == 0)
     1500                else if (!strcmp(nics[n], "null"))
    14391501                {
    14401502                    CHECK_ERROR_RET(nic, COMSETTER(Enabled) (TRUE), 1);
    14411503                    CHECK_ERROR_RET(nic, Detach(), 1);
    14421504                }
    1443                 else if (strcmp(nics[n], "nat") == 0)
     1505                else if (!strcmp(nics[n], "nat"))
    14441506                {
    14451507                    CHECK_ERROR_RET(nic, COMSETTER(Enabled) (TRUE), 1);
    14461508                    CHECK_ERROR_RET(nic, AttachToNAT(), 1);
    14471509                }
    1448                 else if (  strcmp(nics[n], "bridged") == 0
    1449                         || strcmp(nics[n], "hostif") == 0) /* backward compatibility */
     1510                else if (  !strcmp(nics[n], "bridged")
     1511                        || !strcmp(nics[n], "hostif")) /* backward compatibility */
    14501512                {
    14511513                    CHECK_ERROR_RET(nic, COMSETTER(Enabled) (TRUE), 1);
    14521514                    CHECK_ERROR_RET(nic, AttachToBridgedInterface(), 1);
    14531515                }
    1454                 else if (strcmp(nics[n], "intnet") == 0)
     1516                else if (!strcmp(nics[n], "intnet"))
    14551517                {
    14561518                    CHECK_ERROR_RET(nic, COMSETTER(Enabled) (TRUE), 1);
     
    14581520                }
    14591521#if defined(VBOX_WITH_NETFLT)
    1460                 else if (strcmp(nics[n], "hostonly") == 0)
     1522                else if (!strcmp(nics[n], "hostonly"))
    14611523                {
    14621524
     
    14761538            if (nictype[n])
    14771539            {
    1478                 if (strcmp(nictype[n], "Am79C970A") == 0)
     1540                if (!strcmp(nictype[n], "Am79C970A"))
    14791541                {
    14801542                    CHECK_ERROR_RET(nic, COMSETTER(AdapterType)(NetworkAdapterType_Am79C970A), 1);
    14811543                }
    1482                 else if (strcmp(nictype[n], "Am79C973") == 0)
     1544                else if (!strcmp(nictype[n], "Am79C973"))
    14831545                {
    14841546                    CHECK_ERROR_RET(nic, COMSETTER(AdapterType)(NetworkAdapterType_Am79C973), 1);
    14851547                }
    14861548#ifdef VBOX_WITH_E1000
    1487                 else if (strcmp(nictype[n], "82540EM") == 0)
     1549                else if (!strcmp(nictype[n], "82540EM"))
    14881550                {
    14891551                    CHECK_ERROR_RET(nic, COMSETTER(AdapterType)(NetworkAdapterType_I82540EM), 1);
    14901552                }
    1491                 else if (strcmp(nictype[n], "82543GC") == 0)
     1553                else if (!strcmp(nictype[n], "82543GC"))
    14921554                {
    14931555                    CHECK_ERROR_RET(nic, COMSETTER(AdapterType)(NetworkAdapterType_I82543GC), 1);
    14941556                }
    1495                 else if (strcmp(nictype[n], "82545EM") == 0)
     1557                else if (!strcmp(nictype[n], "82545EM"))
    14961558                {
    14971559                    CHECK_ERROR_RET(nic, COMSETTER(AdapterType)(NetworkAdapterType_I82545EM), 1);
     
    15101572            {
    15111573                /* generate one? */
    1512                 if (strcmp(macs[n], "auto") == 0)
     1574                if (!strcmp(macs[n], "auto"))
    15131575                {
    15141576                    CHECK_ERROR_RET(nic, COMSETTER(MACAddress)(NULL), 1);
     
    15291591                if (u32LineSpeed < 1000 || u32LineSpeed > 4000000)
    15301592                {
    1531                     errorArgument("Invalid -nicspeed%lu argument '%s'", n + 1, nicspeed[n]);
     1593                    errorArgument("Invalid --nicspeed%lu argument '%s'", n + 1, nicspeed[n]);
    15321594                    rc = E_FAIL;
    15331595                    break;
     
    15391601            if (cableconnected[n])
    15401602            {
    1541                 if (strcmp(cableconnected[n], "on") == 0)
     1603                if (!strcmp(cableconnected[n], "on"))
    15421604                {
    15431605                    CHECK_ERROR_RET(nic, COMSETTER(CableConnected)(TRUE), 1);
    15441606                }
    1545                 else if (strcmp(cableconnected[n], "off") == 0)
     1607                else if (!strcmp(cableconnected[n], "off"))
    15461608                {
    15471609                    CHECK_ERROR_RET(nic, COMSETTER(CableConnected)(FALSE), 1);
     
    15491611                else
    15501612                {
    1551                     errorArgument("Invalid -cableconnected%lu argument '%s'", n + 1, cableconnected[n]);
     1613                    errorArgument("Invalid --cableconnected%lu argument '%s'", n + 1, cableconnected[n]);
    15521614                    rc = E_FAIL;
    15531615                    break;
     
    15581620            if (nictrace[n])
    15591621            {
    1560                 if (strcmp(nictrace[n], "on") == 0)
     1622                if (!strcmp(nictrace[n], "on"))
    15611623                {
    15621624                    CHECK_ERROR_RET(nic, COMSETTER(TraceEnabled)(TRUE), 1);
    15631625                }
    1564                 else if (strcmp(nictrace[n], "off") == 0)
     1626                else if (!strcmp(nictrace[n], "off"))
    15651627                {
    15661628                    CHECK_ERROR_RET(nic, COMSETTER(TraceEnabled)(FALSE), 1);
     
    15681630                else
    15691631                {
    1570                     errorArgument("Invalid -nictrace%lu argument '%s'", n + 1, nictrace[n]);
     1632                    errorArgument("Invalid --nictrace%lu argument '%s'", n + 1, nictrace[n]);
    15711633                    rc = E_FAIL;
    15721634                    break;
     
    15841646            {
    15851647                /* remove it? */
    1586                 if (strcmp(hostifdev[n], "none") == 0)
     1648                if (!strcmp(hostifdev[n], "none"))
    15871649                {
    15881650                    CHECK_ERROR_RET(nic, COMSETTER(HostInterface)(NULL), 1);
     
    15981660            {
    15991661                /* remove it? */
    1600                 if (strcmp(intnet[n], "none") == 0)
     1662                if (!strcmp(intnet[n], "none"))
    16011663                {
    16021664                    CHECK_ERROR_RET(nic, COMSETTER(InternalNetwork)(NULL), 1);
     
    16391701            if (uarts_mode[n])
    16401702            {
    1641                 if (strcmp(uarts_mode[n], "disconnected") == 0)
     1703                if (!strcmp(uarts_mode[n], "disconnected"))
    16421704                {
    16431705                    CHECK_ERROR_RET(uart, COMSETTER(HostMode) (PortMode_Disconnected), 1);
     
    16461708                {
    16471709                    CHECK_ERROR_RET(uart, COMSETTER(Path) (Bstr(uarts_path[n])), 1);
    1648                     if (strcmp(uarts_mode[n], "server") == 0)
     1710                    if (!strcmp(uarts_mode[n], "server"))
    16491711                    {
    16501712                        CHECK_ERROR_RET(uart, COMSETTER(HostMode) (PortMode_HostPipe), 1);
    16511713                        CHECK_ERROR_RET(uart, COMSETTER(Server) (TRUE), 1);
    16521714                    }
    1653                     else if (strcmp(uarts_mode[n], "client") == 0)
     1715                    else if (!strcmp(uarts_mode[n], "client"))
    16541716                    {
    16551717                        CHECK_ERROR_RET(uart, COMSETTER(HostMode) (PortMode_HostPipe), 1);
     
    16761738                if (vrdp)
    16771739                {
    1678                     if (strcmp(vrdp, "on") == 0)
     1740                    if (!strcmp(vrdp, "on"))
    16791741                    {
    16801742                        CHECK_ERROR(vrdpServer, COMSETTER(Enabled)(true));
    16811743                    }
    1682                     else if (strcmp(vrdp, "off") == 0)
     1744                    else if (!strcmp(vrdp, "off"))
    16831745                    {
    16841746                        CHECK_ERROR(vrdpServer, COMSETTER(Enabled)(false));
     
    16861748                    else
    16871749                    {
    1688                         errorArgument("Invalid -vrdp argument '%s'", vrdp);
     1750                        errorArgument("Invalid --vrdp argument '%s'", vrdp);
    16891751                        rc = E_FAIL;
    16901752                        break;
     
    17011763                if (vrdpauthtype)
    17021764                {
    1703                     if (strcmp(vrdpauthtype, "null") == 0)
     1765                    if (!strcmp(vrdpauthtype, "null"))
    17041766                    {
    17051767                        CHECK_ERROR(vrdpServer, COMSETTER(AuthType)(VRDPAuthType_Null));
    17061768                    }
    1707                     else if (strcmp(vrdpauthtype, "external") == 0)
     1769                    else if (!strcmp(vrdpauthtype, "external"))
    17081770                    {
    17091771                        CHECK_ERROR(vrdpServer, COMSETTER(AuthType)(VRDPAuthType_External));
    17101772                    }
    1711                     else if (strcmp(vrdpauthtype, "guest") == 0)
     1773                    else if (!strcmp(vrdpauthtype, "guest"))
    17121774                    {
    17131775                        CHECK_ERROR(vrdpServer, COMSETTER(AuthType)(VRDPAuthType_Guest));
     
    17151777                    else
    17161778                    {
    1717                         errorArgument("Invalid -vrdpauthtype argument '%s'", vrdpauthtype);
     1779                        errorArgument("Invalid --vrdpauthtype argument '%s'", vrdpauthtype);
    17181780                        rc = E_FAIL;
    17191781                        break;
     
    17221784                if (vrdpmulticon)
    17231785                {
    1724                     if (strcmp(vrdpmulticon, "on") == 0)
     1786                    if (!strcmp(vrdpmulticon, "on"))
    17251787                    {
    17261788                        CHECK_ERROR(vrdpServer, COMSETTER(AllowMultiConnection)(true));
    17271789                    }
    1728                     else if (strcmp(vrdpmulticon, "off") == 0)
     1790                    else if (!strcmp(vrdpmulticon, "off"))
    17291791                    {
    17301792                        CHECK_ERROR(vrdpServer, COMSETTER(AllowMultiConnection)(false));
     
    17321794                    else
    17331795                    {
    1734                         errorArgument("Invalid -vrdpmulticon argument '%s'", vrdpmulticon);
     1796                        errorArgument("Invalid --vrdpmulticon argument '%s'", vrdpmulticon);
    17351797                        rc = E_FAIL;
    17361798                        break;
     
    17391801                if (vrdpreusecon)
    17401802                {
    1741                     if (strcmp(vrdpreusecon, "on") == 0)
     1803                    if (!strcmp(vrdpreusecon, "on"))
    17421804                    {
    17431805                        CHECK_ERROR(vrdpServer, COMSETTER(ReuseSingleConnection)(true));
    17441806                    }
    1745                     else if (strcmp(vrdpreusecon, "off") == 0)
     1807                    else if (!strcmp(vrdpreusecon, "off"))
    17461808                    {
    17471809                        CHECK_ERROR(vrdpServer, COMSETTER(ReuseSingleConnection)(false));
     
    17491811                    else
    17501812                    {
    1751                         errorArgument("Invalid -vrdpreusecon argument '%s'", vrdpreusecon);
     1813                        errorArgument("Invalid --vrdpreusecon argument '%s'", vrdpreusecon);
    17521814                        rc = E_FAIL;
    17531815                        break;
     
    17851847        if (snapshotFolder)
    17861848        {
    1787             if (strcmp(snapshotFolder, "default") == 0)
     1849            if (!strcmp(snapshotFolder, "default"))
    17881850            {
    17891851                CHECK_ERROR(machine, COMSETTER(SnapshotFolder)(NULL));
     
    18201882            if (hdds[i])
    18211883            {
    1822                 if (strcmp(hdds[i], "none") == 0)
     1884                if (!strcmp(hdds[i], "none"))
    18231885                {
    18241886                    machine->DetachHardDisk(Bstr("SATA"), i-4, 0);
     
    18841946            {
    18851947                ComPtr<IStorageController> ctl;
    1886                 if (fScsiLsiLogic == 0)
     1948                if (!fScsiLsiLogic)
    18871949                {
    18881950                    CHECK_ERROR(machine, AddStorageController(Bstr("BusLogic"), StorageBus_SCSI, ctl.asOutParam()));
     
    19071969            if (hdds[i])
    19081970            {
    1909                 if (strcmp(hdds[i], "none") == 0)
     1971                if (!strcmp(hdds[i], "none"))
    19101972                {
    19111973                    rc = machine->DetachHardDisk(Bstr("LsiLogic"), i-34, 0);
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette