Chromium Code Reviews Chromium Code Reviews
Issue 1450013002:
styleguide: Allow std::move for move semantics. (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 <title>C++11 use in Chromium</title> | 10 <title>C++11 use in Chromium</title> |
| (...skipping 270 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 281 <td>Allows getting rid of <a href="http://crbug.com/554289">ScopedVector</a></td > | 281 <td>Allows getting rid of <a href="http://crbug.com/554289">ScopedVector</a></td > |
| 282 </tr> | 282 </tr> |
| 283 | 283 |
| 284 <tr> | 284 <tr> |
| 285 <td>Lexicographical struct comparison</td> | 285 <td>Lexicographical struct comparison</td> |
| 286 <td><code>tie(a, b, c) <<br> tie(rhs.a, rhs.b, rhs.c)</code></td> | 286 <td><code>tie(a, b, c) <<br> tie(rhs.a, rhs.b, rhs.c)</code></td> |
| 287 <td>Idiom for <code>operator<</code> implementation</td> | 287 <td>Idiom for <code>operator<</code> implementation</td> |
| 288 <td><a href="http://en.cppreference.com/w/cpp/utility/tuple/tie">std::tie</a></t d> | 288 <td><a href="http://en.cppreference.com/w/cpp/utility/tuple/tie">std::tie</a></t d> |
| 289 <td>General use of <code>std::tuple</code>, and <code>std::tie</code> for unpack ing or multiple assignments is still not allowed. <a href="https://groups.google .com/a/chromium.org/d/topic/cxx/3DZ64dIMRTY/discussion">Discussion thread</a></t d> | 289 <td>General use of <code>std::tuple</code>, and <code>std::tie</code> for unpack ing or multiple assignments is still not allowed. <a href="https://groups.google .com/a/chromium.org/d/topic/cxx/3DZ64dIMRTY/discussion">Discussion thread</a></t d> |
| 290 </tr> | 290 </tr> |
| 291 | |
| 292 <tr> | |
| 293 <td>Move Semantics</td> | |
| 294 <td><code>std::move()</code></td> | |
| 295 <td>Facilitates efficient move operations</td> | |
| 296 <td><a href="http://en.cppreference.com/w/cpp/utility/move"> | |
| 297 <code>std::move</code> reference</a></td> | |
| 298 <td></td> | |
|
Nico
2015/11/16 18:47:04
Can you put "Note: std::move() is allowed but writ
| |
| 299 </tr> | |
| 300 | |
| 291 </tbody> | 301 </tbody> |
| 292 </table> | 302 </table> |
| 293 | 303 |
| 294 <h2 id="blacklist">C++11 Blacklist (Disallowed and Banned Features)</h2> | 304 <h2 id="blacklist">C++11 Blacklist (Disallowed and Banned Features)</h2> |
| 295 | 305 |
| 296 <p>This section lists features that are not allowed to be used yet. | 306 <p>This section lists features that are not allowed to be used yet. |
| 297 | 307 |
| 298 <h3 id="blacklist_banned">C++11 Banned Features</h3> | 308 <h3 id="blacklist_banned">C++11 Banned Features</h3> |
| 299 | 309 |
| 300 <p>This section lists C++11 features that are not allowed in the Chromium | 310 <p>This section lists C++11 features that are not allowed in the Chromium |
| (...skipping 501 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 802 | 812 |
| 803 <tr> | 813 <tr> |
| 804 <td>Initializer Lists</td> | 814 <td>Initializer Lists</td> |
| 805 <td><code>std::initializer_list<T></code></td> | 815 <td><code>std::initializer_list<T></code></td> |
| 806 <td>Allows containers to be initialized with aggregate elements</td> | 816 <td>Allows containers to be initialized with aggregate elements</td> |
| 807 <td>TODO: documentation link</td> | 817 <td>TODO: documentation link</td> |
| 808 <td></td> | 818 <td></td> |
| 809 </tr> | 819 </tr> |
| 810 | 820 |
| 811 <tr> | 821 <tr> |
| 812 <td>Move Semantics</td> | |
| 813 <td><code>std::move()</code></td> | |
| 814 <td>Facilitates efficient move operations</td> | |
| 815 <td><a href="http://en.cppreference.com/w/cpp/utility/move"> | |
| 816 <code>std::move</code> reference</a></td> | |
| 817 <td></td> | |
| 818 </tr> | |
| 819 | |
| 820 <tr> | |
| 821 <td>Pointer Traits Class Template</td> | 822 <td>Pointer Traits Class Template</td> |
| 822 <td><code>std::pointer_traits</code></td> | 823 <td><code>std::pointer_traits</code></td> |
| 823 <td>Provides a standard way to access properties | 824 <td>Provides a standard way to access properties |
| 824 of pointers and pointer-like types</td> | 825 of pointers and pointer-like types</td> |
| 825 <td><a href="http://en.cppreference.com/w/cpp/memory/pointer_traits"> | 826 <td><a href="http://en.cppreference.com/w/cpp/memory/pointer_traits"> |
| 826 std::pointer_traits</a></td> | 827 std::pointer_traits</a></td> |
| 827 <td>Useful for determining the element type | 828 <td>Useful for determining the element type |
| 828 pointed at by a (possibly smart) pointer.</td> | 829 pointed at by a (possibly smart) pointer.</td> |
| 829 </tr> | 830 </tr> |
| 830 | 831 |
| (...skipping 203 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1034 C++ Style Guide</a>. However, may be useful for | 1035 C++ Style Guide</a>. However, may be useful for |
| 1035 consuming non-ASCII data.</td> | 1036 consuming non-ASCII data.</td> |
| 1036 </tr> | 1037 </tr> |
| 1037 | 1038 |
| 1038 </tbody> | 1039 </tbody> |
| 1039 </table> | 1040 </table> |
| 1040 | 1041 |
| 1041 </div> | 1042 </div> |
| 1042 </body> | 1043 </body> |
| 1043 </html> | 1044 </html> |
| OLD | NEW |
