VirtualBox

Changeset 85307 in vbox


Ignore:
Timestamp:
Jul 13, 2020 12:38:15 PM (4 years ago)
Author:
vboxsync
Message:

Main/comimpl.xsl,++: Try use the Utf8Str versions of the event stuff where possible. bugref:9790

Location:
trunk/src/VBox/Main
Files:
8 edited

Legend:

Unmodified
Added
Removed

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

    r84618 r85307  
    204204    HRESULT i_onBandwidthGroupChange(IBandwidthGroup *aBandwidthGroup);
    205205    HRESULT i_onStorageDeviceChange(IMediumAttachment *aMediumAttachment, BOOL aRemove, BOOL aSilent);
    206     HRESULT i_onExtraDataChange(IN_BSTR aMachineId, IN_BSTR aKey, IN_BSTR aVal);
     206    HRESULT i_onExtraDataChange(const Bstr &aMachineId, const Bstr &aKey, const Bstr &aVal);
    207207
    208208    HRESULT i_getGuestProperty(const Utf8Str &aName, Utf8Str *aValue, LONG64 *aTimestamp, Utf8Str *aFlags);

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

    r85306 r85307  
    195195                                NATProtocol_T aProto, IN_BSTR aHostIp, uint16_t aHostPort,
    196196                                IN_BSTR aGuestIp, uint16_t aGuestPort);
    197     void i_onNATNetworkChanged(IN_BSTR aNetworkName);
    198     void i_onNATNetworkStartStop(IN_BSTR aNetworkName, BOOL aStart);
    199     void i_onNATNetworkSetting(IN_BSTR aNetworkName, BOOL aEnabled, IN_BSTR aNetwork,
    200                                IN_BSTR aGateway, BOOL aAdvertiseDefaultIpv6RouteEnabled,
     197    void i_onNATNetworkChanged(const Utf8Str &aNetworkName);
     198    void i_onNATNetworkStartStop(const Utf8Str &aNetworkName, BOOL aStart);
     199    void i_onNATNetworkSetting(const Utf8Str &aNetworkName, BOOL aEnabled, const Utf8Str &aNetwork,
     200                               const Utf8Str &aGateway, BOOL aAdvertiseDefaultIpv6RouteEnabled,
    201201                               BOOL fNeedDhcpServer);
    202     void i_onNATNetworkPortForward(IN_BSTR aNetworkName, BOOL create, BOOL fIpv6,
    203                                    IN_BSTR aRuleName, NATProtocol_T proto,
    204                                    IN_BSTR aHostIp, LONG aHostPort,
    205                                    IN_BSTR aGuestIp, LONG aGuestPort);
     202    void i_onNATNetworkPortForward(const Utf8Str &aNetworkName, BOOL create, BOOL fIpv6,
     203                                   const Utf8Str &aRuleName, NATProtocol_T proto,
     204                                   const Utf8Str &aHostIp, LONG aHostPort,
     205                                   const Utf8Str &aGuestIp, LONG aGuestPort);
    206206    void i_onHostNameResolutionConfigurationChange();
    207207

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

    r85300 r85307  
    911911        ULONG actualPercent = 0;
    912912        getPercent(&actualPercent);
    913         ::FireProgressPercentageChangedEvent(pEventSource, mId.toUtf16().raw(), (LONG)actualPercent);
     913        ::FireProgressPercentageChangedEvent(pEventSource, mId.toString(), (LONG)actualPercent);
    914914    }
    915915
    … …  
    10681068    ULONG actualPercent = 0;
    10691069    getPercent(&actualPercent);
    1070     ::FireProgressPercentageChangedEvent(pEventSource, mId.toUtf16().raw(), (LONG)actualPercent);
     1070    ::FireProgressPercentageChangedEvent(pEventSource, mId.toString(), (LONG)actualPercent);
    10711071
    10721072    return S_OK;
    … …  
    11621162        RTSemEventMultiSignal(mCompletedSem);
    11631163
    1164     ::FireProgressTaskCompletedEvent(pEventSource, mId.toUtf16().raw());
     1164    ::FireProgressTaskCompletedEvent(pEventSource, mId.toString());
    11651165
    11661166    return S_OK;

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

    r85301 r85307  
    345345                ComPtr<IExtraDataChangedEvent> pEDCEv = aEvent;
    346346                Bstr strMachineId;
     347                HRESULT hrc = pEDCEv->COMGETTER(MachineId)(strMachineId.asOutParam());
     348                if (FAILED(hrc)) break;
     349
    347350                Bstr strKey;
    348                 Bstr strVal;
    349                 HRESULT hrc = S_OK;
    350 
    351                 hrc = pEDCEv->COMGETTER(MachineId)(strMachineId.asOutParam());
    352                 if (FAILED(hrc)) break;
    353 
    354351                hrc = pEDCEv->COMGETTER(Key)(strKey.asOutParam());
    355352                if (FAILED(hrc)) break;
    356353
     354                Bstr strVal;
    357355                hrc = pEDCEv->COMGETTER(Value)(strVal.asOutParam());
    358356                if (FAILED(hrc)) break;
    … …  
    52245222    AssertComRCReturnRC(autoCaller.rc());
    52255223
    5226     ::FireStorageControllerChangedEvent(mEventSource, Bstr(aMachineId.toString()).raw(), Bstr(aControllerName).raw());
     5224    ::FireStorageControllerChangedEvent(mEventSource, aMachineId.toString(), aControllerName);
    52275225
    52285226    LogFlowThisFunc(("Leaving rc=%#x\n", S_OK));
    … …  
    60656063}
    60666064
    6067 HRESULT Console::i_onExtraDataChange(IN_BSTR aMachineId, IN_BSTR aKey, IN_BSTR aVal)
     6065HRESULT Console::i_onExtraDataChange(const Bstr &aMachineId, const Bstr &aKey, const Bstr &aVal)
    60686066{
    60696067    LogFlowThisFunc(("\n"));
    … …  
    60736071        return autoCaller.rc();
    60746072
    6075     if (!aMachineId)
     6073    if (aMachineId != i_getId())
    60766074        return S_OK;
    60776075
    6078     HRESULT hrc = S_OK;
    6079     Bstr idMachine(aMachineId);
    6080     if (   FAILED(hrc)
    6081         || idMachine != i_getId())
    6082         return hrc;
    6083 
    60846076    /* don't do anything if the VM isn't running */
    6085     SafeVMPtrQuiet ptrVM(this);
    6086     if (ptrVM.isOk())
    6087     {
    6088         if (RTUtf16CmpAscii(aKey, "VBoxInternal2/TurnResetIntoPowerOff") == 0)
    6089         {
    6090             mfTurnResetIntoPowerOff = RTUtf16CmpAscii(aVal, "1") == 0;
     6077    if (aKey == "VBoxInternal2/TurnResetIntoPowerOff")
     6078    {
     6079        SafeVMPtrQuiet ptrVM(this);
     6080        if (ptrVM.isOk())
     6081        {
     6082            mfTurnResetIntoPowerOff = aVal == "1";
    60916083            int vrc = VMR3SetPowerOffInsteadOfReset(ptrVM.rawUVM(), mfTurnResetIntoPowerOff);
    60926084            AssertRC(vrc);
    6093         }
    6094 
    6095         ptrVM.release();
     6085
     6086            ptrVM.release();
     6087        }
    60966088    }
    60976089
    60986090    /* notify console callbacks on success */
    6099     if (SUCCEEDED(hrc))
    6100         ::FireExtraDataChangedEvent(mEventSource, aMachineId, aKey, aVal);
    6101 
    6102     LogFlowThisFunc(("Leaving hrc=%#x\n", hrc));
    6103     return hrc;
     6091    ::FireExtraDataChangedEvent(mEventSource, aMachineId.raw(), aKey.raw(), aVal.raw());
     6092
     6093    LogFlowThisFunc(("Leaving S_OK\n"));
     6094    return S_OK;
    61046095}
    61056096

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

    r85300 r85307  
    789789        if (fDisabled)
    790790            ::FireGuestMonitorChangedEvent(mParent->i_getEventSource(),
    791                                            GuestMonitorChangedEventType_Disabled,  uScreenId, 0, 0, 0, 0);
     791                                           GuestMonitorChangedEventType_Disabled, uScreenId, 0, 0, 0, 0);
    792792        else
    793793            ::FireGuestMonitorChangedEvent(mParent->i_getEventSource(),

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

    r85300 r85307  
    527527
    528528                com::SafeArray<BYTE> data((size_t)cbRead);
    529                 data.initFrom((BYTE*)dataCb.u.read.pvData, cbRead);
     529                data.initFrom((BYTE *)dataCb.u.read.pvData, cbRead);
    530530
    531531                ::FireGuestFileReadEvent(mEventSource, mSession, this, mData.mOffCurrent, cbRead, ComSafeArrayAsInParam(data));

  • trunk/src/VBox/Main/src-server/NATNetworkImpl.cpp

    r85300 r85307  
    171171    data = m->s;
    172172
    173     m->pVirtualBox->i_onNATNetworkSetting(Bstr(m->s.strNetworkName).raw(),
     173    m->pVirtualBox->i_onNATNetworkSetting(m->s.strNetworkName,
    174174                                          m->s.fEnabled,
    175                                           Bstr(m->s.strIPv4NetworkCidr).raw(),
    176                                           Bstr(m->IPv4Gateway).raw(),
     175                                          m->s.strIPv4NetworkCidr,
     176                                          m->IPv4Gateway,
    177177                                          m->s.fAdvertiseDefaultIPv6Route,
    178178                                          m->s.fNeedDhcpServer);
    … …  
    180180    /* Notify listerners listening on this network only */
    181181    ::FireNATNetworkSettingEvent(m->pEventSource,
    182                                  Bstr(m->s.strNetworkName).raw(),
     182                                 m->s.strNetworkName,
    183183                                 m->s.fEnabled,
    184                                  Bstr(m->s.strIPv4NetworkCidr).raw(),
    185                                  Bstr(m->IPv4Gateway).raw(),
     184                                 m->s.strIPv4NetworkCidr,
     185                                 m->IPv4Gateway,
    186186                                 m->s.fAdvertiseDefaultIPv6Route,
    187187                                 m->s.fNeedDhcpServer);
    … …  
    584584    }
    585585
    586     m->pVirtualBox->i_onNATNetworkPortForward(Bstr(m->s.strNetworkName).raw(), TRUE, aIsIpv6,
    587                                               Bstr(aPortForwardRuleName).raw(), aProto,
    588                                               Bstr(aHostIp).raw(), aHostPort,
    589                                               Bstr(aGuestIp).raw(), aGuestPort);
     586    m->pVirtualBox->i_onNATNetworkPortForward(m->s.strNetworkName, TRUE, aIsIpv6,
     587                                              aPortForwardRuleName, aProto,
     588                                              aHostIp, aHostPort,
     589                                              aGuestIp, aGuestPort);
    590590
    591591    /* Notify listerners listening on this network only */
    592     ::FireNATNetworkPortForwardEvent(m->pEventSource, Bstr(m->s.strNetworkName).raw(), TRUE,
    593                                      aIsIpv6, Bstr(aPortForwardRuleName).raw(), aProto,
    594                                      Bstr(aHostIp).raw(), aHostPort,
    595                                      Bstr(aGuestIp).raw(), aGuestPort);
     592    ::FireNATNetworkPortForwardEvent(m->pEventSource, m->s.strNetworkName, TRUE,
     593                                     aIsIpv6, aPortForwardRuleName, aProto,
     594                                     aHostIp, aHostPort,
     595                                     aGuestIp, aGuestPort);
    596596
    597597    return S_OK;
    … …  
    629629    }
    630630
    631     m->pVirtualBox->i_onNATNetworkPortForward(Bstr(m->s.strNetworkName).raw(), FALSE, aIsIpv6,
    632                                               Bstr(aPortForwardRuleName).raw(), proto,
    633                                               Bstr(strHostIP).raw(), u16HostPort,
    634                                               Bstr(strGuestIP).raw(), u16GuestPort);
     631    m->pVirtualBox->i_onNATNetworkPortForward(m->s.strNetworkName, FALSE, aIsIpv6, aPortForwardRuleName, proto,
     632                                              strHostIP, u16HostPort, strGuestIP, u16GuestPort);
    635633
    636634    /* Notify listerners listening on this network only */
    637     ::FireNATNetworkPortForwardEvent(m->pEventSource, Bstr(m->s.strNetworkName).raw(), FALSE,
    638                                      aIsIpv6, Bstr(aPortForwardRuleName).raw(), proto,
    639                                      Bstr(strHostIP).raw(), u16HostPort,
    640                                      Bstr(strGuestIP).raw(), u16GuestPort);
     635    ::FireNATNetworkPortForwardEvent(m->pEventSource, m->s.strNetworkName, FALSE, aIsIpv6, aPortForwardRuleName, proto,
     636                                     strHostIP, u16HostPort, strGuestIP, u16GuestPort);
    641637    return S_OK;
    642638}
    … …  
    854850    if (RT_SUCCESS(m->NATRunner.start(false /* KillProcOnStop */)))
    855851    {
    856         m->pVirtualBox->i_onNATNetworkStartStop(Bstr(m->s.strNetworkName).raw(), TRUE);
     852        m->pVirtualBox->i_onNATNetworkStartStop(m->s.strNetworkName, TRUE);
    857853        return S_OK;
    858854    }
    … …  
    867863{
    868864#ifdef VBOX_WITH_NAT_SERVICE
    869     m->pVirtualBox->i_onNATNetworkStartStop(Bstr(m->s.strNetworkName).raw(), FALSE);
     865    m->pVirtualBox->i_onNATNetworkStartStop(m->s.strNetworkName, FALSE);
    870866
    871867    if (!m->dhcpServer.isNull())

  • trunk/src/VBox/Main/src-server/VirtualBoxImpl.cpp

    r85306 r85307  
    37323732
    37333733/** @todo Unused!!  */
    3734 void VirtualBox::i_onNATNetworkChanged(IN_BSTR aName)
     3734void VirtualBox::i_onNATNetworkChanged(const Utf8Str &aName)
    37353735{
    37363736    ::FireNATNetworkChangedEvent(m->pEventSource, aName);
    37373737}
    37383738
    3739 void VirtualBox::i_onNATNetworkStartStop(IN_BSTR aName, BOOL fStart)
     3739void VirtualBox::i_onNATNetworkStartStop(const Utf8Str &aName, BOOL fStart)
    37403740{
    37413741    ::FireNATNetworkStartStopEvent(m->pEventSource, aName, fStart);
    37423742}
    37433743
    3744 void VirtualBox::i_onNATNetworkSetting(IN_BSTR aNetworkName, BOOL aEnabled,
    3745                                        IN_BSTR aNetwork, IN_BSTR aGateway,
     3744void VirtualBox::i_onNATNetworkSetting(const Utf8Str &aNetworkName, BOOL aEnabled,
     3745                                       const Utf8Str &aNetwork, const Utf8Str &aGateway,
    37463746                                       BOOL aAdvertiseDefaultIpv6RouteEnabled,
    37473747                                       BOOL fNeedDhcpServer)
    … …  
    37513751}
    37523752
    3753 void VirtualBox::i_onNATNetworkPortForward(IN_BSTR aNetworkName, BOOL create, BOOL fIpv6,
    3754                                            IN_BSTR aRuleName, NATProtocol_T proto,
    3755                                            IN_BSTR aHostIp, LONG aHostPort,
    3756                                            IN_BSTR aGuestIp, LONG aGuestPort)
     3753void VirtualBox::i_onNATNetworkPortForward(const Utf8Str &aNetworkName, BOOL create, BOOL fIpv6,
     3754                                           const Utf8Str &aRuleName, NATProtocol_T proto,
     3755                                           const Utf8Str &aHostIp, LONG aHostPort,
     3756                                           const Utf8Str &aGuestIp, LONG aGuestPort)
    37573757{
    37583758    ::FireNATNetworkPortForwardEvent(m->pEventSource, aNetworkName, create, fIpv6, aRuleName, proto,
    … …  
    59195919    natNetwork.queryInterfaceTo(aNetwork.asOutParam());
    59205920
    5921     ::FireNATNetworkCreationDeletionEvent(m->pEventSource, Bstr(aNetworkName).raw(), TRUE);
     5921    ::FireNATNetworkCreationDeletionEvent(m->pEventSource, aNetworkName, TRUE);
    59225922
    59235923    return rc;
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