From 536447c3d1fd4b519e5ca8f9e38dba11b001a4f2 Mon Sep 17 00:00:00 2001 From: perkj Date: Fri, 4 Nov 2016 05:33:44 -0700 Subject: [PATCH] Changed I420BufferPool members to be const. This is a follow up cl to https://codereview.webrtc.org/2474783005/ to addressed magjeds comments. TBR=tommi@webrtc.org, magjed@webrtc.org BUG=none Review-Url: https://codereview.webrtc.org/2473383002 Cr-Commit-Position: refs/heads/master@{#14931} --- webrtc/common_video/include/i420_buffer_pool.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/webrtc/common_video/include/i420_buffer_pool.h b/webrtc/common_video/include/i420_buffer_pool.h index 53950945c2..f0562634ae 100644 --- a/webrtc/common_video/include/i420_buffer_pool.h +++ b/webrtc/common_video/include/i420_buffer_pool.h @@ -29,7 +29,7 @@ namespace webrtc { class I420BufferPool { public: I420BufferPool() - : I420BufferPool(false, std::numeric_limits::max()) {} + : I420BufferPool(false) {} explicit I420BufferPool(bool zero_initialize) : I420BufferPool(zero_initialize, std::numeric_limits::max()) {} I420BufferPool(bool zero_initialze, size_t max_number_of_buffers); @@ -54,9 +54,9 @@ class I420BufferPool { // FFmpeg according to http://crbug.com/390941, which only requires it for the // initial allocation (as shown by FFmpeg's own buffer allocation code). It // has to do with "Use-of-uninitialized-value" on "Linux_msan_chrome". - bool zero_initialize_; + const bool zero_initialize_; // Max number of buffers this pool can have pending. - size_t max_number_of_buffers_; + const size_t max_number_of_buffers_; }; } // namespace webrtc