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

Delta Between Two Patch Sets: src/plugin/COM_Value.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 | « src/plugin/AdblockPlusDomTraverser.cpp ('k') | src/plugin/COM_Value.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 COM_Value.h Support for the values used in COM and Automation. 2 * \file COM_Value.h Support for the values used in COM and Automation.
3 */ 3 */
4 #ifndef COM_VALUE_H 4 #ifndef COM_VALUE_H
5 #define COM_VALUE_H 5 #define COM_VALUE_H
6 6
7 #include <wtypes.h> 7 #include <wtypes.h>
8 #include <string> 8 #include <string>
9 9
10 namespace AdblockPlus 10 namespace AdblockPlus
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 operator std::wstring() const; 83 operator std::wstring() const;
84 84
85 /** 85 /**
86 * Address-of operator. 86 * Address-of operator.
87 * 87 *
88 * This operator is used for assignment directly into our underlying point er. 88 * This operator is used for assignment directly into our underlying point er.
89 * In order to avoid leaking memory, this operator also implicitly assigns the null string. 89 * In order to avoid leaking memory, this operator also implicitly assigns the null string.
90 * Specifically, this operator is not 'const'. 90 * Specifically, this operator is not 'const'.
91 * 91 *
92 * \par postcondition 92 * \par postcondition
93 * bstr == nullptr 93 * - bstr == nullptr
94 * - return value is not null
95 * - does not throw
94 */ 96 */
95 BSTR* operator&(); 97 BSTR* operator&(); // noexcept
96 98
97 private: 99 private:
98 /** 100 /**
99 * Copy constructor is deleted 101 * Copy constructor is deleted
100 */ 102 */
101 BSTR_Argument(const BSTR_Argument&); // = delete 103 BSTR_Argument(const BSTR_Argument&); // = delete
102 104
103 /** 105 /**
104 * Move constructor is deleted 106 * Move constructor is deleted
105 */ 107 */
106 BSTR_Argument(BSTR_Argument&&); // = delete 108 BSTR_Argument(BSTR_Argument&&); // = delete
107 109
108 /** 110 /**
109 * Copy assignment is deleted 111 * Copy assignment is deleted
110 */ 112 */
111 BSTR_Argument& operator=(const BSTR_Argument&); // = delete 113 BSTR_Argument& operator=(const BSTR_Argument&); // = delete
112 114
113 /** 115 /**
114 * Move assignment is deleted 116 * Move assignment is deleted
115 */ 117 */
116 BSTR_Argument& operator=(BSTR_Argument&&); // = delete 118 BSTR_Argument& operator=(BSTR_Argument&&); // = delete
117 }; 119 };
118 } 120 }
119 } 121 }
120 122
121 #endif 123 #endif
LEFTRIGHT

Powered by Google App Engine
This is Rietveld