VirtualBox

Ignore:
Timestamp:
Feb 16, 2010 4:57:09 PM (15 years ago)
Author:
vboxsync
Message:

Main: Bstr makeover (second attempt) -- make Bstr(NULL) and Bstr() behave the same; resulting cleanup; make some more internal methods use Utf8Str instead of Bstr; fix a lot of CheckComArgNotNull?() usage

Location:
trunk/src/VBox/Frontends/VBoxManage
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Frontends/VBoxManage/VBoxManageDHCPServer.cpp

    r26553 r26587  
    230230
    231231        ComPtr<IHostNetworkInterface> hif;
    232         CHECK_ERROR(host, FindHostNetworkInterfaceByName(Bstr(pIfName).mutableRaw(), hif.asOutParam()));
     232        CHECK_ERROR(host, FindHostNetworkInterfaceByName(Bstr(pIfName), hif.asOutParam()));
    233233        if(FAILED(rc))
    234234            return errorArgument("could not find interface '%s'", pIfName);
    … …  
    244244
    245245    ComPtr<IDHCPServer> svr;
    246     rc = a->virtualBox->FindDHCPServerByNetworkName(NetName.mutableRaw(), svr.asOutParam());
     246    rc = a->virtualBox->FindDHCPServerByNetworkName(NetName, svr.asOutParam());
    247247    if(enmCode == OP_ADD)
    248248    {
    … …  
    250250            return errorArgument("dhcp server already exists");
    251251
    252         CHECK_ERROR(a->virtualBox, CreateDHCPServer(NetName.mutableRaw(), svr.asOutParam()));
     252        CHECK_ERROR(a->virtualBox, CreateDHCPServer(NetName, svr.asOutParam()));
    253253        if(FAILED(rc))
    254254            return errorArgument("failed to create server");
    … …  
    263263        if (pIp || pNetmask || pLowerIp || pUpperIp)
    264264        {
    265             CHECK_ERROR(svr, SetConfiguration (Bstr(pIp).mutableRaw(), Bstr(pNetmask).mutableRaw(), Bstr(pLowerIp).mutableRaw(), Bstr(pUpperIp).mutableRaw()));
     265            CHECK_ERROR(svr, SetConfiguration(Bstr(pIp), Bstr(pNetmask), Bstr(pLowerIp), Bstr(pUpperIp)));
    266266            if(FAILED(rc))
    267267                return errorArgument("failed to set configuration");

  • trunk/src/VBox/Frontends/VBoxManage/VBoxManageDisk.cpp

    r26553 r26587  
    255255
    256256    /* check the outcome */
    257     if (   !filename
     257    if (   filename.isEmpty()
    258258        || sizeMB == 0)
    259259        return errorSyntax(USAGE_CREATEHD, "Parameters --filename and --size are required");
    … …  
    281281        bool doClose = false;
    282282
    283         if (!comment.isNull())
     283        if (!comment.isEmpty())
    284284        {
    285285            CHECK_ERROR(hardDisk,COMSETTER(Description)(comment));
    … …  
    10121012
    10131013    /* check for required options */
    1014     if (!server || !target)
     1014    if (server.isEmpty() || target.isEmpty())
    10151015        return errorSyntax(USAGE_ADDISCSIDISK, "Parameters --server and --target are required");
    10161016
    … …  
    10381038        if (FAILED(rc)) break;
    10391039
    1040         if (!port.isNull())
     1040        if (!port.isEmpty())
    10411041            server = BstrFmt ("%ls:%ls", server.raw(), port.raw());
    10421042
    … …  
    10471047        server.detachTo (values.appendedRaw());
    10481048        Bstr ("TargetName").detachTo (names.appendedRaw());
    1049         target.detachTo (values.appendedRaw());
    1050 
    1051         if (!lun.isNull())
     1049        target.detachTo(values.appendedRaw());
     1050
     1051        if (!lun.isEmpty())
    10521052        {
    10531053            Bstr ("LUN").detachTo (names.appendedRaw());
    10541054            lun.detachTo (values.appendedRaw());
    10551055        }
    1056         if (!username.isNull())
     1056        if (!username.isEmpty())
    10571057        {
    10581058            Bstr ("InitiatorUsername").detachTo (names.appendedRaw());
    10591059            username.detachTo (values.appendedRaw());
    10601060        }
    1061         if (!password.isNull())
     1061        if (!password.isEmpty())
    10621062        {
    10631063            Bstr ("InitiatorSecret").detachTo (names.appendedRaw());

  • trunk/src/VBox/Frontends/VBoxManage/VBoxManageGuestProp.cpp

    r24024 r26587  
    248248            RTPrintf("No value set!\n");
    249249        if (value)
     250        {
    250251            RTPrintf("Value: %lS\n", value.raw());
    251         if (value && verbose)
    252         {
    253             RTPrintf("Timestamp: %lld\n", u64Timestamp);
    254             RTPrintf("Flags: %lS\n", flags.raw());
     252            if (verbose)
     253            {
     254                RTPrintf("Timestamp: %lld\n", u64Timestamp);
     255                RTPrintf("Flags: %lS\n", flags.raw());
     256            }
    255257        }
    256258    }

  • trunk/src/VBox/Frontends/VBoxManage/VBoxManageUSB.cpp

    r26553 r26587  
    364364                        && !cmd.mMachine)
    365365                    || (   cmd.mGlobal
    366                         && cmd.mFilter.mRemote)
     366                        && !cmd.mFilter.mRemote.isEmpty())
    367367                   )
    368368                {
    … …  
    441441                if (!f.mActive.isNull())
    442442                    CHECK_ERROR_BREAK (flt, COMSETTER(Active) (f.mActive));
    443                 if (!f.mVendorId.isNull())
    444                     CHECK_ERROR_BREAK (flt, COMSETTER(VendorId) (f.mVendorId.setNullIfEmpty()));
    445                 if (!f.mProductId.isNull())
    446                     CHECK_ERROR_BREAK (flt, COMSETTER(ProductId) (f.mProductId.setNullIfEmpty()));
    447                 if (!f.mRevision.isNull())
    448                     CHECK_ERROR_BREAK (flt, COMSETTER(Revision) (f.mRevision.setNullIfEmpty()));
    449                 if (!f.mManufacturer.isNull())
    450                     CHECK_ERROR_BREAK (flt, COMSETTER(Manufacturer) (f.mManufacturer.setNullIfEmpty()));
    451                 if (!f.mSerialNumber.isNull())
    452                     CHECK_ERROR_BREAK (flt, COMSETTER(SerialNumber) (f.mSerialNumber.setNullIfEmpty()));
     443                if (!f.mVendorId.isEmpty())
     444                    CHECK_ERROR_BREAK (flt, COMSETTER(VendorId)(f.mVendorId));
     445                if (!f.mProductId.isEmpty())
     446                    CHECK_ERROR_BREAK (flt, COMSETTER(ProductId) (f.mProductId));
     447                if (!f.mRevision.isEmpty())
     448                    CHECK_ERROR_BREAK (flt, COMSETTER(Revision) (f.mRevision));
     449                if (!f.mManufacturer.isEmpty())
     450                    CHECK_ERROR_BREAK (flt, COMSETTER(Manufacturer) (f.mManufacturer));
     451                if (!f.mSerialNumber.isEmpty())
     452                    CHECK_ERROR_BREAK (flt, COMSETTER(SerialNumber) (f.mSerialNumber));
    453453                if (!f.mMaskedInterfaces.isNull())
    454454                    CHECK_ERROR_BREAK (flt, COMSETTER(MaskedInterfaces) (f.mMaskedInterfaces));
    … …  
    466466                if (!f.mActive.isNull())
    467467                    CHECK_ERROR_BREAK (flt, COMSETTER(Active) (f.mActive));
    468                 if (!f.mVendorId.isNull())
    469                     CHECK_ERROR_BREAK (flt, COMSETTER(VendorId) (f.mVendorId.setNullIfEmpty()));
    470                 if (!f.mProductId.isNull())
    471                     CHECK_ERROR_BREAK (flt, COMSETTER(ProductId) (f.mProductId.setNullIfEmpty()));
    472                 if (!f.mRevision.isNull())
    473                     CHECK_ERROR_BREAK (flt, COMSETTER(Revision) (f.mRevision.setNullIfEmpty()));
    474                 if (!f.mManufacturer.isNull())
    475                     CHECK_ERROR_BREAK (flt, COMSETTER(Manufacturer) (f.mManufacturer.setNullIfEmpty()));
    476                 if (!f.mRemote.isNull())
    477                     CHECK_ERROR_BREAK (flt, COMSETTER(Remote) (f.mRemote.setNullIfEmpty()));
    478                 if (!f.mSerialNumber.isNull())
    479                     CHECK_ERROR_BREAK (flt, COMSETTER(SerialNumber) (f.mSerialNumber.setNullIfEmpty()));
     468                if (!f.mVendorId.isEmpty())
     469                    CHECK_ERROR_BREAK (flt, COMSETTER(VendorId) (f.mVendorId));
     470                if (!f.mProductId.isEmpty())
     471                    CHECK_ERROR_BREAK (flt, COMSETTER(ProductId) (f.mProductId));
     472                if (!f.mRevision.isEmpty())
     473                    CHECK_ERROR_BREAK (flt, COMSETTER(Revision) (f.mRevision));
     474                if (!f.mManufacturer.isEmpty())
     475                    CHECK_ERROR_BREAK (flt, COMSETTER(Manufacturer) (f.mManufacturer));
     476                if (!f.mRemote.isEmpty())
     477                    CHECK_ERROR_BREAK (flt, COMSETTER(Remote) (f.mRemote));
     478                if (!f.mSerialNumber.isEmpty())
     479                    CHECK_ERROR_BREAK (flt, COMSETTER(SerialNumber) (f.mSerialNumber));
    480480                if (!f.mMaskedInterfaces.isNull())
    481481                    CHECK_ERROR_BREAK (flt, COMSETTER(MaskedInterfaces) (f.mMaskedInterfaces));
    … …  
    494494                ComPtr <IHostUSBDeviceFilter> flt = coll[cmd.mIndex];
    495495
    496                 if (!f.mName.isNull())
    497                     CHECK_ERROR_BREAK (flt, COMSETTER(Name) (f.mName.setNullIfEmpty()));
     496                if (!f.mName.isEmpty())
     497                    CHECK_ERROR_BREAK (flt, COMSETTER(Name) (f.mName));
    498498                if (!f.mActive.isNull())
    499499                    CHECK_ERROR_BREAK (flt, COMSETTER(Active) (f.mActive));
    500                 if (!f.mVendorId.isNull())
    501                     CHECK_ERROR_BREAK (flt, COMSETTER(VendorId) (f.mVendorId.setNullIfEmpty()));
    502                 if (!f.mProductId.isNull())
    503                     CHECK_ERROR_BREAK (flt, COMSETTER(ProductId) (f.mProductId.setNullIfEmpty()));
    504                 if (!f.mRevision.isNull())
    505                     CHECK_ERROR_BREAK (flt, COMSETTER(Revision) (f.mRevision.setNullIfEmpty()));
    506                 if (!f.mManufacturer.isNull())
    507                     CHECK_ERROR_BREAK (flt, COMSETTER(Manufacturer) (f.mManufacturer.setNullIfEmpty()));
    508                 if (!f.mSerialNumber.isNull())
    509                     CHECK_ERROR_BREAK (flt, COMSETTER(SerialNumber) (f.mSerialNumber.setNullIfEmpty()));
     500                if (!f.mVendorId.isEmpty())
     501                    CHECK_ERROR_BREAK (flt, COMSETTER(VendorId) (f.mVendorId));
     502                if (!f.mProductId.isEmpty())
     503                    CHECK_ERROR_BREAK (flt, COMSETTER(ProductId) (f.mProductId));
     504                if (!f.mRevision.isEmpty())
     505                    CHECK_ERROR_BREAK (flt, COMSETTER(Revision) (f.mRevision));
     506                if (!f.mManufacturer.isEmpty())
     507                    CHECK_ERROR_BREAK (flt, COMSETTER(Manufacturer) (f.mManufacturer));
     508                if (!f.mSerialNumber.isEmpty())
     509                    CHECK_ERROR_BREAK (flt, COMSETTER(SerialNumber) (f.mSerialNumber));
    510510                if (!f.mMaskedInterfaces.isNull())
    511511                    CHECK_ERROR_BREAK (flt, COMSETTER(MaskedInterfaces) (f.mMaskedInterfaces));
    … …  
    521521                ComPtr <IUSBDeviceFilter> flt = coll[cmd.mIndex];
    522522
    523                 if (!f.mName.isNull())
    524                     CHECK_ERROR_BREAK (flt, COMSETTER(Name) (f.mName.setNullIfEmpty()));
     523                if (!f.mName.isEmpty())
     524                    CHECK_ERROR_BREAK (flt, COMSETTER(Name) (f.mName));
    525525                if (!f.mActive.isNull())
    526526                    CHECK_ERROR_BREAK (flt, COMSETTER(Active) (f.mActive));
    527                 if (!f.mVendorId.isNull())
    528                     CHECK_ERROR_BREAK (flt, COMSETTER(VendorId) (f.mVendorId.setNullIfEmpty()));
    529                 if (!f.mProductId.isNull())
    530                     CHECK_ERROR_BREAK (flt, COMSETTER(ProductId) (f.mProductId.setNullIfEmpty()));
    531                 if (!f.mRevision.isNull())
    532                     CHECK_ERROR_BREAK (flt, COMSETTER(Revision) (f.mRevision.setNullIfEmpty()));
    533                 if (!f.mManufacturer.isNull())
    534                     CHECK_ERROR_BREAK (flt, COMSETTER(Manufacturer) (f.mManufacturer.setNullIfEmpty()));
    535                 if (!f.mRemote.isNull())
    536                     CHECK_ERROR_BREAK (flt, COMSETTER(Remote) (f.mRemote.setNullIfEmpty()));
    537                 if (!f.mSerialNumber.isNull())
    538                     CHECK_ERROR_BREAK (flt, COMSETTER(SerialNumber) (f.mSerialNumber.setNullIfEmpty()));
     527                if (!f.mVendorId.isEmpty())
     528                    CHECK_ERROR_BREAK (flt, COMSETTER(VendorId) (f.mVendorId));
     529                if (!f.mProductId.isEmpty())
     530                    CHECK_ERROR_BREAK (flt, COMSETTER(ProductId) (f.mProductId));
     531                if (!f.mRevision.isEmpty())
     532                    CHECK_ERROR_BREAK (flt, COMSETTER(Revision) (f.mRevision));
     533                if (!f.mManufacturer.isEmpty())
     534                    CHECK_ERROR_BREAK (flt, COMSETTER(Manufacturer) (f.mManufacturer));
     535                if (!f.mRemote.isEmpty())
     536                    CHECK_ERROR_BREAK (flt, COMSETTER(Remote) (f.mRemote));
     537                if (!f.mSerialNumber.isEmpty())
     538                    CHECK_ERROR_BREAK (flt, COMSETTER(SerialNumber) (f.mSerialNumber));
    539539                if (!f.mMaskedInterfaces.isNull())
    540540                    CHECK_ERROR_BREAK (flt, COMSETTER(MaskedInterfaces) (f.mMaskedInterfaces));
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