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

Unified Diff: webrtc/modules/video_coding/main/source/timestamp_map.cc

Issue 1417283007: modules/video_coding refactorings (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Fix the other copy of the mock include header Created 5 years, 1 month 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
Index: webrtc/modules/video_coding/main/source/timestamp_map.cc
diff --git a/webrtc/modules/video_coding/main/source/timestamp_map.cc b/webrtc/modules/video_coding/main/source/timestamp_map.cc
deleted file mode 100644
index d11f9491ea98ee3feff9aeedcb7154961fbe712d..0000000000000000000000000000000000000000
--- a/webrtc/modules/video_coding/main/source/timestamp_map.cc
+++ /dev/null
@@ -1,65 +0,0 @@
-/*
- * Copyright (c) 2011 The WebRTC project authors. All Rights Reserved.
- *
- * Use of this source code is governed by a BSD-style license
- * that can be found in the LICENSE file in the root of the source
- * tree. An additional intellectual property rights grant can be found
- * in the file PATENTS. All contributing project authors may
- * be found in the AUTHORS file in the root of the source tree.
- */
-
-#include <assert.h>
-#include <stdlib.h>
-
-#include "webrtc/modules/include/module_common_types.h"
-#include "webrtc/modules/video_coding/main/source/timestamp_map.h"
-
-namespace webrtc {
-
-VCMTimestampMap::VCMTimestampMap(size_t capacity)
- : ring_buffer_(new TimestampDataTuple[capacity]),
- capacity_(capacity),
- next_add_idx_(0),
- next_pop_idx_(0) {
-}
-
-VCMTimestampMap::~VCMTimestampMap() {
-}
-
-void VCMTimestampMap::Add(uint32_t timestamp, VCMFrameInformation* data) {
- ring_buffer_[next_add_idx_].timestamp = timestamp;
- ring_buffer_[next_add_idx_].data = data;
- next_add_idx_ = (next_add_idx_ + 1) % capacity_;
-
- if (next_add_idx_ == next_pop_idx_) {
- // Circular list full; forget oldest entry.
- next_pop_idx_ = (next_pop_idx_ + 1) % capacity_;
- }
-}
-
-VCMFrameInformation* VCMTimestampMap::Pop(uint32_t timestamp) {
- while (!IsEmpty()) {
- if (ring_buffer_[next_pop_idx_].timestamp == timestamp) {
- // Found start time for this timestamp.
- VCMFrameInformation* data = ring_buffer_[next_pop_idx_].data;
- ring_buffer_[next_pop_idx_].data = nullptr;
- next_pop_idx_ = (next_pop_idx_ + 1) % capacity_;
- return data;
- } else if (IsNewerTimestamp(ring_buffer_[next_pop_idx_].timestamp,
- timestamp)) {
- // The timestamp we are looking for is not in the list.
- return nullptr;
- }
-
- // Not in this position, check next (and forget this position).
- next_pop_idx_ = (next_pop_idx_ + 1) % capacity_;
- }
-
- // Could not find matching timestamp in list.
- return nullptr;
-}
-
-bool VCMTimestampMap::IsEmpty() const {
- return (next_add_idx_ == next_pop_idx_);
-}
-}
« no previous file with comments | « webrtc/modules/video_coding/main/source/timestamp_map.h ('k') | webrtc/modules/video_coding/main/source/timing.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698