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

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

Issue 611623002: Consolidate all banned features into one table. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
146 <td><a href="http://stackoverflow.com/questions/4786649/are-variadic-macros-nons tandard"> 146 <td><a href="http://stackoverflow.com/questions/4786649/are-variadic-macros-nons tandard">
147 Are Variadic macros nonstandard?</a></td> 147 Are Variadic macros nonstandard?</a></td>
148 <td>Usage should be rare. <a href="https://groups.google.com/a/chromium.org/foru m/#!topic/chromium-dev/sRx9j3CQqyA">Discussion thread</a></td> 148 <td>Usage should be rare. <a href="https://groups.google.com/a/chromium.org/foru m/#!topic/chromium-dev/sRx9j3CQqyA">Discussion thread</a></td>
149 </tr> 149 </tr>
150 150
151 <tr> 151 <tr>
152 <td>Variadic Templates</td> 152 <td>Variadic Templates</td>
153 <td><code>template &lt;<i>typename</i> ... <i>arg</i>&gt;</code></td> 153 <td><code>template &lt;<i>typename</i> ... <i>arg</i>&gt;</code></td>
154 <td>Allows templates that accept a variable number of arguments</td> 154 <td>Allows templates that accept a variable number of arguments</td>
155 <td>TODO: documentation link</td> 155 <td>TODO: documentation link</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> 156 <td>Usage should be rare. Use instead of .pump files. <a href="https://groups.go ogle.com/a/chromium.org/forum/#!topic/chromium-dev/6ItymeMXpMc">Discussion threa d</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&amp;&amp; t)</code> and <code>T&amp; operator=(T&amp;&amp; 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>
187
188 <h3 id="blacklist_banned">C++11 Banned Features</h3> 166 <h3 id="blacklist_banned">C++11 Banned Features</h3>
189 167
190 <p>This section will list C++11 features that are not allowed in the Chromium 168 <p>This section will list C++11 features that are not allowed in the Chromium
191 codebase. 169 codebase.
192 </p> 170 </p>
193 171
194 <table id="banned_list" class="unlined striped"> 172 <table id="banned_list" class="unlined striped">
195 <tbody> 173 <tbody>
196 174
197 <tr> 175 <tr>
(...skipping 18 matching lines...) Expand all
216 <td>Function Local Variable</td> 194 <td>Function Local Variable</td>
217 <td><code>__func__</code></td> 195 <td><code>__func__</code></td>
218 <td>Provides a local variable of the name of the enclosing 196 <td>Provides a local variable of the name of the enclosing
219 function for logging purposes</td> 197 function for logging purposes</td>
220 <td><a href="http://www.informit.com/guides/content.aspx?g=cplusplus&amp;seqNum= 338"> 198 <td><a href="http://www.informit.com/guides/content.aspx?g=cplusplus&amp;seqNum= 338">
221 The __func__ Predeclared Identifier is Coming to C++</a></td> 199 The __func__ Predeclared Identifier is Coming to C++</a></td>
222 <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> 200 <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>
223 </tr> 201 </tr>
224 202
225 <tr> 203 <tr>
204 <td>Rvalue References (and Move Semantics)</td>
205 <td><code>T(T&amp;&amp; t)</code> and <code>T&amp; operator=(T&amp;&amp; t)</cod e></td>
206 <td>Reference that only binds to a temporary object</td>
207 <td>TODO: documentation link</td>
208 <td>To be revisited in the future. Allowed in exceptional cases where approved by the OWNERS of src/styleguide/c++/.</td>
209 </tr>
210
211 <tr>
226 <td>UTF-16 and UTF-32 Support (16-Bit and 32-Bit Character Types)</td> 212 <td>UTF-16 and UTF-32 Support (16-Bit and 32-Bit Character Types)</td>
227 <td><code>char16_t</code> and <code>char32_t</code></td> 213 <td><code>char16_t</code> and <code>char32_t</code></td>
228 <td>Provides character types for handling 16-bit 214 <td>Provides character types for handling 16-bit
229 and 32-bit code units (useful for encoding 215 and 32-bit code units (useful for encoding
230 UTF-16 and UTF-32 string data)</td> 216 UTF-16 and UTF-32 string data)</td>
231 <td><a href="http://en.cppreference.com/w/cpp/language/types"> 217 <td><a href="http://en.cppreference.com/w/cpp/language/types">
232 Fundamental types</a></td> 218 Fundamental types</a></td>
233 <td>Doesn't work in MSVS2013. Reevaluate once it does. Non-UTF-8 text is 219 <td>Doesn't work in MSVS2013. Reevaluate once it does. Non-UTF-8 text is
234 banned by the 220 banned by the
235 <a href="https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Non-AS CII_Characters"> 221 <a href="https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Non-AS CII_Characters">
(...skipping 719 matching lines...) Expand 10 before | Expand all | Expand 10 after
955 </tr> 941 </tr>
956 942
957 </tbody> 943 </tbody>
958 </table> 944 </table>
959 945
960 </details> 946 </details>
961 947
962 </div> 948 </div>
963 </body> 949 </body>
964 </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