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

Unified Diff: webrtc/api/rtcerror.h

Issue 2762963002: Make lint errors fatal in presubmit and fix files in whitelisted paths (Closed)
Patch Set: Make lint errors fatal in presubmit Created 3 years, 9 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 | « PRESUBMIT.py ('k') | webrtc/modules/remote_bitrate_estimator/bwe_simulations.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/api/rtcerror.h
diff --git a/webrtc/api/rtcerror.h b/webrtc/api/rtcerror.h
index 2ba78371eb0034f17b19833be32dc9fddf6fdc59..5cb123227d4c2b4521b73e54361e0cace3fde31a 100644
--- a/webrtc/api/rtcerror.h
+++ b/webrtc/api/rtcerror.h
@@ -203,7 +203,7 @@ class RTCErrorOr {
// is marked 'explicit' to try to catch cases like 'return {};', where people
// think RTCErrorOr<std::vector<int>> will be initialized with an empty
// vector, instead of a RTCErrorType::INTERNAL_ERROR error.
- explicit RTCErrorOr() : error_(RTCErrorType::INTERNAL_ERROR) {}
+ RTCErrorOr() : error_(RTCErrorType::INTERNAL_ERROR) {}
// Constructs a new RTCErrorOr with the given non-ok error. After calling
// this constructor, calls to value() will DCHECK-fail.
@@ -213,7 +213,7 @@ class RTCErrorOr {
// RTCError(...)' when the return type is RTCErrorOr<T>.
//
// REQUIRES: !error.ok(). This requirement is DCHECKed.
- RTCErrorOr(RTCError&& error) : error_(std::move(error)) {
+ RTCErrorOr(RTCError&& error) : error_(std::move(error)) { // NOLINT
RTC_DCHECK(!error.ok());
}
@@ -224,7 +224,7 @@ class RTCErrorOr {
// NOTE: Not explicit - we want to use RTCErrorOr<T> as a return type
// so it is convenient and sensible to be able to do 'return T()'
// when the return type is RTCErrorOr<T>.
- RTCErrorOr(T&& value) : value_(std::move(value)) {}
+ RTCErrorOr(T&& value) : value_(std::move(value)) {} // NOLINT
// Delete the copy constructor and assignment operator; there aren't any use
// cases where you should need to copy an RTCErrorOr, as opposed to moving
@@ -248,7 +248,7 @@ class RTCErrorOr {
// Conversion constructor and assignment operator; T must be copy or move
// constructible from U.
template <typename U>
- RTCErrorOr(RTCErrorOr<U> other)
+ RTCErrorOr(RTCErrorOr<U> other) // NOLINT
: error_(std::move(other.error_)), value_(std::move(other.value_)) {}
template <typename U>
RTCErrorOr& operator=(RTCErrorOr<U> other) {
« no previous file with comments | « PRESUBMIT.py ('k') | webrtc/modules/remote_bitrate_estimator/bwe_simulations.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698