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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 691803004: styleguide: ban inherited constructors (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: s/will list/lists/ Created 6 years, 1 month 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 179 matching lines...) Expand 10 before | Expand all | Expand 10 after
190 190
191 </tbody> 191 </tbody>
192 </table> 192 </table>
193 193
194 <h2 id="blacklist">C++11 Blacklist (Disallowed and Banned Features)</h2> 194 <h2 id="blacklist">C++11 Blacklist (Disallowed and Banned Features)</h2>
195 195
196 <p>This section lists features that are not allowed to be used yet. 196 <p>This section lists features that are not allowed to be used yet.
197 197
198 <h3 id="blacklist_banned">C++11 Banned Features</h3> 198 <h3 id="blacklist_banned">C++11 Banned Features</h3>
199 199
200 <p>This section will list C++11 features that are not allowed in the Chromium 200 <p>This section lists C++11 features that are not allowed in the Chromium
201 codebase. 201 codebase.
202 </p> 202 </p>
203 203
204 <table id="banned_list" class="unlined striped"> 204 <table id="banned_list" class="unlined striped">
205 <tbody> 205 <tbody>
206 206
207 <tr> 207 <tr>
208 <th style='width:240px;'>Feature or Library</th> 208 <th style='width:240px;'>Feature or Library</th>
209 <th style='width:240px;'>Snippet</th> 209 <th style='width:240px;'>Snippet</th>
210 <th style='width:240px;'>Description</th> 210 <th style='width:240px;'>Description</th>
(...skipping 25 matching lines...) Expand all
236 <td>Function Local Variable</td> 236 <td>Function Local Variable</td>
237 <td><code>__func__</code></td> 237 <td><code>__func__</code></td>
238 <td>Provides a local variable of the name of the enclosing 238 <td>Provides a local variable of the name of the enclosing
239 function for logging purposes</td> 239 function for logging purposes</td>
240 <td><a href="http://www.informit.com/guides/content.aspx?g=cplusplus&amp;seqNum= 338"> 240 <td><a href="http://www.informit.com/guides/content.aspx?g=cplusplus&amp;seqNum= 338">
241 The __func__ Predeclared Identifier is Coming to C++</a></td> 241 The __func__ Predeclared Identifier is Coming to C++</a></td>
242 <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> 242 <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>
243 </tr> 243 </tr>
244 244
245 <tr> 245 <tr>
246 <td>Inherited Constructors</td>
247 <td><code>class Derived : Base {
248 <br />&nbsp;&nbsp;using Base::Base;
249 <br />};</code></td>
250 <td>Allow derived classes to inherit constructors from base classes</td>
251 <td><a href="http://en.cppreference.com/w/cpp/language/using_declaration">Using- declaration</a></td>
252 <td>Doesn't work in MSVS2013. Reevaluate once it does. <a
253 href="https://groups.google.com/a/chromium.org/d/msg/chromium-dev/BULzgIKZ-Ao/PL O7_GoVNvYJ">Discussion thread</a></td>
254 </tr>
255
256 <tr>
246 <td><code>long long</code> Type</td> 257 <td><code>long long</code> Type</td>
247 <td><code>long long <i>var</i>= <i>value</i>;</code></td> 258 <td><code>long long <i>var</i>= <i>value</i>;</code></td>
248 <td>An integer of at least 64 bits</td> 259 <td>An integer of at least 64 bits</td>
249 <td><a href="http://en.cppreference.com/w/cpp/language/types"> 260 <td><a href="http://en.cppreference.com/w/cpp/language/types">
250 Fundamental types</a></td> 261 Fundamental types</a></td>
251 <td>Use an stdint.h type if you need a 64bit number. <a href="https://groups.goo gle.com/a/chromium.org/forum/#!topic/chromium-dev/RxugZ-pIDxk">Discussion thread </a></td> 262 <td>Use an stdint.h type if you need a 64bit number. <a href="https://groups.goo gle.com/a/chromium.org/forum/#!topic/chromium-dev/RxugZ-pIDxk">Discussion thread </a></td>
252 </tr> 263 </tr>
253 264
254 <tr> 265 <tr>
255 <td>Raw String Literals</td> 266 <td>Raw String Literals</td>
(...skipping 689 matching lines...) Expand 10 before | Expand all | Expand 10 after
945 </tr> 956 </tr>
946 957
947 </tbody> 958 </tbody>
948 </table> 959 </table>
949 960
950 </details> 961 </details>
951 962
952 </div> 963 </div>
953 </body> 964 </body>
954 </html> 965 </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