chromium/src
0
Fork 0
Code Activity

Move absl::Status into allowed feature list

Browse Source 
This CL updates absl::Status as allowed in the Chromium codebase given
the request to allow discussion thread:
https://groups.google.com/a/chromium.org/g/cxx/c/ImdFCSZ-NMA

Change-Id: I7dfb8e1193caa4abfb2d130365c2f773bdd720f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2560079
Reviewed-by: danakj <danakj@chromium.org>
Commit-Queue: Alan Cutter <alancutter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#831439}
This commit is contained in:
Alan Cutter
2020-11-26 23:38:28 +00:00
committed by Commit Bot
parent bc2e0d9505
commit f4b12dce46
1 changed files with 9 additions and 8 deletions

17
styleguide/c++/c++11.html

@ -292,6 +292,15 @@ The current status of existing standards and Abseil features is:
<th style='width:240px;'>Notes and Discussion Thread</th>
</tr>
<tr>
<td>Status</td>
<td><code>absl::Status</code></td>
<td>Type for returning detailed errors.</td>
<td><a href="https://source.chromium.org/chromium/chromium/src/+/master:third_party/abseil-cpp/absl/status/status.h">status.h</a></td>
<td>Approved for use inside a wrapper type. Use <a href="https://source.chromium.org/chromium/chromium/src/+/master:base/strings/abseil_string_conversions.h">abseil_string_conversions.h</a> to convert to and from <a href="https://source.chromium.org/chromium/chromium/src/+/master:third_party/abseil-cpp/absl/strings/string_view.h">absl::string_view</a> so the wrapper can expose <a href="https://source.chromium.org/chromium/chromium/src/+/master:base/strings/string_piece.h">base::StringPiece</a>. Use <a href="https://source.chromium.org/chromium/chromium/src/+/master:third_party/abseil-cpp/absl/strings/cord.h">absl::Cord</a> directly as minimally necessary to interface; do not expose in the wrapper type API.<br>
<a href="https://groups.google.com/a/chromium.org/g/cxx/c/ImdFCSZ-NMA">Discussion thread</a></td>
</tr>
<tr>
<td>Variant</td>
<td><code>absl::variant</code></td>
@ -436,14 +445,6 @@ size_t index = absl::Uniform(bitgen, 0u, elems.size());</code></td>
<td>Overlaps with <code>base/rand_util.h</code>.</td>
</tr>
<tr>
<td>Status</td>
<td><code>absl::Status</code></td>
<td>Type for returning detailed errors.</td>
<td><a href="https://source.chromium.org/chromium/chromium/src/+/master:third_party/abseil-cpp/absl/status/status.h">status.h</a></td>
<td></td>
</tr>
<tr>
<td>StatusOr</td>
<td><code>absl::StatusOr&lt;T&gt;</td>