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

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

Issue 1885183002: Allow constexpr, and use it in a few places. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add guidance Created 4 years, 8 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 | « chrome/browser/themes/theme_properties.cc ('k') | ui/gfx/color_palette.h » ('j') | 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 <title>C++11 use in Chromium</title> 10 <title>C++11 use in Chromium</title>
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 <td><code>auto</code></td> 103 <td><code>auto</code></td>
104 <td>Automatic type deduction</td> 104 <td>Automatic type deduction</td>
105 <td><a href="http://en.cppreference.com/w/cpp/language/auto"> 105 <td><a href="http://en.cppreference.com/w/cpp/language/auto">
106 auto specifier</a></td> 106 auto specifier</a></td>
107 <td>Use according to the <a 107 <td>Use according to the <a
108 href="https://google.github.io/styleguide/cppguide.html#auto">Google 108 href="https://google.github.io/styleguide/cppguide.html#auto">Google
109 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>. <a href=" https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/5-Bt3BJzAo0" >Another discussion thread</a>.</td> 109 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>. <a href=" https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/5-Bt3BJzAo0" >Another discussion thread</a>.</td>
110 </tr> 110 </tr>
111 111
112 <tr> 112 <tr>
113 <td>Constant Expressions</td>
114 <td><code>constexpr</code></td>
115 <td>Compile-time constant expressions</td>
116 <td><a href="http://en.cppreference.com/w/cpp/language/constexpr">constexpr spec ifier</a></td>
117 <td>Prefer to <code>const</code> for variables where possible. Use cautiously on functions. Don't go out of the way to convert existing code. <a href="https://g oogle.github.io/styleguide/cppguide.html#Use_of_constexpr">Google Style Guide</a ></td>
118 </tr>
119
120 <tr>
113 <td>Declared Type Accessor</td> 121 <td>Declared Type Accessor</td>
114 <td><code>decltype(<i>expression</i>)</code></td> 122 <td><code>decltype(<i>expression</i>)</code></td>
115 <td>Provides a means to determine the type of an expression at compile-time, 123 <td>Provides a means to determine the type of an expression at compile-time,
116 useful most often in templates.</td> 124 useful most often in templates.</td>
117 <td><a href="http://en.cppreference.com/w/cpp/language/decltype"> 125 <td><a href="http://en.cppreference.com/w/cpp/language/decltype">
118 decltype specifier</a></td> 126 decltype specifier</a></td>
119 <td>Usage should be rare. <a href="https://groups.google.com/a/chromium.org/foru m/#!topic/chromium-dev/_zoNvZd_dSo">Discussion thread</a></td> 127 <td>Usage should be rare. <a href="https://groups.google.com/a/chromium.org/foru m/#!topic/chromium-dev/_zoNvZd_dSo">Discussion thread</a></td>
120 </tr> 128 </tr>
121 129
122 <tr> 130 <tr>
(...skipping 420 matching lines...) Expand 10 before | Expand all | Expand 10 after
543 <td> 551 <td>
544 <a href="https://codereview.chromium.org/1497963002/">Doesn't work in 552 <a href="https://codereview.chromium.org/1497963002/">Doesn't work in
545 MSVS2013</a>. 553 MSVS2013</a>.
546 <a href="https://msdn.microsoft.com/en-us/library/dn956970.aspx">MSVS2015 554 <a href="https://msdn.microsoft.com/en-us/library/dn956970.aspx">MSVS2015
547 supports them</a>; reevaluate after MSVS2015 is available. 555 supports them</a>; reevaluate after MSVS2015 is available.
548 <a href="https://groups.google.com/a/chromium.org/d/msg/cxx/rwXN02jzzq0/CpUc1ZzM BQAJ">Discussion thread</a> 556 <a href="https://groups.google.com/a/chromium.org/d/msg/cxx/rwXN02jzzq0/CpUc1ZzM BQAJ">Discussion thread</a>
549 </td> 557 </td>
550 </tr> 558 </tr>
551 559
552 <tr> 560 <tr>
553 <td>Constant Expressions</td>
554 <td><code>constexpr</code></td>
555 <td>Compile-time constant expressions</td>
556 <td><a href="http://en.cppreference.com/w/cpp/language/constexpr">
557 constexpr specifier</a></td>
558 <td>Doesn't work in MSVS2013. Reevalute once it does. <a
559 href="https://google.github.io/styleguide/cppguide.html#Use_of_constexpr">Google
560 Style Guide on <code>constexpr</code></a></td>
561 </tr>
562
563 <tr>
564 <td>Function Local Variable</td> 561 <td>Function Local Variable</td>
565 <td><code>__func__</code></td> 562 <td><code>__func__</code></td>
566 <td>Provides a local variable of the name of the enclosing 563 <td>Provides a local variable of the name of the enclosing
567 function for logging purposes</td> 564 function for logging purposes</td>
568 <td><a href="http://www.informit.com/guides/content.aspx?g=cplusplus&amp;seqNum= 338"> 565 <td><a href="http://www.informit.com/guides/content.aspx?g=cplusplus&amp;seqNum= 338">
569 The __func__ Predeclared Identifier is Coming to C++</a></td> 566 The __func__ Predeclared Identifier is Coming to C++</a></td>
570 <td>Doesn't work in MSVS2013. Reevaluate once it does. <a href="https://groups.g oogle.com/a/chromium.org/forum/#!topic/chromium-dev/ojGfcgSDzHM">Discussion thre ad</a></td> 567 <td>Doesn't work in MSVS2013. Reevaluate once it does. <a href="https://groups.g oogle.com/a/chromium.org/forum/#!topic/chromium-dev/ojGfcgSDzHM">Discussion thre ad</a></td>
571 </tr> 568 </tr>
572 569
573 <tr> 570 <tr>
(...skipping 534 matching lines...) Expand 10 before | Expand all | Expand 10 after
1108 C++ Style Guide</a>. However, may be useful for 1105 C++ Style Guide</a>. However, may be useful for
1109 consuming non-ASCII data.</td> 1106 consuming non-ASCII data.</td>
1110 </tr> 1107 </tr>
1111 1108
1112 </tbody> 1109 </tbody>
1113 </table> 1110 </table>
1114 1111
1115 </div> 1112 </div>
1116 </body> 1113 </body>
1117 </html> 1114 </html>
OLDNEW
« no previous file with comments | « chrome/browser/themes/theme_properties.cc ('k') | ui/gfx/color_palette.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698