zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Using absl::string_view to stringify an RTCErrorType.

Browse Source 
Bug: webrtc:10198
Change-Id: Ie7fdba08df219a03ebe2ee5521c2840f28571bba
Reviewed-on: https://webrtc-review.googlesource.com/c/117162
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26277}
This commit is contained in:
Mirko Bonadei
2019-01-16 12:14:29 +01:00
committed by Commit Bot
parent 0acffb5b36
commit 254ecffacf
2 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
api/rtc_error.cc
View File

@ -10,11 +10,12 @@
#include "api/rtc_error.h"
#include "absl/strings/string_view.h"
#include "rtc_base/arraysize.h"
namespace {
static const char* const kRTCErrorTypeNames[] = {
const absl::string_view kRTCErrorTypeNames[] = {
"NONE",
"UNSUPPORTED_OPERATION",
"UNSUPPORTED_PARAMETER",
@ -52,10 +53,9 @@ void RTCError::set_message(std::string message) {
message_ = std::move(message);
}
// TODO(jonasolsson): Change to use absl::string_view when it's available.
std::string ToString(RTCErrorType error) {
absl::string_view ToString(RTCErrorType error) {
int index = static_cast<int>(error);
return std::string(kRTCErrorTypeNames[index]);
return kRTCErrorTypeNames[index];
}
} // namespace webrtc

6
api/rtc_error.h
View File

@ -17,6 +17,7 @@
#include <string>
#include <utility> // For std::move.
#include "absl/strings/string_view.h"
#include "rtc_base/checks.h"
#include "rtc_base/logging.h"
@ -128,8 +129,9 @@ class RTCError {
// Outputs the error as a friendly string. Update this method when adding a new
// error type.
//
// Only intended to be used for logging/disagnostics.
std::string ToString(RTCErrorType error);
// Only intended to be used for logging/diagnostics. The string_view points
// to literal string that lives for the whole duration of the program.
absl::string_view ToString(RTCErrorType error);
#ifdef UNIT_TEST
inline std::ostream& operator<<( // no-presubmit-check TODO(webrtc:8982)