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

Unified Diff: webrtc/modules/video_coding/fec_rate_table.h

Issue 2625903004: Rename fec_tables_xor.h -> fec_rate_table.h (Closed)
Patch Set: Created 3 years, 11 months 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/fec_rate_table.h
diff --git a/webrtc/modules/video_coding/fec_tables_xor.h b/webrtc/modules/video_coding/fec_rate_table.h
similarity index 98%
rename from webrtc/modules/video_coding/fec_tables_xor.h
rename to webrtc/modules/video_coding/fec_rate_table.h
index fa5bd7bde47283a7d685deaeb87627431ca04e31..0d1ad2b21af826aa055bb864ce7c2aca8e7181fd 100644
--- a/webrtc/modules/video_coding/fec_tables_xor.h
+++ b/webrtc/modules/video_coding/fec_rate_table.h
@@ -8,8 +8,8 @@
* be found in the AUTHORS file in the root of the source tree.
*/
-#ifndef WEBRTC_MODULES_VIDEO_CODING_FEC_TABLES_XOR_H_
-#define WEBRTC_MODULES_VIDEO_CODING_FEC_TABLES_XOR_H_
+#ifndef WEBRTC_MODULES_VIDEO_CODING_FEC_RATE_TABLE_H_
+#define WEBRTC_MODULES_VIDEO_CODING_FEC_RATE_TABLE_H_
// This is a private header for media_opt_util.cc.
// It should not be included by other files.
@@ -18,10 +18,10 @@ namespace webrtc {
// Table for Protection factor (code rate) of delta frames, for the XOR FEC.
// Input is the packet loss and an effective rate (bits/frame).
-// Output is array kCodeRateXORTable[k], where k = rate_i*129 + loss_j;
+// Output is array kFecRateTable[k], where k = rate_i*129 + loss_j;
// loss_j = 0,1,..128, and rate_i varies over some range.
-static const int kSizeCodeRateXORTable = 6450;
-static const unsigned char kCodeRateXORTable[kSizeCodeRateXORTable] = {
+static const int kFecRateTableSize = 6450;
+static const unsigned char kFecRateTable[kFecRateTableSize] = {
sprang_webrtc 2017/01/12 10:37:11 Was this data previously duplicated, or where did
brandtr 2017/01/12 11:56:19 This is the same data -- I just renamed the array
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11,
@@ -456,4 +456,4 @@ static const unsigned char kCodeRateXORTable[kSizeCodeRateXORTable] = {
} // namespace webrtc
-#endif // WEBRTC_MODULES_VIDEO_CODING_FEC_TABLES_XOR_H_
+#endif // WEBRTC_MODULES_VIDEO_CODING_FEC_RATE_TABLE_H_

Powered by Google App Engine
This is Rietveld 408576698