Chromium Code Reviews Chromium Code Reviews
Issue 611523003:
styleguide: Put RValue references in the right place. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| 1 <!DOCTYPE html> | 1 <!DOCTYPE html> |
| 2 <!-- | 2 <!-- |
| 3 Copyright 2014 The Chromium Authors. All rights reserved. | 3 Copyright 2014 The Chromium Authors. All rights reserved. |
| 4 Use of this source code is governed by a BSD-style license that can be | 4 Use of this source code is governed by a BSD-style license that can be |
| 5 found in the LICENSE file. | 5 found in the LICENSE file. |
| 6 --> | 6 --> |
| 7 <html> | 7 <html> |
| 8 <head> | 8 <head> |
| 9 <meta charset="utf-8"> | 9 <meta charset="utf-8"> |
| 10 <link rel="stylesheet" href="c++11.css"> | 10 <link rel="stylesheet" href="c++11.css"> |
| (...skipping 145 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 156 <td>Usage should be rare. Use instead of .pump files.<a href="https://groups.goo gle.com/a/chromium.org/forum/#!topic/chromium-dev/6ItymeMXpMc">Discussion thread </a></td> | 156 <td>Usage should be rare. Use instead of .pump files.<a href="https://groups.goo gle.com/a/chromium.org/forum/#!topic/chromium-dev/6ItymeMXpMc">Discussion thread </a></td> |
| 157 </tr> | 157 </tr> |
| 158 | 158 |
| 159 </tbody> | 159 </tbody> |
| 160 </table> | 160 </table> |
| 161 | 161 |
| 162 <h2 id="blacklist">C++11 Blacklist (Disallowed and Banned Features)</h2> | 162 <h2 id="blacklist">C++11 Blacklist (Disallowed and Banned Features)</h2> |
| 163 | 163 |
| 164 <p>This section lists features that are not allowed to be used yet. | 164 <p>This section lists features that are not allowed to be used yet. |
| 165 | 165 |
| 166 <table id="banned_list" class="unlined striped"> | |
| 167 <tbody> | |
| 168 | |
| 169 <tr> | |
| 170 <th style='width:240px;'>Feature or Library</th> | |
| 171 <th style='width:240px;'>Snippet</th> | |
| 172 <th style='width:240px;'>Description</th> | |
| 173 <th style='width:240px;'>Documentation Link</th> | |
| 174 <th style='width:240px;'>Notes</th> | |
| 175 </tr> | |
| 176 | |
| 177 <tr> | |
| 178 <td>Rvalue References (and Move Semantics)</td> | |
| 179 <td><code>T(T&& t)</code> and <code>T& operator=(T&& t)</cod e></td> | |
| 180 <td>Reference that only binds to a temporary object</td> | |
| 181 <td>TODO: documentation link</td> | |
| 182 <td>To be revisited in the future. Allowed in exceptional cases where approved by the OWNERS of src/styleguide/c++/.</td> | |
| 183 </tr> | |
| 184 | |
| 185 </tbody> | |
| 186 </table> | |
| 187 | |
| 188 <h3 id="blacklist_banned">C++11 Banned Features</h3> | 166 <h3 id="blacklist_banned">C++11 Banned Features</h3> |
| 189 | 167 |
| 190 <p>This section will list C++11 features that are not allowed in the Chromium | 168 <p>This section will list C++11 features that are not allowed in the Chromium |
| 191 codebase. | 169 codebase. |
| 192 </p> | 170 </p> |
| 193 | 171 |
| 194 <table id="banned_list" class="unlined striped"> | 172 <table id="banned_list" class="unlined striped"> |
| 195 <tbody> | 173 <tbody> |
| 196 | 174 |
| 197 <tr> | 175 <tr> |
| (...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 239 | 217 |
| 240 <tr> | 218 <tr> |
| 241 <td>UTF-8, UTF-16, UTF-32 String Literals</td> | 219 <td>UTF-8, UTF-16, UTF-32 String Literals</td> |
| 242 <td><code>u8"<i>string</i>", u"<i>string</i>", U"<i>str ing</i>"</code></td> | 220 <td><code>u8"<i>string</i>", u"<i>string</i>", U"<i>str ing</i>"</code></td> |
| 243 <td>Enforces UTF-8, UTF-16, UTF-32 encoding on all string literals</td> | 221 <td>Enforces UTF-8, UTF-16, UTF-32 encoding on all string literals</td> |
| 244 <td><a href="http://en.cppreference.com/w/cpp/language/string_literal"> | 222 <td><a href="http://en.cppreference.com/w/cpp/language/string_literal"> |
| 245 string literal</a></td> | 223 string literal</a></td> |
| 246 <td>Does not yet work in MSVS2013. Reevaluate once it does. <a href="https://gro ups.google.com/a/chromium.org/forum/#!topic/chromium-dev/gcoUbcjfsII">Discussion thread</a></td> | 224 <td>Does not yet work in MSVS2013. Reevaluate once it does. <a href="https://gro ups.google.com/a/chromium.org/forum/#!topic/chromium-dev/gcoUbcjfsII">Discussion thread</a></td> |
| 247 </tr> | 225 </tr> |
| 248 | 226 |
| 227 <tr> | |
| 228 <td>Rvalue References (and Move Semantics)</td> | |
| 229 <td><code>T(T&& t)</code> and <code>T& operator=(T&& t)</cod e></td> | |
| 230 <td>Reference that only binds to a temporary object</td> | |
| 231 <td>TODO: documentation link</td> | |
| 232 <td>To be revisited in the future. Allowed in exceptional cases where approved by the OWNERS of src/styleguide/c++/.</td> | |
| 233 </tr> | |
|
Avi (use Gerrit)
2014/09/26 21:03:58
alphabetical?
| |
| 234 | |
| 249 </tbody> | 235 </tbody> |
| 250 </table> | 236 </table> |
| 251 | 237 |
| 252 <h3 id="blacklist_review">C++11 Features To Be Discussed</h3> | 238 <h3 id="blacklist_review">C++11 Features To Be Discussed</h3> |
| 253 | 239 |
| 254 <p>The following C++ language features are currently disallowed. | 240 <p>The following C++ language features are currently disallowed. |
| 255 See the top of this page on how to propose moving a feature from this list | 241 See the top of this page on how to propose moving a feature from this list |
| 256 into the allowed or banned sections. Note that not all of these features | 242 into the allowed or banned sections. Note that not all of these features |
| 257 work in all our compilers yet.</p> | 243 work in all our compilers yet.</p> |
| 258 | 244 |
| (...skipping 696 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 955 </tr> | 941 </tr> |
| 956 | 942 |
| 957 </tbody> | 943 </tbody> |
| 958 </table> | 944 </table> |
| 959 | 945 |
| 960 </details> | 946 </details> |
| 961 | 947 |
| 962 </div> | 948 </div> |
| 963 </body> | 949 </body> |
| 964 </html> | 950 </html> |
| OLD | NEW |
