Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(704)

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: styleguide/c++/c++11.html

Issue 667483003: styleguide: Allow C++11 "enum class" (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git/+/master
Patch Set: Remove recommendation per pkasting@ feedback Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
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 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 <td>Automatic Types</td> 77 <td>Automatic Types</td>
78 <td><code>auto</code></td> 78 <td><code>auto</code></td>
79 <td>Automatic type deduction</td> 79 <td>Automatic type deduction</td>
80 <td>TODO: documentation link</td> 80 <td>TODO: documentation link</td>
81 <td>Use according to the <a 81 <td>Use according to the <a
82 href="https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#auto">Goo gle 82 href="https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#auto">Goo gle
83 Style Guide on <code>auto</code></a>. <a href="https://groups.google.com/a/chrom ium.org/forum/#!topic/chromium-dev/OQyYSfH9m2M">Discussion thread</a></td> 83 Style Guide on <code>auto</code></a>. <a href="https://groups.google.com/a/chrom ium.org/forum/#!topic/chromium-dev/OQyYSfH9m2M">Discussion thread</a></td>
84 </tr> 84 </tr>
85 85
86 <tr> 86 <tr>
87 <td>Enumerated Type Classes</td>
88 <td><code>enum class <i>classname</i></code></td>
89 <td>Provide enums as full classes, with no implicit
90 conversion to booleans or integers</td>
91 <td><a href="http://stackoverflow.com/questions/6936030/do-we-really-need-enum-c lass-in-c11">
92 enum-class</a></td>
93 <td><a href="https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev /Q5WmkAImanc">Discussion thread</a></td>
94 </tr>
jamesr 2014/10/17 23:10:07 we should explicitly call out that enum classes ar
95
96 <tr>
87 <td>Final Specifier</td> 97 <td>Final Specifier</td>
88 <td><code>final</code></td> 98 <td><code>final</code></td>
89 <td> Indicates that a class or function is final and cannot be overridden</td> 99 <td> Indicates that a class or function is final and cannot be overridden</td>
90 <td><a href="http://en.cppreference.com/w/cpp/language/final">final Language Ref erence</a></td> 100 <td><a href="http://en.cppreference.com/w/cpp/language/final">final Language Ref erence</a></td>
91 <td>Recommended for new code. Existing uses of the <code>FINAL</code> macro will be <a href="https://crbug.com/417463">replaced throughout the codebase</a>. <a href="https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/VTNZzi zN0zo">Discussion thread</a></td> 101 <td>Recommended for new code. Existing uses of the <code>FINAL</code> macro will be <a href="https://crbug.com/417463">replaced throughout the codebase</a>. <a href="https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/VTNZzi zN0zo">Discussion thread</a></td>
92 </tr> 102 </tr>
93 103
94 <tr> 104 <tr>
95 <td>Local Types as Template Arguments</td> 105 <td>Local Types as Template Arguments</td>
96 <td></td> 106 <td></td>
(...skipping 230 matching lines...) Expand 10 before | Expand all | Expand 10 after
327 <td>Delegated Constructors</td> 337 <td>Delegated Constructors</td>
328 <td><code>Class() : Class(0) {}</code><br /> 338 <td><code>Class() : Class(0) {}</code><br />
329 <code>Class(<i>type</i> <i>var</i>) : Class(<i>var</i>, 0)</code></td> 339 <code>Class(<i>type</i> <i>var</i>) : Class(<i>var</i>, 0)</code></td>
330 <td>Allow overloaded constructors to use common initialization code</td> 340 <td>Allow overloaded constructors to use common initialization code</td>
331 <td><a href="https://www.ibm.com/developerworks/community/blogs/5894415f-be62-4b c0-81c5-3956e82276f3/entry/introduction_to_the_c_11_feature_delegating_construct ors?lang=en"> 341 <td><a href="https://www.ibm.com/developerworks/community/blogs/5894415f-be62-4b c0-81c5-3956e82276f3/entry/introduction_to_the_c_11_feature_delegating_construct ors?lang=en">
332 Introduction to the C++11 feature: delegating constructors</a></td> 342 Introduction to the C++11 feature: delegating constructors</a></td>
333 <td><a href="https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev /0zVA8Ctx3Xo">Discussion thread</a></td> 343 <td><a href="https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev /0zVA8Ctx3Xo">Discussion thread</a></td>
334 </tr> 344 </tr>
335 345
336 <tr> 346 <tr>
337 <td>Enumerated Type Classes</td>
338 <td><code>enum class <i>classname</i></code></td>
339 <td>Provide enums as full classes, with no implicit
340 conversion to booleans or integers</td>
341 <td><a href="http://stackoverflow.com/questions/6936030/do-we-really-need-enum-c lass-in-c11">
342 enum-class</a></td>
343 <td><a href="https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev /Q5WmkAImanc">Discussion thread</a></td>
344 </tr>
345
346 <tr>
347 <td>Exception Features</td> 347 <td>Exception Features</td>
348 <td><code>noexcept</code>, <code>exception_ptr</code>, 348 <td><code>noexcept</code>, <code>exception_ptr</code>,
349 <code>current_exception()</code>, <code>rethrow_exception</code>, 349 <code>current_exception()</code>, <code>rethrow_exception</code>,
350 and <code>nested_exception</code></td> 350 and <code>nested_exception</code></td>
351 <td>Enhancements to exception throwing and handling</td> 351 <td>Enhancements to exception throwing and handling</td>
352 <td><a href="http://en.cppreference.com/w/cpp/error/exception"> 352 <td><a href="http://en.cppreference.com/w/cpp/error/exception">
353 std::exception</a></td> 353 std::exception</a></td>
354 <td>Exceptions are banned by the 354 <td>Exceptions are banned by the
355 <a href="https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Except ions"> C++ Style Guide</a>. <a href="https://groups.google.com/a/chromium.org/fo rum/#!topic/chromium-dev/8i4tMqNpHhg">Discussion thread</a></td> 355 <a href="https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Except ions"> C++ Style Guide</a>. <a href="https://groups.google.com/a/chromium.org/fo rum/#!topic/chromium-dev/8i4tMqNpHhg">Discussion thread</a></td>
356 </tr> 356 </tr>
(...skipping 584 matching lines...) Expand 10 before | Expand all | Expand 10 after
941 </tr> 941 </tr>
942 942
943 </tbody> 943 </tbody>
944 </table> 944 </table>
945 945
946 </details> 946 </details>
947 947
948 </div> 948 </div>
949 </body> 949 </body>
950 </html> 950 </html>
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698