From 8fb355705211c60b7ea610eb0b69543dc5f7f8bf Mon Sep 17 00:00:00 2001 From: kwiberg Date: Thu, 11 Feb 2016 13:36:43 -0800 Subject: [PATCH] rtc::Buffer: Replace an internal rtc::scoped_ptr with std::unique_ptr We'd like to completely replace rtc::scoped_ptr with std::unique_ptr. This is a first trial CL to see if using unique_ptr causes any problems. (As a side effect of removing the scoped_ptr.h include in buffer.h, I had to fix broken includes in no less than three files.) BUG=webrtc:5520 Review URL: https://codereview.webrtc.org/1687833006 Cr-Commit-Position: refs/heads/master@{#11588} --- talk/app/webrtc/objc/RTCDataChannel.mm | 1 + webrtc/api/objc/RTCDataChannel.mm | 2 ++ webrtc/base/buffer.h | 7 ++++--- 3 files changed, 7 insertions(+), 3 deletions(-) diff --git a/talk/app/webrtc/objc/RTCDataChannel.mm b/talk/app/webrtc/objc/RTCDataChannel.mm index ef45fbe77e..00d51b10ee 100644 --- a/talk/app/webrtc/objc/RTCDataChannel.mm +++ b/talk/app/webrtc/objc/RTCDataChannel.mm @@ -32,6 +32,7 @@ #import "RTCDataChannel+Internal.h" #include "webrtc/api/datachannelinterface.h" +#include "webrtc/base/scoped_ptr.h" namespace webrtc { diff --git a/webrtc/api/objc/RTCDataChannel.mm b/webrtc/api/objc/RTCDataChannel.mm index f7ca288f15..a1f484a1a2 100644 --- a/webrtc/api/objc/RTCDataChannel.mm +++ b/webrtc/api/objc/RTCDataChannel.mm @@ -13,6 +13,8 @@ #import "webrtc/api/objc/RTCDataChannel+Private.h" #import "webrtc/base/objc/NSString+StdString.h" +#include "webrtc/base/scoped_ptr.h" + namespace webrtc { class DataChannelDelegateAdapter : public DataChannelObserver { diff --git a/webrtc/base/buffer.h b/webrtc/base/buffer.h index ff9bb73d3f..5c9380afee 100644 --- a/webrtc/base/buffer.h +++ b/webrtc/base/buffer.h @@ -14,10 +14,11 @@ #include // std::swap (pre-C++11) #include #include +#include #include // std::swap (C++11 and later) +#include "webrtc/base/constructormagic.h" #include "webrtc/base/deprecation.h" -#include "webrtc/base/scoped_ptr.h" namespace rtc { @@ -164,7 +165,7 @@ class Buffer { assert(IsConsistent()); if (capacity <= capacity_) return; - scoped_ptr new_data(new uint8_t[capacity]); + std::unique_ptr new_data(new uint8_t[capacity]); std::memcpy(new_data.get(), data_.get(), size_); data_ = std::move(new_data); capacity_ = capacity; @@ -222,7 +223,7 @@ class Buffer { size_t size_; size_t capacity_; - scoped_ptr data_; + std::unique_ptr data_; }; } // namespace rtc