generic-library/vpx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity
16,930 Commits 55 Branches 16 Tags 63 MiB
a14a987c82
Commit Graph

230 Commits

Author SHA1 Message Date
clang-format
4b402746ca apply clang-format 
Change-Id: I75e4a9e0b37bd4586f26c8d6c1fa27f3f6ff1bce
 2017-02-14 12:45:52 -08:00
Johann
8a1fb40273 Remove UNINITIALIZED_IS_SAFE 
Where clang static analysis or gcc -Wmaybe-uninitialized warns of
uninitialized values, assign 0 to ints, MB_MODE_COUNT to
MB_PREDICTION_MODE, and B_MODE_COUNT to B_PREDICTION_MODE.

Assert that the modes have been changed from the invalid value by
the end of the function.

Change-Id: Ib11e1ffb08f0a6fe4b6c6729dc93b83b1c4b6350
 2017-02-14 00:56:08 +00:00
Johann
1364cb58b4 Remove vp8_clear_system_state 
Use vpx_clear_system_state instead.

Change-Id: Ia3e9122f69a2c690ddd7c7bc54f92ccb9ec18b3e
 2016-09-29 13:22:49 -07:00
clang-format
5f6d143b41 apply clang-format 
Change-Id: I501597b7c1e0f0c7ae2aea3ee8073f0a641b3487
 2016-09-15 15:07:53 -07:00
James Zern
fd85664ae6 vp8/{bitstream,rdopt},y4minput: correct break placement 
these should be placed within {}s when present

Change-Id: If00e9766fa8cb039cc070467f353a468f99460fb
 2016-07-19 20:51:25 -07:00
clang-tidy
7f3e07f1c8 vp8: apply clang-tidy google-readability-braces-around-statements 
applied against an x86_64 configure

clang-tidy-3.7.1 \
  -checks='-*,google-readability-braces-around-statements' \
  -header-filter='.*' -fix
+ clang-format afterward

Change-Id: I6694edeaee89b58b8b3082187e6756561136b459
 2016-07-19 12:38:03 -07:00
Jim Bankoski
3e04114f3d prepend ++ instead of post in for loops. 
Applied the following regex  :
search for: (for.*\(.*;.*;) ([a-zA-Z_]*)\+\+\)
replace with: \1 ++\2)

This misses some for loops:
ie : for (mb_col = 0; mb_col < oci->mb_cols; mb_col++, mi++)

Change-Id: Icf5f6fb93cced0992e0bb71d2241780f7fb1f0a8
 2016-07-18 06:54:50 -07:00
clang-format
81a6739533 vp8: apply clang-format 
Change-Id: I7605b6678014a5426ceb45c27b54885e0c4e06ed
 2016-07-15 19:28:44 -07:00
Yaowu Xu
787b38ebb9 Fix VP8 encoder UBSAN/IOC errors 
1. vp8/decoder/dboolhuff.c
2. vp8/decoder/dboolhuff.h
3. vp8/encoder/bitstream.c
4. vp8/encoder/boolhuff.h
5. vp8/encoder/rdopt.c

BUG=https://bugs.chromium.org/p/webm/issues/detail?id=1218

Change-Id: I5d315d63fd7aeaee6f3bd79178e593f3db38a6b1
 2016-06-01 16:00:56 +00:00
Marco
bc6c199785 vp8 denoiser: Some adjustments to usage of skin and motion. 
Switch to use new skin model.

And fix condition for denoising skin block.
Previous condition did not denoise skin blocks if the selected
mode was non-zero motion in current frame. Modify condition to
also force no denoising if that mode was not selected as zero motion
now and for at least "x" past frames in a row (x = 2).

Change-Id: I00753e3fe45b9a308a7ef43c58f11868e3bfc6b0
 2016-04-01 11:36:45 -07:00
Ronald S. Bultje
54d48955f6 vp8: change build_intra_predictors_mby_s to use vpx_dsp. 
Change-Id: I2000820e0c04de2c975d370a0cf7145330289bb2
 2015-09-30 18:45:40 -04:00
Johann
79a00d71bd Rename vp8 quantize.c 
Move it to vp8_quantize.c and make sure to
use the full path for all vp8 includes of quantize.h

Change-Id: I284651ff681707385f4924ea7db1541905c1624a
 2015-08-18 11:44:32 -07:00
Johann
6a82f0d7fb Move sub pixel variance to vpx_dsp 
Change-Id: I66bf6720c396c89aa2d1fd26d5d52bf5d5e3dff1
 2015-07-07 15:51:04 -07:00
Johann
c3bdffb0a5 Move variance functions to vpx_dsp 
subpel functions will be moved in another patch.

Change-Id: Idb2e049bad0b9b32ac42cc7731cd6903de2826ce
 2015-05-26 12:01:52 -07:00
James Zern
fd3658b0e4 replace DECLARE_ALIGNED_ARRAY w/DECLARE_ALIGNED 
this macro was used inconsistently and only differs in behavior from
DECLARE_ALIGNED when an alignment attribute is unavailable. this macro
is used with calls to assembly, while generic c-code doesn't rely on it,
so in a c-only build without an alignment attribute the code will
function as expected.

Change-Id: Ie9d06d4028c0de17c63b3a27e6c1b0491cc4ea79
 2015-05-07 11:55:08 -07:00
Johann
d5d9289800 Move shared SAD code to vpx_dsp 
Create a new component, vpx_dsp, for code that can be shared
between codecs. Move the SAD code into the component.

This reduces the size of vpxenc/dec by 36k on x86_64 builds.

Change-Id: I73f837ddaecac6b350bf757af0cfe19c4ab9327a
 2015-05-06 16:58:20 -07:00
James Zern
f58011ada5 vpx_mem: remove vpx_memset 
vestigial. replace instances with memset() which they already were being
defined to.

Change-Id: Ie030cfaaa3e890dd92cf1a995fcb1927ba175201
 2015-04-28 20:00:59 -07:00
James Zern
f274c2199b vpx_mem: remove vpx_memcpy 
vestigial. replace instances with memcpy() which they already were being
defined to.

Change-Id: Icfd1b0bc5d95b70efab91b9ae777ace1e81d2d7c
 2015-04-28 19:59:41 -07:00
James Zern
0c5a140a02 rdopt: remove dead stores 
Change-Id: Ia8a20c6751cc6d63c60bb00b99c78faca1e61051
 2015-04-03 16:44:14 -07:00
Johann
80b344dec5 Silence -Werror=unused-parameter 
Cast away remaining issues so that new ones don't get lost in the noise.

Change-Id: Iacd6999b0686ce80f9835730d68db6382690fa92
 2014-12-16 12:47:08 -08:00
Marco Paniconi
354a2e612d vp8: Code cleanup for control of denoiser mode. 
Change-Id: Icb9918dd38d15061d62852e6a2d905e8ceb2c1ac
 2014-08-08 16:15:28 -07:00
Marco Paniconi
21042c7154 vp8: Allow for on/off control of UV temporal denoiser. 
Use noise_sensitivity level for enabling UV denoiser.

Change-Id: Ib208786a6fdf654981bcd96a3cf44e8e678025c1
 2014-07-15 09:51:03 -07:00
Marco Paniconi
d08b2ba172 Allow for deblocking temporal-denoised signal. 
Allow for an option to selectively apply the deblocking loop filter to the denoised
raw block, based on the denoised state (no-filter, filter with zero motion, or filter with non-zero motion)
of the current block and its upper and left denoised block.
This helps to reduce some blocking artifacts from the motion-compensated denoising.

Change-Id: I0ac4e70076df69a98c5391979e739a2681e24ae6
 2014-06-13 14:34:05 -07:00
Tim Kopp
0ba6e29515 s/INT_MAX/UINT_MAX/ where appropriate 
Change-Id: I0156d85671305326525c4644510e240021eca461
 2014-06-04 13:34:46 -07:00
Dmitry Kovalev
1f0b2f95af Removing vp8/common/pragmas.h. 
Change-Id: I80630a7350e884ebc4fef73fb5b52ec25f908523
 2014-05-23 13:03:15 -07:00
Jim Bankoski
a16794dd31 Revert "Remove Wextra warnings from vp9_sad.c" 
This reverts commit 7ab9a9587b

Nightly test http://build.webmproject.org/jenkins/view/libvpx-nightly-tests/job/libvpx%20unit%20tests%20(valgrind-2)/arch=x86_64-linux-gcc,filter=-*VP8*:*Large.*/276/console

Failed 

This patch did not address all the assembly issues 
some of the vp8 assembly counts on 5 arguments being passed in to this function:   

one example : vp8_sad8x16_wmt

Please address or split this into vp9 and vp8 patches.

Change-Id: I78afcc171649894f887bb8ee3c66de24aaddc7ca
 2014-05-15 08:31:20 -07:00
Deb Mukherjee
7ab9a9587b Remove Wextra warnings from vp9_sad.c 
As a side-effect, the max_sad check is removed from the
C-implementation of VP8, for consistency with VP9, and to
ensure that the SAD tests common to VP8/VP9 pass.
That will make the VP8 C implementation of sad a little slower
but given that is rarely used in practice, the impact will be
minimal.

Change-Id: I7f43089fdea047fbf1862e40c21e4715c30f07ca
 2014-05-14 03:17:31 -07:00
James Zern
268f32db21 rdopt: quiet -Warray-bounds warnings 
eob is limited by GetCoeffs

Change-Id: Id48a92e600375a1d4fb956757c93c91ebb5df59a
 2014-03-15 10:37:49 -07:00
Yaowu Xu
fbb62c6d2b fix integer overflow in vp8 
Change-Id: I62550a7a883115f3ce878710cf3bb039bea54390
 2013-09-23 17:15:55 -07:00
James Zern
9581eb6e8a use consistent framerate naming 
s/frame_rate/framerate/g

Change-Id: I6fc3e088e419c5f46e3a9390dd8a2cad2677a2fc
 2013-07-16 14:12:47 -07:00
Scott LaVarnway
3971fab3e5 Merge "Removed shadow warnings : mcomp.c rdopt.c" 2013-04-19 07:16:34 -07:00
Scott LaVarnway
699196dc3d Removed mode_chosen_counts 
This eliminates a tsan warning.

Change-Id: I204d109b12adb4e347762e63d12859244f28d1a9
 2013-03-20 14:41:24 -07:00
Scott LaVarnway
5d79720d57 Removed shadow warnings : mcomp.c rdopt.c 
Adding -Wshadow to CFLAGS generated a bunch of warnings.  This patch
removes these warnings.

Change-Id: Ib498de4b8652051d257cf86dcb40d2968a5013ae
 2013-03-14 14:23:13 -07:00
Scott LaVarnway
27825e9e3a Merge "Remove cpi parameter from update_mvcount()" 2013-01-08 10:04:21 -08:00
John Koleszar
16810c10c1 Merge branch 'vp9-preview' of review:webm/libvpx 
Merge the vp9-preview branch into master.

Change-Id: If700b9054676f24bed9deb59050af546c1ca5296
 2012-12-27 09:37:19 -08:00
Scott LaVarnway
74efda4bd6 Moved zbin_mode_boost to macroblock struct 
Fixing unsafe threading in VP8 encoder.

Change-Id: Ibf4c89a2043654834747811bc11eb283de0bb830
 2012-12-10 12:42:24 -08:00
Scott LaVarnway
3a19eebe4d Moved zbin_over_quant to macroblock struct 
Change-Id: I76fe20ade099573997404b8733cf7f79e82fb21e
WIP: Fixing unsafe threading in VP8 encoder.
 2012-12-10 10:51:42 -08:00
Scott LaVarnway
bfca084fcd Moving mbs_tested_so_far, mode_test_hit_counts to macroblock struct 
Change-Id: Ifa78c0a953fab3e5dd7af0446924846c7022cd09
 2012-12-04 16:52:47 -08:00
Scott LaVarnway
358b0efec5 Remove cpi parameter from update_mvcount() 
cpi is no longer used .

Change-Id: Ie86b994059e506cc1944212e12d4f75c041c1d15
 2012-12-04 15:43:25 -08:00
John Koleszar
a9c7597adc support building vp8 and vp9 into a single lib 
Change-Id: Ib8f8a66c9fd31e508cdc9caa662192f38433aa3d
 2012-11-15 10:46:17 -08:00
Scott LaVarnway
fe91e47bc7 Moving rd_thresh_mult, rd_threshes to macroblock struct 
Change-Id: I650a593162280ab40e71e527ec6518303e2d5723
 2012-11-06 16:27:00 -08:00
Scott LaVarnway
01824d1848 Moving MVcount to macroblock struct 
Change-Id: Ie22841d096f3c86694b95bd06fc3a8ce1f032a10
 2012-11-06 08:51:11 -08:00
Scott LaVarnway
9016fa22e0 Removed unnecessary VP8_COMP *cpi parameters 
Code cleanup

Change-Id: I82f9d787a2f511d39895fd8dfd5347a1676d9dbc
 2012-11-02 12:10:07 -07:00
Johann
4b0b43632b Pass B_PREDICTION_MODE to intra4x4_predict 
Use the correct type when calling and inside the function.

Change-Id: Idbaf1bf58b9574f0cba73d78780e9aedd8889dfd
 2012-08-02 15:26:24 -07:00
Johann
a497cb59cd Rename vp8_intra4x4_predict_d 
predict_d has become canonical. Remove previous helper function.

Disable ARM assembly pending update.

Change-Id: Idd84ac8a28f9b0221ea97904a77de1e705d06a7d
 2012-08-01 11:17:57 -07:00
Johann
a82c58c40f Change vp8_intra4x4_predict call sites 
Use the _d variant from the decoder. It moves the pointer calculations
to the caller.

Change-Id: Iae2a793433ef082980a3ffa0a1cabf0264a6a24d
 2012-08-01 10:48:46 -07:00
Johann
1161055129 Be consistent with SAD values 
SAD returns unsigned values. Make all the declarations the same.

Remove bestsad initialization and check. It is always set to the
result of a SAD call so it will never remain UINT_MAX

Use ja instead of jg to test unsigned comparison instead of signed.

Update test.

Change-Id: I46336ab45f4e60fc37caf20bd36bc5782079c7a5
 2012-07-27 16:15:31 -07:00
John Koleszar
0164a1cc5b Fix pedantic compiler warnings 
Allows building the library with the gcc -pedantic option, for improved
portabilty. In particular, this commit removes usage of C99/C++ style
single-line comments and dynamic struct initializers. This is a
continuation of the work done in commit 97b766a46, which removed most
of these warnings for decode only builds.

Change-Id: Id453d9c1d9f44cc0381b10c3869fabb0184d5966
 2012-06-11 15:14:58 -07:00
John Koleszar
52ec78be7d Fix bad merge 
Change-Id: I45849dde0ee9b7e87fa32adb65ade8486bb66556
 2012-06-11 12:24:14 -07:00
John Koleszar
71618e542c Merge "Reset Q for key frame when spatial resizing occurs." 2012-06-11 12:02:56 -07:00