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

Unified Diff: webrtc/base/buffer.cc

Issue 1749693002: rtc::Buffer: Use RTC_DCHECK instead of assert (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 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 | « webrtc/base/buffer.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/base/buffer.cc
diff --git a/webrtc/base/buffer.cc b/webrtc/base/buffer.cc
index d0965a34bf275045959a79f178a3d2300b20d1c9..3096cc92ba5043ba2cc02ef931fd19c7398a4d4c 100644
--- a/webrtc/base/buffer.cc
+++ b/webrtc/base/buffer.cc
@@ -11,13 +11,12 @@
#include "webrtc/base/buffer.h"
#include <algorithm>
-#include <cassert>
#include <utility>
namespace rtc {
Buffer::Buffer() : size_(0), capacity_(0), data_(nullptr) {
- assert(IsConsistent());
+ RTC_DCHECK(IsConsistent());
}
Buffer::Buffer(const Buffer& buf) : Buffer(buf.data(), buf.size()) {
@@ -27,7 +26,7 @@ Buffer::Buffer(Buffer&& buf)
: size_(buf.size()),
capacity_(buf.capacity()),
data_(std::move(buf.data_)) {
- assert(IsConsistent());
+ RTC_DCHECK(IsConsistent());
buf.OnMovedFrom();
}
@@ -38,7 +37,7 @@ Buffer::Buffer(size_t size, size_t capacity)
: size_(size),
capacity_(std::max(size, capacity)),
data_(new uint8_t[capacity_]) {
- assert(IsConsistent());
+ RTC_DCHECK(IsConsistent());
}
// Note: The destructor works even if the buffer has been moved from.
« no previous file with comments | « webrtc/base/buffer.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698