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: Created July 29, 2014, 4:58 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 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
121 #endif 121 #endif
122 122
123 /* 123 /*
124 * Deprecation pragmas use the compiler to identify refactoring targets. 124 * Deprecation pragmas use the compiler to identify refactoring targets.
125 */ 125 */
126 #if DISABLE_ATL_FORWARDING 126 #if DISABLE_ATL_FORWARDING
127 namespace ATL { 127 namespace ATL {
128 #else 128 #else
129 namespace OLD_ATL { 129 namespace OLD_ATL {
130 #endif 130 #endif
131 //#pragma deprecated( CComBSTR ) 131 #pragma deprecated( CComBSTR )
132 #pragma deprecated( CRect ) 132 #pragma deprecated( CRect )
133 #pragma deprecated( CSimpleArray ) 133 #pragma deprecated( CSimpleArray )
134 } 134 }
135 135
136 /* 136 /*
137 * Note: 137 * Note:
138 * The preprocessor symbols ATLASSERT and ATLTRACE appear in the source. 138 * The preprocessor symbols ATLASSERT and ATLTRACE appear in the source.
139 * These resolve to nothing in Release configurations. 139 * These resolve to nothing in Release configurations.
140 * In Debug configurations, these are non-trivial. 140 * In Debug configurations, these are non-trivial.
141 * ATLASSERT resolves to an expression with _CrtDbgReportW and _CrtDbgBreak. 141 * ATLASSERT resolves to an expression with _CrtDbgReportW and _CrtDbgBreak.
(...skipping 15 matching lines...) Expand all
157 * 157 *
158 * Reference: 158 * Reference:
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 */ 168 */
168 std::wstring to_wstring(ATL::CString& s); 169 std::wstring to_wstring(const ATL::CString& s);
169 170
170 /** 171 /**
171 * Conversion function from std::wstring to ATL::CString 172 * Conversion function from std::wstring to ATL::CString
172 */ 173 */
173 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