admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Changed I420BufferPool members to be const.

Browse Source 
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}
This commit is contained in:
perkj 2016-11-04 05:33:44 -07:00 committed by Commit bot
parent 00983572b0
commit 536447c3d1
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
webrtc/common_video/include/i420_buffer_pool.h
View File

@ -29,7 +29,7 @@ namespace webrtc {
class I420BufferPool {
public:
I420BufferPool()
: I420BufferPool(false, std::numeric_limits<size_t>::max()) {}
: I420BufferPool(false) {}
explicit I420BufferPool(bool zero_initialize)
: I420BufferPool(zero_initialize, std::numeric_limits<size_t>::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