Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: src/plugin/ATL_Deprecate.h

Issue 5070706781978624: Issue #276 - introduce class BSTR_Argument (Closed)
Left Patch Set: small correction over patch set 2 Created July 29, 2014, 7:22 p.m.
Right Patch Set: clarified documentation for operator&, updated to_wstring(CString) Created July 30, 2014, 2:13 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « adblockplus.gyp ('k') | src/plugin/ATL_Deprecate.cpp » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 /** 1 /**
2 * \file ATL_Deprecate.h Include package for all the ATL headers, with optional forwarding to detect ATL dependencies. 2 * \file ATL_Deprecate.h Include package for all the ATL headers, with optional forwarding to detect ATL dependencies.
3 * 3 *
4 * This header is a nexus for refactoring work to remove ATL. 4 * This header is a nexus for refactoring work to remove ATL.
5 * The main purpose of this mechanism is to determine the exact scope of the ref actoring effort. 5 * The main purpose of this mechanism is to determine the exact scope of the ref actoring effort.
6 * The lifetime of this header is during the work on #276 Stop using ATL https:/ /issues.adblockplus.org/ticket/276 6 * The lifetime of this header is during the work on #276 Stop using ATL https:/ /issues.adblockplus.org/ticket/276
7 * 7 *
8 * This file implements namespace forwarding that allows detection of ATL symbol s. 8 * This file implements namespace forwarding that allows detection of ATL symbol s.
9 * This forwarding is ordinarily disabled, and always disabled in Release config uration for safety. 9 * This forwarding is ordinarily disabled, and always disabled in Release config uration for safety.
10 * When forwarding is disabled, it simply includes the ATL headers. 10 * When forwarding is disabled, it simply includes the ATL headers.
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 * MSDN CString http://msdn.microsoft.com/en-us/library/aa300688%28v=vs.60%29. aspx 159 * MSDN CString http://msdn.microsoft.com/en-us/library/aa300688%28v=vs.60%29. aspx
160 */ 160 */
161 161
162 #include <string> 162 #include <string>
163 /** 163 /**
164 * Conversion function from ATL:CString to std::wstring 164 * Conversion function from ATL:CString to std::wstring
165 * 165 *
166 * The argument cannot be declared 'const' because of the CString API. 166 * The argument cannot be declared 'const' because of the CString API.
167 * Reference argument usually does not require explicit temporaries. 167 * Reference argument usually does not require explicit temporaries.
168 */ 168 */
169 std::wstring to_wstring(ATL::CString& s); 169 std::wstring to_wstring(const ATL::CString& s);
170 170
171 /** 171 /**
172 * Conversion function from std::wstring to ATL::CString 172 * Conversion function from std::wstring to ATL::CString
173 */ 173 */
174 ATL::CString to_CString(const std::wstring& s); 174 ATL::CString to_CString(const std::wstring& s);
LEFTRIGHT

Powered by Google App Engine
This is Rietveld