VirtualBox

Changeset 80875 in vbox for trunk


Ignore:
Timestamp:
Sep 18, 2019 12:04:18 AM (5 years ago)
Author:
vboxsync
Message:

Main/GuestSessionImpl.cpp: Removed most of the AutoCaller instances from (what appears to be) public interface methods since the API wrapper code deals with these already.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Main/src-client/GuestSessionImpl.cpp

    r80873 r80875  
    29422942HRESULT GuestSession::close()
    29432943{
    2944     AutoCaller autoCaller(this);
    2945     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    2946 
    29472944    LogFlowThisFuncEnter();
    29482945
    … …  
    29882985                                        ComPtr<IProgress> &aProgress)
    29892986{
    2990     AutoCaller autoCaller(this);
    2991     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    2992 
    29932987    uint32_t fFlags = FileCopyFlag_None;
    29942988    if (aFlags.size())
    … …  
    30163010                                      const std::vector<FileCopyFlag_T> &aFlags, ComPtr<IProgress> &aProgress)
    30173011{
    3018     AutoCaller autoCaller(this);
    3019     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3020 
    30213012    uint32_t fFlags = FileCopyFlag_None;
    30223013    if (aFlags.size())
    … …  
    30503041                                    ComPtr<IProgress> &aProgress)
    30513042{
    3052     AutoCaller autoCaller(this);
    3053     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3054 
    30553043    const size_t cSources = aSources.size();
    30563044    if (   (aFilters.size() && aFilters.size() != cSources)
    … …  
    31303118                                  ComPtr<IProgress> &aProgress)
    31313119{
    3132     AutoCaller autoCaller(this);
    3133     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3134 
    31353120    const size_t cSources = aSources.size();
    31363121    if (   (aFilters.size() && aFilters.size() != cSources)
    … …  
    32113196                                             const std::vector<DirectoryCopyFlag_T> &aFlags, ComPtr<IProgress> &aProgress)
    32123197{
    3213     AutoCaller autoCaller(this);
    3214     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3215 
    32163198    uint32_t fFlags = DirectoryCopyFlag_None;
    32173199    if (aFlags.size())
    … …  
    32403222                                           const std::vector<DirectoryCopyFlag_T> &aFlags, ComPtr<IProgress> &aProgress)
    32413223{
    3242     AutoCaller autoCaller(this);
    3243     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3244 
    32453224    uint32_t fFlags = DirectoryCopyFlag_None;
    32463225    if (aFlags.size())
    … …  
    32703249                                      const std::vector<DirectoryCreateFlag_T> &aFlags)
    32713250{
    3272     AutoCaller autoCaller(this);
    3273     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3274 
    32753251    if (RT_UNLIKELY((aPath.c_str()) == NULL || *(aPath.c_str()) == '\0'))
    32763252        return setError(E_INVALIDARG, tr("No directory to create specified"));
    … …  
    33273303{
    33283304    RT_NOREF(aMode, aSecure); /** @todo r=bird: WTF? */
    3329 
    3330     AutoCaller autoCaller(this);
    3331     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    33323305
    33333306    if (RT_UNLIKELY((aTemplateName.c_str()) == NULL || *(aTemplateName.c_str()) == '\0'))
    … …  
    34243397                                    const std::vector<DirectoryOpenFlag_T> &aFlags, ComPtr<IGuestDirectory> &aDirectory)
    34253398{
    3426     AutoCaller autoCaller(this);
    3427     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3428 
    34293399    if (RT_UNLIKELY((aPath.c_str()) == NULL || *(aPath.c_str()) == '\0'))
    34303400        return setError(E_INVALIDARG, tr("No directory to open specified"));
    … …  
    34843454HRESULT GuestSession::directoryRemove(const com::Utf8Str &aPath)
    34853455{
    3486     AutoCaller autoCaller(this);
    3487     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3488 
    3489     if (RT_UNLIKELY((aPath.c_str()) == NULL || *(aPath.c_str()) == '\0'))
     3456    if (RT_UNLIKELY(aPath.c_str() == NULL || *aPath.c_str() == '\0'))
    34903457        return setError(E_INVALIDARG, tr("No directory to remove specified"));
    34913458
    … …  
    35263493                                               ComPtr<IProgress> &aProgress)
    35273494{
    3528     AutoCaller autoCaller(this);
    3529     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3530 
    3531     if (RT_UNLIKELY((aPath.c_str()) == NULL || *(aPath.c_str()) == '\0'))
     3495    if (RT_UNLIKELY(aPath.c_str() == NULL || *aPath.c_str() == '\0'))
    35323496        return setError(E_INVALIDARG, tr("No directory to remove recursively specified"));
    35333497
    … …  
    37053669HRESULT GuestSession::fileExists(const com::Utf8Str &aPath, BOOL aFollowSymlinks, BOOL *aExists)
    37063670{
    3707     AutoCaller autoCaller(this);
    3708     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3709 
    37103671    /* By default we return non-existent. */
    37113672    *aExists = FALSE;
    … …  
    37723733                                 const std::vector<FileOpenExFlag_T> &aFlags, ComPtr<IGuestFile> &aFile)
    37733734{
    3774     AutoCaller autoCaller(this);
    3775     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3776 
    37773735    if (RT_UNLIKELY((aPath.c_str()) == NULL || *(aPath.c_str()) == '\0'))
    37783736        return setError(E_INVALIDARG, tr("No file to open specified"));
    … …  
    38823840HRESULT GuestSession::fileQuerySize(const com::Utf8Str &aPath, BOOL aFollowSymlinks, LONG64 *aSize)
    38833841{
    3884     AutoCaller autoCaller(this);
    3885     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3886 
    38873842    if (aPath.isEmpty())
    38883843        return setError(E_INVALIDARG, tr("No path specified"));
    … …  
    39113866HRESULT GuestSession::fsObjExists(const com::Utf8Str &aPath, BOOL aFollowSymlinks, BOOL *aExists)
    39123867{
    3913     AutoCaller autoCaller(this);
    3914     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3915 
    39163868    if (aPath.isEmpty())
    39173869        return setError(E_INVALIDARG, tr("No path specified"));
    … …  
    39553907HRESULT GuestSession::fsObjQueryInfo(const com::Utf8Str &aPath, BOOL aFollowSymlinks, ComPtr<IGuestFsObjInfo> &aInfo)
    39563908{
    3957     AutoCaller autoCaller(this);
    3958     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3959 
    39603909    if (aPath.isEmpty())
    39613910        return setError(E_INVALIDARG, tr("No path specified"));
    … …  
    39953944HRESULT GuestSession::fsObjRemove(const com::Utf8Str &aPath)
    39963945{
    3997     AutoCaller autoCaller(this);
    3998     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    3999 
    40003946    if (RT_UNLIKELY(aPath.isEmpty()))
    40013947        return setError(E_INVALIDARG, tr("No path specified"));
    … …  
    40223968HRESULT GuestSession::fsObjRemoveArray(const std::vector<com::Utf8Str> &aPaths, ComPtr<IProgress> &aProgress)
    40233969{
    4024     AutoCaller autoCaller(this);
    4025     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    4026 
    40273970    RT_NOREF(aPaths, aProgress);
    4028 
    40293971    return E_NOTIMPL;
    40303972}
    … …  
    40343976                                  const std::vector<FsObjRenameFlag_T> &aFlags)
    40353977{
    4036     AutoCaller autoCaller(this);
    4037     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    4038 
    40393978    if (RT_UNLIKELY(aSource.isEmpty()))
    40403979        return setError(E_INVALIDARG, tr("No source path specified"));
    … …  
    41404079                                      ComPtr<IGuestProcess> &aGuestProcess)
    41414080{
    4142     AutoCaller autoCaller(this);
    4143     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    4144 
    41454081    HRESULT hr = i_isStartedExternal();
    41464082    if (FAILED(hr))
    … …  
    42474183
    42484184{
    4249     AutoCaller autoCaller(this);
    4250     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    4251 
    42524185    if (aPid == 0)
    42534186        return setError(E_INVALIDARG, tr("No valid process ID (PID) specified"));
    … …  
    42954228HRESULT GuestSession::waitFor(ULONG aWaitFor, ULONG aTimeoutMS, GuestSessionWaitResult_T *aReason)
    42964229{
    4297     AutoCaller autoCaller(this);
    4298     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    4299 
    43004230    /* Note: No call to i_isReadyExternal() needed here, as the session might not has been started (yet). */
    43014231
    … …  
    43414271                                   GuestSessionWaitResult_T *aReason)
    43424272{
    4343     AutoCaller autoCaller(this);
    4344     if (FAILED(autoCaller.rc())) return autoCaller.rc();
    4345 
    43464273    /* Note: No call to i_isReadyExternal() needed here, as the session might not has been started (yet). */
    43474274
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