Chromium Code Reviews Chromium Code Reviews
Issue 607773002:
styleguide: Move Rvalue References to the Blacklist with Exceptions. (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> | |
|
Avi (use Gerrit)
2014/09/26 20:33:23
Is there a reason you put this in a different tabl
| |
| 187 | |
| 166 <h3 id="blacklist_banned">C++11 Banned Features</h3> | 188 <h3 id="blacklist_banned">C++11 Banned Features</h3> |
| 167 | 189 |
| 168 <p>This section will list C++11 features that are not allowed in the Chromium | 190 <p>This section will list C++11 features that are not allowed in the Chromium |
| 169 codebase. | 191 codebase. |
| 170 </p> | 192 </p> |
| 171 | 193 |
| 172 <table id="banned_list" class="unlined striped"> | 194 <table id="banned_list" class="unlined striped"> |
| 173 <tbody> | 195 <tbody> |
| 174 | 196 |
| 175 <tr> | 197 <tr> |
| (...skipping 231 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 407 <tr> | 429 <tr> |
| 408 <td>Raw String Literals</td> | 430 <td>Raw String Literals</td> |
| 409 <td><code>string <i>var</i>=R"(<i>raw_string</i>)";</code></td> | 431 <td><code>string <i>var</i>=R"(<i>raw_string</i>)";</code></td> |
| 410 <td>Allows a string to be encoded without any escape | 432 <td>Allows a string to be encoded without any escape |
| 411 sequences, easing parsing in regex expressions, for example</td> | 433 sequences, easing parsing in regex expressions, for example</td> |
| 412 <td>TODO: documentation link</td> | 434 <td>TODO: documentation link</td> |
| 413 <td><a href="https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev /2kWQHbbuMHI">Discussion thread</a></td> | 435 <td><a href="https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev /2kWQHbbuMHI">Discussion thread</a></td> |
| 414 </tr> | 436 </tr> |
| 415 | 437 |
| 416 <tr> | 438 <tr> |
| 417 <td>Rvalue References (and Move Semantics)</td> | |
| 418 <td><code>T(T&& t)</code> and <code>T& operator=(T&& t)</cod e></td> | |
| 419 <td>Reference that only binds to a temporary object</td> | |
| 420 <td>TODO: documentation link</td> | |
| 421 <td></td> | |
| 422 </tr> | |
| 423 | |
| 424 <tr> | |
| 425 <td>Union Class Members</td> | 439 <td>Union Class Members</td> |
| 426 <td><code>union <i>name</i> { <i>class</i> <i>var</i>}</code></td> | 440 <td><code>union <i>name</i> { <i>class</i> <i>var</i>}</code></td> |
| 427 <td>Allows class type members</td> | 441 <td>Allows class type members</td> |
| 428 <td><a href="http://en.cppreference.com/w/cpp/language/union"> | 442 <td><a href="http://en.cppreference.com/w/cpp/language/union"> |
| 429 Union declarations</a></td> | 443 Union declarations</a></td> |
| 430 <td></td> | 444 <td></td> |
| 431 </tr> | 445 </tr> |
| 432 | 446 |
| 433 <tr> | 447 <tr> |
| 434 <td>User-Defined Literals</td> | 448 <td>User-Defined Literals</td> |
| (...skipping 506 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 941 </tr> | 955 </tr> |
| 942 | 956 |
| 943 </tbody> | 957 </tbody> |
| 944 </table> | 958 </table> |
| 945 | 959 |
| 946 </details> | 960 </details> |
| 947 | 961 |
| 948 </div> | 962 </div> |
| 949 </body> | 963 </body> |
| 950 </html> | 964 </html> |
| OLD | NEW |
