Changes between Initial Version and Version 1 of Ticket #2940, comment 2


Ignore:
Timestamp:
08/20/2015 05:33:10 PM (5 years ago)
Author:
eric@…
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #2940, comment 2

    initial v1  
    1 Furthermore, this defect was completely avoidable. It arises because a utility function was defined with an argument of type "ATL::CComBSTR" instead of "std::wstring". We have been endeavoring to use the standard library string classes wherever possible. 
     1Furthermore, this defect was completely avoidable. It arises because a utility function was defined with an argument of type `ATL::CComBSTR` instead of `std::wstring`. We have been endeavoring to use the standard library string classes wherever possible. 
    22 
    3 There's exactly one good use of CComBSTR, and that's RIAA for managing system allocation of BSTR arguments to API functions. The use in the present change set is in the signature of a utility function, which is manifestly not necessary. 
     3There's exactly one good use of `CComBSTR`, and that's RIAA for managing system allocation of `BSTR` arguments to API functions. The use in the present change set is in the signature of a utility function, which is manifestly not necessary. 
    44