From 45a2cd2b32aebeeca88ee12fe4a29ccfebbf8305 Mon Sep 17 00:00:00 2001 From: Amit Hilbuch Date: Tue, 12 Mar 2019 16:58:02 -0700 Subject: [PATCH] Fixing documentation for CopyOnWriteBuffer. The copy constructor mistakenly claims that data is copied. This change corrects the documentation to indicate that the buffer data will be shared between the two objects. Bug: None Change-Id: Ie0d513bfd2f4de660d60c46e87afbf02f0ea3991 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/127343 Reviewed-by: Steve Anton Reviewed-by: Benjamin Wright Commit-Queue: Amit Hilbuch Cr-Commit-Position: refs/heads/master@{#27111} --- rtc_base/copy_on_write_buffer.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rtc_base/copy_on_write_buffer.h b/rtc_base/copy_on_write_buffer.h index d0954adec7..c7df325624 100644 --- a/rtc_base/copy_on_write_buffer.h +++ b/rtc_base/copy_on_write_buffer.h @@ -29,7 +29,7 @@ class CopyOnWriteBuffer { public: // An empty buffer. CopyOnWriteBuffer(); - // Copy size and contents of an existing buffer. + // Share the data with an existing buffer. CopyOnWriteBuffer(const CopyOnWriteBuffer& buf); // Move contents from an existing buffer. CopyOnWriteBuffer(CopyOnWriteBuffer&& buf);