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

Unified Diff: styleguide/c++/c++11.html

Issue 2670873002: Remove base's ALIGNOF/ALIGNAS in favor of alignof/alignas. (Closed)
Patch Set: rebase Created 3 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sandbox/linux/services/credentials.cc ('k') | third_party/crashpad/crashpad/minidump/minidump_context.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: styleguide/c++/c++11.html
diff --git a/styleguide/c++/c++11.html b/styleguide/c++/c++11.html
index b83449f05a047542fe1276351e1708d17437926e..424f5ff389ea605c0766d6d2fd91711dac1f701b 100644
--- a/styleguide/c++/c++11.html
+++ b/styleguide/c++/c++11.html
@@ -368,6 +368,14 @@ template &lt;typename T&gt;<br/>void Function(T&& t) { ... }</code></td>
</tr>
<tr>
+<td>Alignment Features</td>
+<td><code>alignas</code> specifier, <code>alignof</code> operator</td>
+<td>Object alignment</td>
+<td><a href="http://en.cppreference.com/w/cpp/language/alignas">alignas specifier</a>, <a href="http://en.cppreference.com/w/cpp/language/alignof">alignof operator</a></td>
+<td>Use when appropriate. <a href="https://groups.google.com/a/chromium.org/forum/#!topic/cxx/rwXN02jzzq0">Discussion thread</a></td>
+</tr>
+
+<tr>
<td>Begin and End Non-Member Functions</td>
<td><code>std::begin()</code>, <code>std::end()</code></td>
<td>Allows use of free functions on any container, including fixed-size arrays</td>
@@ -702,14 +710,6 @@ std::move(t).f();&nbsp;&nbsp;// second</code></td>
</tr>
<tr>
-<td>Alignment Features</td>
-<td><code>alignas</code> specifier, <code>alignof</code> operator</td>
-<td>Object alignment</td>
-<td><a href="http://en.cppreference.com/w/cpp/language/alignas">alignas specifier</a>, <a href="http://en.cppreference.com/w/cpp/language/alignof">alignof operator</a></td>
-<td>Reevaluate now that MSVS2015 is available. <a href="https://groups.google.com/a/chromium.org/forum/#!topic/cxx/rwXN02jzzq0">Discussion thread</a></td>
-</tr>
-
-<tr>
<td>Attributes</td>
<td><code>[[<i>attribute_name</i>]]</code></td>
<td>Attaches properties to declarations that specific compiler implementations may use.</td>
« no previous file with comments | « sandbox/linux/services/credentials.cc ('k') | third_party/crashpad/crashpad/minidump/minidump_context.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698