VirtualBox

Changeset 101293 in vbox for trunk


Ignore:
Timestamp:
Sep 27, 2023 2:20:49 PM (19 months ago)
Author:
vboxsync
svn:sync-xref-src-repo-rev:
159279
Message:

Main/ExtPack*: tidying up. (Please, do NOT use 'm' as a local variable, that's just so confusing.)

Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/include/VBox/ExtPack/ExtPack.h

    r98103 r101293  
    346346     *
    347347     * @returns Translated string on success the pszSourceText otherwise.
    348      * @param   pHlp                      Pointer to this helper structure.
    349      * @param   aComponent                Translation context e.g. class name
    350      * @param   pszSourceText             String to translate
    351      * @param   pszComment                Comment to the string to resolve possible ambiguities
    352      *                                    (NULL means no comment).
    353      * @param   aNum                      Number used to define plural form of the translation
    354      */
     348     * @param   pHlp            Pointer to this helper structure.
     349     * @param   pszComponent    Translation context e.g. class name
     350     * @param   pszSourceText   String to translate
     351     * @param   pszComment      Comment to the string to resolve possible
     352     *                          ambiguities (NULL means no comment).
     353     * @param   uNum            Number used to define plural form of the
     354     *                          translation.
     355                                                                          */
    355356    DECLR3CALLBACKMEMBER(const char *, pfnTranslate,(PCVBOXEXTPACKHLP pHlp,
    356                                                      const char  *pszComponent,
    357                                                      const char  *pszSourceText,
    358                                                      const char  *pszComment,
    359                                                      const size_t aNum));
     357                                                     const char      *pszComponent,
     358                                                     const char      *pszSourceText,
     359                                                     const char      *pszComment,
     360                                                     const size_t     uNum));
    360361
    361362    DECLR3CALLBACKMEMBER(int, pfnReserved1,(PCVBOXEXTPACKHLP pHlp)); /**< Reserved for minor structure revisions. */

  • trunk/src/VBox/Main/include/ExtPackManagerImpl.h

    r98103 r101293  
    198198                                                       VBOXEXTPACK_IF_CS(IVetoEvent) **ppEventOut);
    199199    static DECLCALLBACK(const char *) i_hlpTranslate(PCVBOXEXTPACKHLP pHlp,
    200                                                      const char  *pszComponent,
    201                                                      const char  *pszSourceText,
    202                                                      const char  *pszComment = NULL,
    203                                                      const size_t aNum = ~(size_t)0);
     200                                                     const char      *pszComponent,
     201                                                     const char      *pszSourceText,
     202                                                     const char      *pszComment = NULL,
     203                                                     const size_t     uNum = ~(size_t)0);
    204204    static DECLCALLBACK(int)      i_hlpReservedN(PCVBOXEXTPACKHLP pHlp);
    205205    /** @}  */

  • trunk/src/VBox/Main/src-all/ExtPackManagerImpl.cpp

    r100737 r101293  
    17741774    AssertPtrReturn(pHlp, VERR_INVALID_POINTER);
    17751775    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, VERR_INVALID_POINTER);
    1776     ExtPack::Data *m = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
    1777     AssertPtrReturn(m, VERR_INVALID_POINTER);
    1778     ExtPack *pThis = m->pThis;
     1776    ExtPack::Data *pData = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
     1777    AssertPtrReturn(pData, VERR_INVALID_POINTER);
     1778    ExtPack *pThis = pData->pThis;
    17791779    AssertPtrReturn(pThis, VERR_INVALID_POINTER);
    17801780
    … …  
    17881788}
    17891789
     1790/**
     1791 * @interface_method_impl{VBOXEXTPACKHLP,pfnGetFilePath}
     1792 */
    17901793/*static*/ DECLCALLBACK(int)
    17911794ExtPack::i_hlpGetFilePath(PCVBOXEXTPACKHLP pHlp, const char *pszFilename, char *pszPath, size_t cbPath)
    … …  
    18001803    AssertPtrReturn(pHlp, VERR_INVALID_POINTER);
    18011804    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, VERR_INVALID_POINTER);
    1802     ExtPack::Data *m = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
    1803     AssertPtrReturn(m, VERR_INVALID_POINTER);
    1804     ExtPack *pThis = m->pThis;
    1805     AssertPtrReturn(pThis, VERR_INVALID_POINTER);
     1805    ExtPack::Data *pData = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
     1806    AssertPtrReturn(pData, VERR_INVALID_POINTER);
    18061807
    18071808    /*
    18081809     * This is a simple RTPathJoin, no checking if things exists or anything.
    18091810     */
    1810     int vrc = RTPathJoin(pszPath, cbPath, pThis->m->strExtPackPath.c_str(), pszFilename);
     1811    int vrc = RTPathJoin(pszPath, cbPath, pData->strExtPackPath.c_str(), pszFilename);
    18111812    if (RT_FAILURE(vrc))
    18121813        RT_BZERO(pszPath, cbPath);
    … …  
    18141815}
    18151816
     1817/**
     1818 * @interface_method_impl{VBOXEXTPACKHLP,pfnGetContext}
     1819 */
    18161820/*static*/ DECLCALLBACK(VBOXEXTPACKCTX)
    18171821ExtPack::i_hlpGetContext(PCVBOXEXTPACKHLP pHlp)
    … …  
    18221826    AssertPtrReturn(pHlp, VBOXEXTPACKCTX_INVALID);
    18231827    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, VBOXEXTPACKCTX_INVALID);
    1824     ExtPack::Data *m = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
    1825     AssertPtrReturn(m, VBOXEXTPACKCTX_INVALID);
    1826     ExtPack *pThis = m->pThis;
    1827     AssertPtrReturn(pThis, VBOXEXTPACKCTX_INVALID);
    1828 
    1829     return pThis->m->enmContext;
    1830 }
    1831 
     1828    ExtPack::Data *pData = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
     1829    AssertPtrReturn(pData, VBOXEXTPACKCTX_INVALID);
     1830
     1831    return pData->enmContext;
     1832}
     1833
     1834/**
     1835 * @interface_method_impl{VBOXEXTPACKHLP,pfnLoadHGCMService}
     1836 */
    18321837/*static*/ DECLCALLBACK(int)
    18331838ExtPack::i_hlpLoadHGCMService(PCVBOXEXTPACKHLP pHlp, VBOXEXTPACK_IF_CS(IConsole) *pConsole,
    … …  
    18431848    AssertPtrReturn(pHlp, VERR_INVALID_POINTER);
    18441849    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, VERR_INVALID_POINTER);
    1845     ExtPack::Data *m = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
    1846     AssertPtrReturn(m, VERR_INVALID_POINTER);
    1847     ExtPack *pThis = m->pThis;
     1850    ExtPack::Data *pData = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
     1851    AssertPtrReturn(pData, VERR_INVALID_POINTER);
     1852    ExtPack *pThis = pData->pThis;
    18481853    AssertPtrReturn(pThis, VERR_INVALID_POINTER);
    18491854    AssertPtrReturn(pConsole, VERR_INVALID_POINTER);
    … …  
    18521857    return pCon->i_hgcmLoadService(pszServiceLibrary, pszServiceName);
    18531858#else
    1854     NOREF(pHlp); NOREF(pConsole); NOREF(pszServiceLibrary); NOREF(pszServiceName);
     1859    RT_NOREF(pHlp, pConsole, pszServiceLibrary, pszServiceName);
    18551860    return VERR_INVALID_STATE;
    18561861#endif
    18571862}
    18581863
     1864/**
     1865 * @interface_method_impl{VBOXEXTPACKHLP,pfnLoadVDPlugin}
     1866 */
    18591867/*static*/ DECLCALLBACK(int)
    18601868ExtPack::i_hlpLoadVDPlugin(PCVBOXEXTPACKHLP pHlp, VBOXEXTPACK_IF_CS(IVirtualBox) *pVirtualBox, const char *pszPluginLibrary)
    … …  
    18681876    AssertPtrReturn(pHlp, VERR_INVALID_POINTER);
    18691877    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, VERR_INVALID_POINTER);
    1870     ExtPack::Data *m = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
    1871     AssertPtrReturn(m, VERR_INVALID_POINTER);
    1872     ExtPack *pThis = m->pThis;
     1878    ExtPack::Data *pData = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
     1879    AssertPtrReturn(pData, VERR_INVALID_POINTER);
     1880    ExtPack *pThis = pData->pThis;
    18731881    AssertPtrReturn(pThis, VERR_INVALID_POINTER);
    18741882    AssertPtrReturn(pVirtualBox, VERR_INVALID_POINTER);
    … …  
    18771885    return pVBox->i_loadVDPlugin(pszPluginLibrary);
    18781886#else
    1879     NOREF(pHlp); NOREF(pVirtualBox); NOREF(pszPluginLibrary);
     1887    RT_NOREF(pHlp, pVirtualBox, pszPluginLibrary);
    18801888    return VERR_INVALID_STATE;
    18811889#endif
    18821890}
    18831891
     1892/**
     1893 * @interface_method_impl{VBOXEXTPACKHLP,pfnUnloadVDPlugin}
     1894 */
    18841895/*static*/ DECLCALLBACK(int)
    18851896ExtPack::i_hlpUnloadVDPlugin(PCVBOXEXTPACKHLP pHlp, VBOXEXTPACK_IF_CS(IVirtualBox) *pVirtualBox, const char *pszPluginLibrary)
    … …  
    18931904    AssertPtrReturn(pHlp, VERR_INVALID_POINTER);
    18941905    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, VERR_INVALID_POINTER);
    1895     ExtPack::Data *m = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
    1896     AssertPtrReturn(m, VERR_INVALID_POINTER);
    1897     ExtPack *pThis = m->pThis;
     1906    ExtPack::Data *pData = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
     1907    AssertPtrReturn(pData, VERR_INVALID_POINTER);
     1908    ExtPack *pThis = pData->pThis;
    18981909    AssertPtrReturn(pThis, VERR_INVALID_POINTER);
    18991910    AssertPtrReturn(pVirtualBox, VERR_INVALID_POINTER);
    … …  
    19021913    return pVBox->i_unloadVDPlugin(pszPluginLibrary);
    19031914#else
    1904     NOREF(pHlp); NOREF(pVirtualBox); NOREF(pszPluginLibrary);
     1915    RT_NOREF(pHlp, pVirtualBox, pszPluginLibrary);
    19051916    return VERR_INVALID_STATE;
    19061917#endif
    19071918}
    19081919
     1920/**
     1921 * @interface_method_impl{VBOXEXTPACKHLP,pfnCreateProgress}
     1922 */
    19091923/*static*/ DECLCALLBACK(uint32_t)
    19101924ExtPack::i_hlpCreateProgress(PCVBOXEXTPACKHLP pHlp, VBOXEXTPACK_IF_CS(IUnknown) *pInitiator,
    … …  
    19261940    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, (uint32_t)E_INVALIDARG);
    19271941#ifndef VBOX_COM_INPROC
    1928     ExtPack::Data *m = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
     1942    ExtPack::Data *pData = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
    19291943#endif
    19301944
    19311945    ComObjPtr<Progress> pProgress;
    19321946    HRESULT hrc = pProgress.createObject();
    1933     if (FAILED(hrc))
    1934         return hrc;
    1935     hrc = pProgress->init(
     1947    if (SUCCEEDED(hrc))
     1948    {
     1949        hrc = pProgress->init(
    19361950#ifndef VBOX_COM_INPROC
    1937                           m->pVirtualBox,
     1951                              pData->pVirtualBox,
    19381952#endif
    1939                           pInitiator, pcszDescription, TRUE /* aCancelable */,
    1940                           cOperations, uTotalOperationsWeight,
    1941                           pcszFirstOperationDescription, uFirstOperationWeight);
    1942     if (FAILED(hrc))
    1943         return hrc;
    1944 
    1945     return pProgress.queryInterfaceTo(ppProgressOut);
    1946 }
    1947 
     1953                              pInitiator, pcszDescription, TRUE /* aCancelable */,
     1954                              cOperations, uTotalOperationsWeight,
     1955                              pcszFirstOperationDescription, uFirstOperationWeight);
     1956        if (SUCCEEDED(hrc))
     1957            hrc = pProgress.queryInterfaceTo(ppProgressOut);
     1958    }
     1959    return hrc;
     1960}
     1961
     1962/**
     1963 * @interface_method_impl{VBOXEXTPACKHLP,pfnGetCanceledProgress}
     1964 */
    19481965/*static*/ DECLCALLBACK(uint32_t)
    19491966ExtPack::i_hlpGetCanceledProgress(PCVBOXEXTPACKHLP pHlp, VBOXEXTPACK_IF_CS(IProgress) *pProgress,
    … …  
    19651982}
    19661983
     1984/**
     1985 * @interface_method_impl{VBOXEXTPACKHLP,pfnUpdateProgress}
     1986 */
    19671987/*static*/ DECLCALLBACK(uint32_t)
    19681988ExtPack::i_hlpUpdateProgress(PCVBOXEXTPACKHLP pHlp, VBOXEXTPACK_IF_CS(IProgress) *pProgress,
    … …  
    19832003}
    19842004
     2005/**
     2006 * @interface_method_impl{VBOXEXTPACKHLP,pfnNextOperationProgress}
     2007 */
    19852008/*static*/ DECLCALLBACK(uint32_t)
    19862009ExtPack::i_hlpNextOperationProgress(PCVBOXEXTPACKHLP pHlp, VBOXEXTPACK_IF_CS(IProgress) *pProgress,
    … …  
    20032026}
    20042027
     2028/**
     2029 * @interface_method_impl{VBOXEXTPACKHLP,pfnWaitOtherProgress}
     2030 */
    20052031/*static*/ DECLCALLBACK(uint32_t)
    20062032ExtPack::i_hlpWaitOtherProgress(PCVBOXEXTPACKHLP pHlp, VBOXEXTPACK_IF_CS(IProgress) *pProgress,
    … …  
    20212047}
    20222048
     2049/**
     2050 * @interface_method_impl{VBOXEXTPACKHLP,pfnCompleteProgress}
     2051 */
    20232052/*static*/ DECLCALLBACK(uint32_t)
    20242053ExtPack::i_hlpCompleteProgress(PCVBOXEXTPACKHLP pHlp, VBOXEXTPACK_IF_CS(IProgress) *pProgress,
    … …  
    20282057     * Validate the input and get our bearings.
    20292058     */
    2030     AssertPtrReturn(pProgress, (uint32_t)E_INVALIDARG);
    2031 
    20322059    AssertPtrReturn(pHlp, (uint32_t)E_INVALIDARG);
    20332060    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, (uint32_t)E_INVALIDARG);
     2061    AssertPtrReturn(pProgress, (uint32_t)E_INVALIDARG);
    20342062
    20352063    ComPtr<IInternalProgressControl> pProgressControl(pProgress);
    … …  
    20452073}
    20462074
    2047 
     2075/**
     2076 * @interface_method_impl{VBOXEXTPACKHLP,pfnCreateEvent}
     2077 */
    20482078/*static*/ DECLCALLBACK(uint32_t)
    20492079ExtPack::i_hlpCreateEvent(PCVBOXEXTPACKHLP pHlp,
    … …  
    20522082                          VBOXEXTPACK_IF_CS(IEvent) **ppEventOut)
    20532083{
    2054     HRESULT hrc;
    2055 
     2084    /*
     2085     * Validate the input and get our bearings.
     2086     */
    20562087    AssertPtrReturn(pHlp, (uint32_t)E_INVALIDARG);
    20572088    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, (uint32_t)E_INVALIDARG);
    … …  
    20592090
    20602091    ComObjPtr<VBoxEvent> pEvent;
    2061 
    2062     hrc = pEvent.createObject();
    2063     if (FAILED(hrc))
    2064         return hrc;
    2065 
    2066     /* default aSource to pVirtualBox? */
    2067     hrc = pEvent->init(aSource, static_cast<VBoxEventType_T>(aType), aWaitable);
    2068     if (FAILED(hrc))
    2069         return hrc;
    2070 
    2071     return pEvent.queryInterfaceTo(ppEventOut);
    2072 }
    2073 
    2074 
     2092    HRESULT hrc = pEvent.createObject();
     2093    if (SUCCEEDED(hrc))
     2094    {
     2095        /* default aSource to pVirtualBox? */
     2096        hrc = pEvent->init(aSource, static_cast<VBoxEventType_T>(aType), aWaitable);
     2097        if (SUCCEEDED(hrc))
     2098            hrc = pEvent.queryInterfaceTo(ppEventOut);
     2099    }
     2100    return hrc;
     2101}
     2102
     2103/**
     2104 * @interface_method_impl{VBOXEXTPACKHLP,pfnCreateVetoEvent}
     2105 */
    20752106/*static*/ DECLCALLBACK(uint32_t)
    20762107ExtPack::i_hlpCreateVetoEvent(PCVBOXEXTPACKHLP pHlp,
    … …  
    20792110                              VBOXEXTPACK_IF_CS(IVetoEvent) **ppEventOut)
    20802111{
    2081     HRESULT hrc;
    2082 
    20832112    AssertPtrReturn(pHlp, (uint32_t)E_INVALIDARG);
    20842113    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, (uint32_t)E_INVALIDARG);
    … …  
    20872116    ComObjPtr<VBoxVetoEvent> pEvent;
    20882117
    2089     hrc = pEvent.createObject();
    2090     if (FAILED(hrc))
    2091         return hrc;
    2092 
    2093     /* default aSource to pVirtualBox? */
    2094     hrc = pEvent->init(aSource, static_cast<VBoxEventType_T>(aType));
    2095     if (FAILED(hrc))
    2096         return hrc;
    2097 
    2098     return pEvent.queryInterfaceTo(ppEventOut);
    2099 }
    2100 
    2101 
     2118    HRESULT hrc = pEvent.createObject();
     2119    if (SUCCEEDED(hrc))
     2120    {
     2121        /* default aSource to pVirtualBox? */
     2122        hrc = pEvent->init(aSource, static_cast<VBoxEventType_T>(aType));
     2123        if (SUCCEEDED(hrc))
     2124            hrc = pEvent.queryInterfaceTo(ppEventOut);
     2125    }
     2126    return hrc;
     2127}
     2128
     2129/**
     2130 * @interface_method_impl{VBOXEXTPACKHLP,pfnTranslate}
     2131 */
    21022132/*static*/ DECLCALLBACK(const char *)
    21032133ExtPack::i_hlpTranslate(PCVBOXEXTPACKHLP pHlp,
    21042134                        const char  *pszComponent,
    21052135                        const char  *pszSourceText,
    2106                         const char  *pszComment /* = NULL */,
    2107                         const size_t aNum /* = -1 */)
     2136                        const char  *pszComment /*=NULL*/,
     2137                        const size_t uNum /*= ~(size_t)0*/)
    21082138{
    21092139    /*
    … …  
    21122142    AssertPtrReturn(pHlp, pszSourceText);
    21132143    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, pszSourceText);
    2114     ExtPack::Data *m = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
    2115     AssertPtrReturn(m, pszSourceText);
     2144    ExtPack::Data *pData = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
     2145    AssertPtrReturn(pData, pszSourceText);
    21162146
    21172147#ifdef VBOX_WITH_MAIN_NLS
    2118     return VirtualBoxTranslator::translate(m->pTrComponent, pszComponent,
    2119                                            pszSourceText,  pszComment, aNum);
     2148    return VirtualBoxTranslator::translate(pData->pTrComponent, pszComponent, pszSourceText, pszComment, uNum);
    21202149#else
    2121     NOREF(pszComponent);
    2122     NOREF(pszComment);
    2123     NOREF(aNum);
     2150    RT_NOREF(pszComponent, pszComment, uNum);
    21242151    return pszSourceText;
    21252152#endif
    … …  
    21352162    AssertPtrReturn(pHlp, VERR_INVALID_POINTER);
    21362163    AssertReturn(pHlp->u32Version == VBOXEXTPACKHLP_VERSION, VERR_INVALID_POINTER);
    2137     ExtPack::Data *m = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
    2138     AssertPtrReturn(m, VERR_INVALID_POINTER);
    2139     ExtPack *pThis = m->pThis;
     2164    ExtPack::Data *pData = RT_FROM_CPP_MEMBER(pHlp, Data, Hlp);
     2165    AssertPtrReturn(pData, VERR_INVALID_POINTER);
     2166    ExtPack *pThis = pData->pThis;
    21402167    AssertPtrReturn(pThis, VERR_INVALID_POINTER);
    21412168
Note: See TracChangeset for help on using the changeset viewer.

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