- 10 Mar, 2011 1 commit
-
-
James Berry authored
The previous calculation of macroblock count (w*h)/256 is not correct when the width/height are not multiples of 16. Use the precalculated macroblock count from cpi->common instead. This manifested itself as a divide by zero when the number of pixels was less than 256. num_mbs updated in estimate_max_q, estimate_q, estimate_kf_group_q, and estimate_cq Change-Id: I92ff98587864c801b1ee5485cfead964673a9973
-
- 09 Mar, 2011 8 commits
-
-
Yunqing Wang authored
-
John Koleszar authored
Change-Id: Icf18b4981afb12ef255fca431d4ba45860dd22c9
-
John Koleszar authored
Missing file causes 'make dist' to not include a complete copy of the source. Change-Id: I3f55aeb5a86d0e81234e4e4588cb8086ba4cfc4a
-
Johann authored
-
Yunqing Wang authored
Added SSSE3 function Change-Id: I8c304c92458618d93fda3a2f62bd09ccb63e75ad
-
Yunqing Wang authored
Removed some unused functions Change-Id: Ifdfc27453e53cfc75997b38492901d193a16b245
-
Yunqing Wang authored
-
John Koleszar authored
-
- 08 Mar, 2011 13 commits
-
-
Johann authored
failed to find headers in the source directory output to stdout instead of a hardcoded file MinGW doesn't support _sopen_s _fstat catches non-existant files Change-Id: I24e0aacc6f6f26e6bcfc25f9ee7821aa3c8cc7e7
-
Yunqing Wang authored
Rewrote these functions to process 16 pixels once instead of 8. Change-Id: Ic67e80124467a446a3df4cfecfb76a4248602adb
-
Johann authored
-
Yunqing Wang authored
-
Yunqing Wang authored
Skip filter at zero offset. Change-Id: I95fc7e211869bc0ab5bcfb7ab2e3259d1c0ccf38
-
Yunqing Wang authored
-
Yunqing Wang authored
1. Process 16 pixels at one time instead of 8. 2. Add check for both xoffset =0 and yoffset=0, which happens during motion search. This change gave encoder 1%~3% performance gain. Change-Id: Idaa39506b48f4f8b2fbbeb45aae8226fa32afb3e
-
Johann authored
enable parsing 64bit mach-o files (OS X) also fixes --enable-debug issue! Change-Id: I250ee69745cd2365e3e63264f9365cd58fbb6678
-
Johann authored
enable parsing 64bit elf files Change-Id: I7981f4769cf1b822f288fe2e32166254e4394bab
-
Ralph Giles authored
GCC 4.5 and 4.6 both issue a warning about the multi-line format string introduced in bc9c30a0, which also changed the whitespace in the associated stt file by line-wrapping the long format string. Instead, use multiple string constants, which the compiler will concatenate. This maintains the original formatting, but remains legible within the standard line length. Change-Id: I27c9f92d46be82d408105a3a4091f145f677e00e
-
Paul Wilkins authored
Change-Id: Icc9f12bd1e1bdaf51256dc8a90d08aa9be89ef34
-
Paul Wilkins authored
* changes: Improved key frame detection. Improved KF insertion after fades to still.
-
John Koleszar authored
Disable zbin boost in SPLITMV mode as intended. Was incorrectly looking at vp8_ref_frame_order instead of vp8_mode_order when comparing against SPLITMV. This condition should have always been false, as SPLITMV is not in the range of valid reference frames. Change-Id: I0408cc7595eff68f00efef6d008e79f5b60d14bf
-
- 07 Mar, 2011 4 commits
-
-
John Koleszar authored
-
Ralph Giles authored
Cast size_t to (unsigned long) and print it with the %lu format string, which is more portable than C99's explict %zu for size_t. This truncates on Windows x64 but otherwise works on 32 and 64 bit platforms. In practice the stats file is unlikely to be so large. Change-Id: I0432b3acf85fc6ba4ad50640942e1ca4614b21cb
-
Paul Wilkins authored
In some cases where clips have been encoded with borders (eg. some wide-screen content where there is a border top and bottom and slide shows containing portrait format photographs (border left and right)) key frames were not being correctly detected. The new code looks to measure cases where a portion of the image can be coded equally easily using intra or inter modes and where the resulting error score is also very low. These "neutral" areas are then discounted in the key frame detection code. Change-Id: I00c3e8230772b8213cdc08020e1990cf83b780d8
-
Paul Wilkins authored
This code extends what was previously done for GFs, to pick cases where insertion of a key frame after a fade (or other transition or complex motion) followed by a still section, will be beneficial and will reduce the number of forced key frames. Change-Id: If8bca00457f0d5f83dc3318a587f61c17d90f135
-
- 04 Mar, 2011 11 commits
-
-
John Koleszar authored
Change-Id: I7d1e7db1866d829f6d4c6638d1c20e99959cc9a3
-
John Koleszar authored
Change-Id: I784ea2b9fabbec1e99d02e97209981ff1b18ac82
-
John Koleszar authored
Add mappings for Tom Finegan, Tero Rintaluoma Change-Id: I014ad5bb7c8eb8261808d98ec0d4f77a8e7c3f35
-
Johann authored
-
Johann authored
-
John Koleszar authored
Rename the common control id enum vp8_{dec,com}_control_id, move VP8_DECODER_CTRL_ID_START to common, wrap long lines. Change-Id: I659abc62f10aa389d496f7f43950775db0ef2f9f
-
John Koleszar authored
-
John Koleszar authored
-
John Koleszar authored
-
John Koleszar authored
-
John Koleszar authored
-
- 03 Mar, 2011 3 commits
-
-
Mikhal Shemer authored
Change-Id: I7b2fb326e1aabc08b032177a7b914a5b8bb7376f
-
Mikhal Shemer authored
Change-Id: I9d8a98a2f7ed1e3116d0bae35164618c41998bac
-
John Koleszar authored
When the modified_error_left accumulator exceeds INT_MAX, an incorrect cast to int resulted in a negative value, causing the rate control to allocate no bits to that keyframe group, leading to severe undershoot and subsequent poor quality. This error was exposed by the recent change to the rolling target and actual spend accumulators in commit 305be4e4 which fixed them to actually calculate the average value rather than be re-initialized on every frame to the average per-frame bitrate. When this bug was triggered, the target bitrate could be 0, so the rolling target becomes small, which causes the undershoot. The code prior to 305be4e4 did not exhibit this behavior because the rolling target was always set to a reasonable value and was independent of the actual target bitrate. With this patch, the actual target bitrate is calculated correctly, and the rate control tracks as expected. This cast was likely added to silence a compiler warning on a comparison between a double (modified_error_left) and an int (0). Instead, this patch removes the cast and changes the comparison to be against 0.0, which should prevent the warning from reoccuring. This fixes issue #289. Special thanks to gnafu for his efforts in reporting and debugging this fix. Change-Id: Ie5cc1a7b516c578a76c3a50c892a6f04a11621fe
-