diff options
author | dcheng <dcheng@chromium.org> | 2014-09-24 15:12:10 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-09-24 22:12:19 +0000 |
commit | 5f98d33d5566474db7b31e0960fb09cef0e7c990 (patch) | |
tree | 40848e180f3a631caa7e8dff63401b3b75531b2f /styleguide | |
parent | 4253aba6222fecb87d07eebe50a2e34a034fb19c (diff) | |
download | chromium_src-5f98d33d5566474db7b31e0960fb09cef0e7c990.zip chromium_src-5f98d33d5566474db7b31e0960fb09cef0e7c990.tar.gz chromium_src-5f98d33d5566474db7b31e0960fb09cef0e7c990.tar.bz2 |
Move override/final to the approved C++11 features section.
Discussion thread:
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/VTNZzizN0zo
NOTRY=true
Review URL: https://codereview.chromium.org/596723003
Cr-Commit-Position: refs/heads/master@{#296541}
Diffstat (limited to 'styleguide')
-rw-r--r-- | styleguide/c++/c++11.html | 52 |
1 files changed, 26 insertions, 26 deletions
diff --git a/styleguide/c++/c++11.html b/styleguide/c++/c++11.html index edebb9a..6b9ff82 100644 --- a/styleguide/c++/c++11.html +++ b/styleguide/c++/c++11.html @@ -66,6 +66,14 @@ C++ Templates Angle Brackets Pitfall</a></td> </tr> <tr> +<td>Final Specifier</td> +<td><code>final</code></td> +<td> Indicates that a class or function is final and cannot be overridden</td> +<td><a href="http://en.cppreference.com/w/cpp/language/final">final Language Reference</a></td> +<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/VTNZzizN0zo">Discussion</a></td> +</tr> + +<tr> <td>Local Types as Template Arguments</td> <td></td> <td>Allows local and unnamed types as template arguments</td> @@ -75,16 +83,6 @@ Local types, types without linkage and unnamed types as template arguments</a></ </tr> <tr> -<td>Standard Integers</td> -<td>Typedefs within <code><stdint.h></code> -and <code><inttypes></code></td> -<td>Provides fixed-size integers independent of platforms</td> -<td><a href="http://www.cplusplus.com/reference/cstdint/"> -<stdint.h> (cstdint)</a></td> -<td>Already in common use in the codebase. Approved without discussion.</td> -</tr> - -<tr> <td>Null Pointer Constant</td> <td><code>nullptr</code></td> <td>Declares a type-safe null pointer</td> @@ -96,6 +94,24 @@ Note: <code>std::nullptr_t</code> is a library feature and not available. </td> </tr> +<tr> +<td>Override Specifier</td> +<td><code>override</code></td> +<td>Indicates that a class or function overrides a base implementation</td> +<td><a href="http://en.cppreference.com/w/cpp/language/override">override Language Reference</a></td> +<td>Recommended for new code. Existing uses of the <code>OVERRIDE</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/VTNZzizN0zo">Discussion</a></td> +</tr> + +<tr> +<td>Standard Integers</td> +<td>Typedefs within <code><stdint.h></code> +and <code><inttypes></code></td> +<td>Provides fixed-size integers independent of platforms</td> +<td><a href="http://www.cplusplus.com/reference/cstdint/"> +<stdint.h> (cstdint)</a></td> +<td>Already in common use in the codebase. Approved without discussion.</td> +</tr> + </tbody> </table> @@ -267,14 +283,6 @@ explicit specifier</a></td> </tr> <tr> -<td>Final Declarations</td> -<td><code>final</code></td> -<td>Indicates that a class or function is final and cannot be overridden</td> -<td>TODO: documentation link</td> -<td><code>FINAL</code> is already widely used in the codebase.</td> -</tr> - -<tr> <td>Function Local Variable</td> <td><code>__func__</code></td> <td>Provides a local variable of the name of the enclosing @@ -343,14 +351,6 @@ Non-static data members</a></td> </tr> <tr> -<td>Overrides</td> -<td><code>override</code></td> -<td>Indicates that a class or function overrides a base implementation</td> -<td>TODO: documentation link</td> -<td><code>OVERRIDE</code> is already widely used in the codebase.</td> -</tr> - -<tr> <td>Range-Based For Loops</td> <td><code>for (<i>type</i> <i>var</i> : <i>range</i>)</code></td> <td>Facilitates a more concise syntax for iterating over the elements |