VirtualBox

Ignore:
Timestamp:
Jul 30, 2009 12:42:08 PM (15 years ago)
Author:
vboxsync
Message:

Main: coding style: have Main obey the standard VirtualBox coding style rules (no functional changes)

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Main/SystemPropertiesImpl.cpp

    r21446 r21878  
    6767HRESULT SystemProperties::init (VirtualBox *aParent)
    6868{
    69     LogFlowThisFunc (("aParent=%p\n", aParent));
     69    LogFlowThisFunc(("aParent=%p\n", aParent));
    7070
    7171    ComAssertRet (aParent, E_FAIL);
    7272
    7373    /* Enclose the state transition NotReady->InInit->Ready */
    74     AutoInitSpan autoInitSpan (this);
    75     AssertReturn (autoInitSpan.isOk(), E_FAIL);
    76 
    77     unconst (mParent) = aParent;
     74    AutoInitSpan autoInitSpan(this);
     75    AssertReturn(autoInitSpan.isOk(), E_FAIL);
     76
     77    unconst(mParent) = aParent;
    7878
    7979    setDefaultMachineFolder (NULL);
    … …  
    9999    int vrc = VDBackendInfo (RT_ELEMENTS (aVDInfo), aVDInfo, &cEntries);
    100100    AssertRC (vrc);
    101     if (RT_SUCCESS (vrc))
     101    if (RT_SUCCESS(vrc))
    102102    {
    103103        for (unsigned i = 0; i < cEntries; ++ i)
    104104        {
    105             ComObjPtr <HardDiskFormat> hdf;
     105            ComObjPtr<HardDiskFormat> hdf;
    106106            rc = hdf.createObject();
    107107            CheckComRCBreakRC (rc);
    … …  
    149149
    150150    /* Confirm a successful initialization */
    151     if (SUCCEEDED (rc))
     151    if (SUCCEEDED(rc))
    152152        autoInitSpan.setSucceeded();
    153153
    … …  
    161161void SystemProperties::uninit()
    162162{
    163     LogFlowThisFunc (("\n"));
     163    LogFlowThisFunc(("\n"));
    164164
    165165    /* Enclose the state transition Ready->InUninit->NotReady */
    166     AutoUninitSpan autoUninitSpan (this);
     166    AutoUninitSpan autoUninitSpan(this);
    167167    if (autoUninitSpan.uninitDone())
    168168        return;
    169169
    170     unconst (mParent).setNull();
     170    unconst(mParent).setNull();
    171171}
    172172
    … …  
    180180        return E_POINTER;
    181181
    182     AutoCaller autoCaller (this);
    183     CheckComRCReturnRC (autoCaller.rc());
     182    AutoCaller autoCaller(this);
     183    CheckComRCReturnRC(autoCaller.rc());
    184184
    185185    /* no need to lock, this is const */
    … …  
    195195        return E_POINTER;
    196196
    197     AutoCaller autoCaller (this);
    198     CheckComRCReturnRC (autoCaller.rc());
     197    AutoCaller autoCaller(this);
     198    CheckComRCReturnRC(autoCaller.rc());
    199199
    200200    /* no need to lock, this is const */
    … …  
    210210        return E_POINTER;
    211211
    212     AutoCaller autoCaller (this);
    213     CheckComRCReturnRC (autoCaller.rc());
     212    AutoCaller autoCaller(this);
     213    CheckComRCReturnRC(autoCaller.rc());
    214214
    215215    /* no need to lock, this is const */
    … …  
    224224        return E_POINTER;
    225225
    226     AutoCaller autoCaller (this);
    227     CheckComRCReturnRC (autoCaller.rc());
     226    AutoCaller autoCaller(this);
     227    CheckComRCReturnRC(autoCaller.rc());
    228228
    229229    /* no need to lock, this is const */
    … …  
    238238        return E_POINTER;
    239239
    240     AutoCaller autoCaller (this);
    241     CheckComRCReturnRC (autoCaller.rc());
     240    AutoCaller autoCaller(this);
     241    CheckComRCReturnRC(autoCaller.rc());
    242242
    243243    /* no need to lock, this is const */
    … …  
    252252        return E_POINTER;
    253253
    254     AutoCaller autoCaller (this);
    255     CheckComRCReturnRC (autoCaller.rc());
     254    AutoCaller autoCaller(this);
     255    CheckComRCReturnRC(autoCaller.rc());
    256256
    257257    /* no need to lock, this is const */
    … …  
    266266        return E_POINTER;
    267267
    268     AutoCaller autoCaller (this);
    269     CheckComRCReturnRC (autoCaller.rc());
     268    AutoCaller autoCaller(this);
     269    CheckComRCReturnRC(autoCaller.rc());
    270270
    271271    /* no need to lock, this is const */
    … …  
    280280        return E_POINTER;
    281281
    282     AutoCaller autoCaller (this);
    283     CheckComRCReturnRC (autoCaller.rc());
     282    AutoCaller autoCaller(this);
     283    CheckComRCReturnRC(autoCaller.rc());
    284284
    285285    /** The BIOS supports currently 32 bit LBA numbers (implementing the full
    … …  
    303303        return E_POINTER;
    304304
    305     AutoCaller autoCaller (this);
    306     CheckComRCReturnRC (autoCaller.rc());
     305    AutoCaller autoCaller(this);
     306    CheckComRCReturnRC(autoCaller.rc());
    307307
    308308    /* no need to lock, this is const */
    … …  
    317317        return E_POINTER;
    318318
    319     AutoCaller autoCaller (this);
    320     CheckComRCReturnRC (autoCaller.rc());
     319    AutoCaller autoCaller(this);
     320    CheckComRCReturnRC(autoCaller.rc());
    321321
    322322    /* no need to lock, this is const */
    … …  
    331331        return E_POINTER;
    332332
    333     AutoCaller autoCaller (this);
    334     CheckComRCReturnRC (autoCaller.rc());
     333    AutoCaller autoCaller(this);
     334    CheckComRCReturnRC(autoCaller.rc());
    335335
    336336    /* no need to lock, this is const */
    … …  
    344344    CheckComArgOutPointerValid(aMaxBootPosition);
    345345
    346     AutoCaller autoCaller (this);
    347     CheckComRCReturnRC (autoCaller.rc());
     346    AutoCaller autoCaller(this);
     347    CheckComRCReturnRC(autoCaller.rc());
    348348
    349349    /* no need to lock, this is const */
    … …  
    357357    CheckComArgOutPointerValid(aDefaultMachineFolder);
    358358
    359     AutoCaller autoCaller (this);
    360     CheckComRCReturnRC (autoCaller.rc());
    361 
    362     AutoReadLock alock (this);
    363 
    364     mDefaultMachineFolderFull.cloneTo (aDefaultMachineFolder);
     359    AutoCaller autoCaller(this);
     360    CheckComRCReturnRC(autoCaller.rc());
     361
     362    AutoReadLock alock(this);
     363
     364    mDefaultMachineFolderFull.cloneTo(aDefaultMachineFolder);
    365365
    366366    return S_OK;
    … …  
    369369STDMETHODIMP SystemProperties::COMSETTER(DefaultMachineFolder) (IN_BSTR aDefaultMachineFolder)
    370370{
    371     AutoCaller autoCaller (this);
    372     CheckComRCReturnRC (autoCaller.rc());
     371    AutoCaller autoCaller(this);
     372    CheckComRCReturnRC(autoCaller.rc());
    373373
    374374    /* VirtualBox::saveSettings() needs a write lock */
    … …  
    376376
    377377    HRESULT rc = setDefaultMachineFolder (aDefaultMachineFolder);
    378     if (SUCCEEDED (rc))
     378    if (SUCCEEDED(rc))
    379379        rc = mParent->saveSettings();
    380380
    … …  
    386386    CheckComArgOutPointerValid(aDefaultHardDiskFolder);
    387387
    388     AutoCaller autoCaller (this);
    389     CheckComRCReturnRC (autoCaller.rc());
    390 
    391     AutoReadLock alock (this);
    392 
    393     mDefaultHardDiskFolderFull.cloneTo (aDefaultHardDiskFolder);
     388    AutoCaller autoCaller(this);
     389    CheckComRCReturnRC(autoCaller.rc());
     390
     391    AutoReadLock alock(this);
     392
     393    mDefaultHardDiskFolderFull.cloneTo(aDefaultHardDiskFolder);
    394394
    395395    return S_OK;
    … …  
    398398STDMETHODIMP SystemProperties::COMSETTER(DefaultHardDiskFolder) (IN_BSTR aDefaultHardDiskFolder)
    399399{
    400     AutoCaller autoCaller (this);
    401     CheckComRCReturnRC (autoCaller.rc());
     400    AutoCaller autoCaller(this);
     401    CheckComRCReturnRC(autoCaller.rc());
    402402
    403403    /* VirtualBox::saveSettings() needs a write lock */
    … …  
    405405
    406406    HRESULT rc = setDefaultHardDiskFolder (aDefaultHardDiskFolder);
    407     if (SUCCEEDED (rc))
     407    if (SUCCEEDED(rc))
    408408        rc = mParent->saveSettings();
    409409
    … …  
    412412
    413413STDMETHODIMP SystemProperties::
    414 COMGETTER(HardDiskFormats) (ComSafeArrayOut (IHardDiskFormat *, aHardDiskFormats))
    415 {
    416     if (ComSafeArrayOutIsNull (aHardDiskFormats))
    417         return E_POINTER;
    418 
    419     AutoCaller autoCaller (this);
    420     CheckComRCReturnRC (autoCaller.rc());
    421 
    422     AutoReadLock alock (this);
    423 
    424     SafeIfaceArray <IHardDiskFormat> hardDiskFormats (mHardDiskFormats);
    425     hardDiskFormats.detachTo (ComSafeArrayOutArg (aHardDiskFormats));
     414COMGETTER(HardDiskFormats) (ComSafeArrayOut(IHardDiskFormat *, aHardDiskFormats))
     415{
     416    if (ComSafeArrayOutIsNull(aHardDiskFormats))
     417        return E_POINTER;
     418
     419    AutoCaller autoCaller(this);
     420    CheckComRCReturnRC(autoCaller.rc());
     421
     422    AutoReadLock alock(this);
     423
     424    SafeIfaceArray<IHardDiskFormat> hardDiskFormats (mHardDiskFormats);
     425    hardDiskFormats.detachTo(ComSafeArrayOutArg(aHardDiskFormats));
    426426
    427427    return S_OK;
    … …  
    432432    CheckComArgOutPointerValid(aDefaultHardDiskFormat);
    433433
    434     AutoCaller autoCaller (this);
    435     CheckComRCReturnRC (autoCaller.rc());
    436 
    437     AutoReadLock alock (this);
    438 
    439     mDefaultHardDiskFormat.cloneTo (aDefaultHardDiskFormat);
     434    AutoCaller autoCaller(this);
     435    CheckComRCReturnRC(autoCaller.rc());
     436
     437    AutoReadLock alock(this);
     438
     439    mDefaultHardDiskFormat.cloneTo(aDefaultHardDiskFormat);
    440440
    441441    return S_OK;
    … …  
    444444STDMETHODIMP SystemProperties::COMSETTER(DefaultHardDiskFormat) (IN_BSTR aDefaultHardDiskFormat)
    445445{
    446     AutoCaller autoCaller (this);
    447     CheckComRCReturnRC (autoCaller.rc());
     446    AutoCaller autoCaller(this);
     447    CheckComRCReturnRC(autoCaller.rc());
    448448
    449449    /* VirtualBox::saveSettings() needs a write lock */
    … …  
    451451
    452452    HRESULT rc = setDefaultHardDiskFormat (aDefaultHardDiskFormat);
    453     if (SUCCEEDED (rc))
     453    if (SUCCEEDED(rc))
    454454        rc = mParent->saveSettings();
    455455
    … …  
    461461    CheckComArgOutPointerValid(aRemoteDisplayAuthLibrary);
    462462
    463     AutoCaller autoCaller (this);
    464     CheckComRCReturnRC (autoCaller.rc());
    465 
    466     AutoReadLock alock (this);
    467 
    468     mRemoteDisplayAuthLibrary.cloneTo (aRemoteDisplayAuthLibrary);
     463    AutoCaller autoCaller(this);
     464    CheckComRCReturnRC(autoCaller.rc());
     465
     466    AutoReadLock alock(this);
     467
     468    mRemoteDisplayAuthLibrary.cloneTo(aRemoteDisplayAuthLibrary);
    469469
    470470    return S_OK;
    … …  
    473473STDMETHODIMP SystemProperties::COMSETTER(RemoteDisplayAuthLibrary) (IN_BSTR aRemoteDisplayAuthLibrary)
    474474{
    475     AutoCaller autoCaller (this);
    476     CheckComRCReturnRC (autoCaller.rc());
     475    AutoCaller autoCaller(this);
     476    CheckComRCReturnRC(autoCaller.rc());
    477477
    478478    /* VirtualBox::saveSettings() needs a write lock */
    … …  
    480480
    481481    HRESULT rc = setRemoteDisplayAuthLibrary (aRemoteDisplayAuthLibrary);
    482     if (SUCCEEDED (rc))
     482    if (SUCCEEDED(rc))
    483483        rc = mParent->saveSettings();
    484484
    … …  
    490490    CheckComArgOutPointerValid(aWebServiceAuthLibrary);
    491491
    492     AutoCaller autoCaller (this);
    493     CheckComRCReturnRC (autoCaller.rc());
    494 
    495     AutoReadLock alock (this);
    496 
    497     mWebServiceAuthLibrary.cloneTo (aWebServiceAuthLibrary);
     492    AutoCaller autoCaller(this);
     493    CheckComRCReturnRC(autoCaller.rc());
     494
     495    AutoReadLock alock(this);
     496
     497    mWebServiceAuthLibrary.cloneTo(aWebServiceAuthLibrary);
    498498
    499499    return S_OK;
    … …  
    502502STDMETHODIMP SystemProperties::COMSETTER(WebServiceAuthLibrary) (IN_BSTR aWebServiceAuthLibrary)
    503503{
    504     AutoCaller autoCaller (this);
    505     CheckComRCReturnRC (autoCaller.rc());
     504    AutoCaller autoCaller(this);
     505    CheckComRCReturnRC(autoCaller.rc());
    506506
    507507    /* VirtualBox::saveSettings() needs a write lock */
    … …  
    509509
    510510    HRESULT rc = setWebServiceAuthLibrary (aWebServiceAuthLibrary);
    511     if (SUCCEEDED (rc))
     511    if (SUCCEEDED(rc))
    512512        rc = mParent->saveSettings();
    513513
    … …  
    520520        return E_POINTER;
    521521
    522     AutoCaller autoCaller (this);
    523     CheckComRCReturnRC (autoCaller.rc());
    524 
    525     AutoReadLock alock (this);
     522    AutoCaller autoCaller(this);
     523    CheckComRCReturnRC(autoCaller.rc());
     524
     525    AutoReadLock alock(this);
    526526
    527527    *count = mLogHistoryCount;
    … …  
    532532STDMETHODIMP SystemProperties::COMSETTER(LogHistoryCount) (ULONG count)
    533533{
    534     AutoCaller autoCaller (this);
    535     CheckComRCReturnRC (autoCaller.rc());
     534    AutoCaller autoCaller(this);
     535    CheckComRCReturnRC(autoCaller.rc());
    536536
    537537    /* VirtualBox::saveSettings() needs a write lock */
    … …  
    550550        return E_POINTER;
    551551
    552     AutoCaller autoCaller (this);
    553     CheckComRCReturnRC (autoCaller.rc());
    554 
    555     AutoReadLock alock (this);
     552    AutoCaller autoCaller(this);
     553    CheckComRCReturnRC(autoCaller.rc());
     554
     555    AutoReadLock alock(this);
    556556
    557557    *aAudioDriver = mDefaultAudioDriver;
    … …  
    567567    using namespace settings;
    568568
    569     AutoCaller autoCaller (this);
    570     CheckComRCReturnRC (autoCaller.rc());
    571 
    572     AutoWriteLock alock (this);
    573 
    574     AssertReturn (!aGlobal.isNull(), E_FAIL);
     569    AutoCaller autoCaller(this);
     570    CheckComRCReturnRC(autoCaller.rc());
     571
     572    AutoWriteLock alock(this);
     573
     574    AssertReturn(!aGlobal.isNull(), E_FAIL);
    575575
    576576    HRESULT rc = S_OK;
    … …  
    582582    bstr = properties.stringValue ("defaultMachineFolder");
    583583    rc = setDefaultMachineFolder (bstr);
    584     CheckComRCReturnRC (rc);
     584    CheckComRCReturnRC(rc);
    585585
    586586    bstr = properties.stringValue ("defaultHardDiskFolder");
    587587    rc = setDefaultHardDiskFolder (bstr);
    588     CheckComRCReturnRC (rc);
     588    CheckComRCReturnRC(rc);
    589589
    590590    bstr = properties.stringValue ("defaultHardDiskFormat");
    591591    rc = setDefaultHardDiskFormat (bstr);
    592     CheckComRCReturnRC (rc);
     592    CheckComRCReturnRC(rc);
    593593
    594594    bstr = properties.stringValue ("remoteDisplayAuthLibrary");
    595595    rc = setRemoteDisplayAuthLibrary (bstr);
    596     CheckComRCReturnRC (rc);
     596    CheckComRCReturnRC(rc);
    597597
    598598    bstr = properties.stringValue ("webServiceAuthLibrary");
    599599    rc = setWebServiceAuthLibrary (bstr);
    600     CheckComRCReturnRC (rc);
     600    CheckComRCReturnRC(rc);
    601601
    602602    mLogHistoryCount = properties.valueOr <ULONG> ("LogHistoryCount", 3);
    … …  
    609609    using namespace settings;
    610610
    611     AutoCaller autoCaller (this);
    612     CheckComRCReturnRC (autoCaller.rc());
    613 
    614     AutoReadLock alock (this);
     611    AutoCaller autoCaller(this);
     612    CheckComRCReturnRC(autoCaller.rc());
     613
     614    AutoReadLock alock(this);
    615615
    616616    ComAssertRet (!aGlobal.isNull(), E_FAIL);
    … …  
    651651 * @return ComObjPtr<HardDiskFormat>
    652652 */
    653 ComObjPtr <HardDiskFormat> SystemProperties::hardDiskFormat (CBSTR aFormat)
    654 {
    655     ComObjPtr <HardDiskFormat> format;
    656 
    657     AutoCaller autoCaller (this);
     653ComObjPtr<HardDiskFormat> SystemProperties::hardDiskFormat (CBSTR aFormat)
     654{
     655    ComObjPtr<HardDiskFormat> format;
     656
     657    AutoCaller autoCaller(this);
    658658    AssertComRCReturn (autoCaller.rc(), format);
    659659
    660     AutoReadLock alock (this);
     660    AutoReadLock alock(this);
    661661
    662662    for (HardDiskFormatList::const_iterator it = mHardDiskFormats.begin();
    … …  
    689689    Utf8Str folder;
    690690    int vrc = mParent->calculateFullPath (path, folder);
    691     if (RT_FAILURE (vrc))
     691    if (RT_FAILURE(vrc))
    692692        return setError (E_FAIL,
    693693            tr ("Invalid default machine folder '%ls' (%Rrc)"),
    … …  
    711711    Utf8Str folder;
    712712    int vrc = mParent->calculateFullPath (path, folder);
    713     if (RT_FAILURE (vrc))
     713    if (RT_FAILURE(vrc))
    714714        return setError (E_FAIL,
    715715            tr ("Invalid default hard disk folder '%ls' (%Rrc)"),
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