summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2012-10-18Merge "Fix encode breakout skip handling." into experimentalRonald S. Bultje
2012-10-18Merge "Detect skippable blocks in splitmv RD loop case." into experimentalRonald S. Bultje
2012-10-18Merge "Use correct context for optimize_b() in vp8_optimize_mby_8x8()." into ↵Ronald S. Bultje
experimental
2012-10-18A fix for crashes when #macroblock rows is odd.Deb Mukherjee
Change-Id: If9fc2df4051ccfb8d804dae9d1be08d12d0f81ce
2012-10-17lint-hunks: exit status for only affected linesJohn Koleszar
Prior to this patch, if there were any lint errors, this script would exit with an error, even if those errors were not in the hunks being tested by this script. This change makes it so that if any lint lines are printed, an error is returned. Change-Id: I69c8bef4367ccf25d287508f29e587b1f4426143
2012-10-17Fix encode breakout skip handling.Ronald S. Bultje
Change-Id: Ib42a5dbded27abf2a01fd656a47a1e96b1fe2912
2012-10-17Merge "lint-hunks: support operating on arbirary revs" into experimentalJohn Koleszar
2012-10-17Detect skippable blocks in splitmv RD loop case.Ronald S. Bultje
Change-Id: Id8a7265dc721e5cdcaa144e0041beeb32a98f51c
2012-10-17Merge "cosmetic: fix '= & ' construct" into experimentalpascal massimino
2012-10-17Merge "rtcd: merge rtcd.c from master" into experimentalJohn Koleszar
2012-10-17Merge "consolidate update_mb_segmentation_map data" into experimentalJohn Koleszar
2012-10-17Merge changes Ic711a378,I43a46c36,If19ee74a into experimentalJohn Koleszar
* changes: Move remaining per-frame data into partition 0 Interleave modes/residual per macroblock Force interleaved decoding
2012-10-17cosmetic: fix '= & ' constructJohn Koleszar
remove useless space after address-of operator. Change-Id: I1fb9e82e8d6cf87558fbd454fb5c0f87599ca2ab
2012-10-17Move remaining per-frame data into partition 0John Koleszar
This commit moves a bit of data that ended up packed with the modes/mv/residual partition during the change to interleaved encoding into partition 0 where it belongs. Change-Id: Ic711a378c58d9d6a17254384f492c213a15bad92
2012-10-17lint-hunks: support operating on arbirary revsJohn Koleszar
Rather than diffing only the index, support checking arbitrary commits. Change-Id: Ia135a487990d8293d1e0799dc062b9f49e020b25
2012-10-17Use correct context for optimize_b() in vp8_optimize_mby_8x8().Ronald S. Bultje
Change-Id: Ic975ab85a9924adc7b3d421f64155cc79f40ffd0
2012-10-17Interleave modes/residual per macroblockJohn Koleszar
Packs the bitstream with each mb's residual following its mode/mv information. TODO: There are still a few fields that should be packed into partition 0 but are included in partition 1, due to them being serialized from write_kfmodes/pack_inter_mode_mvs, which execute after the first partition is finalized. These need to be separated out into a separate function, similar to mb_mode_mv_init() in decodemv.c. Change-Id: I43a46c363601ab36954d07ebe498760e1e2e3af4
2012-10-17Use 8x8 block bases for non-4x4 splitmv RD.Ronald S. Bultje
Change-Id: I76095d5a02edcc5d404ea830157780b24fd509d7
2012-10-17Merge "Removed the loopfilter rtcd invoke macro code" into experimentalScott LaVarnway
2012-10-17removed obselete build dependencyJim Bankoski
this commit fixes the build on windows with visual studio 2008. Change-Id: I0baa4044e9e54237da29f2e17332ea6f766dbbec
2012-10-16Removed the loopfilter rtcd invoke macro codeScott LaVarnway
Change-Id: I446b2ffcbe732ffb112dbd97a4799272d4c01a84
2012-10-16Force interleaved decodingJohn Koleszar
Rather than decoding all modes/mvs separately, decode them per MB. This forces the mode which was already used form the CONFIG_NEWBESTREFMV and CONFIG_SUPERBLOCKS experiments, and is a precursor to changing to interleaved encoding. Change-Id: If19ee74ac8a987846d1cd0cf2b2e02a82f1a43ad
2012-10-16Merge "removed the recon rtcd invoke macro code (unrevert)" into experimentalJim Bankoski
2012-10-16Fix a mismatch if a keyframe contains skip-blocks.Ronald S. Bultje
Change-Id: I0681d3183f51627be8c2bb76f343b7270f9116d8
2012-10-16removed the recon rtcd invoke macro code (unrevert)Jim Bankoski
This reinstates reverted commit 2113a831575d81faeadd9966e256d58b6b2b1633 Change-Id: I9a9af13497d1e58d4f467e3e083fddf06b1b786c
2012-10-15Merge "Remove test from B_MODE_INFO." into experimentalRonald S. Bultje
2012-10-15Merge "Remove mode_rdopt from MB_MODE_INFO." into experimentalRonald S. Bultje
2012-10-15Merge "Properly use chroma block object for chroma coefficient costing." ↵Ronald S. Bultje
into experimental
2012-10-15Merge "Changed to use real pixels only for evaluating MVs" into experimentalYaowu Xu
2012-10-15Changed to use real pixels only for evaluating MVsYaowu Xu
The commit changed to avoid using pixels from extended border in in evaluating and select best reference motion vector. Change-Id: I39b758889373e42ed2889d59744388e5b9c1a20a
2012-10-15Remove test from B_MODE_INFO.Ronald S. Bultje
It is essentially a duplicate of mode for RD-only purposes. Removing it saves us 4 bytes per B_MODE_INFO, or ~0.5MB for a 1080p video encode. Change-Id: I0a54db5f51658b3946d7efb1ca6e8cfbda0cdf88
2012-10-15Remove mode_rdopt from MB_MODE_INFO.Ronald S. Bultje
The variable is essentially a duplicate of mode for RD-only purposes. Removing it gives identical results, and saves 4 bytes per macroblock (i.e. 32.5kB for a 1080p HD video encode). Change-Id: I22d5058fdb80ab0b69862caee825e9d86bb148b3
2012-10-15Properly use chroma block object for chroma coefficient costing.Ronald S. Bultje
Change-Id: Ieb38c7aae91dbaca4a8add204fa84e1cfc459933
2012-10-15Add a new token stuffing function vp8_stuff_mb().Ronald S. Bultje
This way a caller doesn't need to implement the logic for which (and how many) tokens to write out to stuff one macroblock worth of EOBs. Make the actual function implementations static, since they are now only used in tokenize.c; also do some minor stylistic changes so it follows the style guide a little more closely; use PLANE_TYPE where appropriate, remove old (stale) frame_type function arguments; hardcode plane type where only a single one is possible (2nd order DC or U/V EOB stuffing); support stuffing 8x8/4x4 transform EOBs with no 2nd order DC. Change-Id: Ia448e251d19a4e3182eddeb9edd034bd7dc16fa3
2012-10-15Add and consistently use PLANE_TYPE.Ronald S. Bultje
Change the macros PLANE_TYPE_{Y_NO_DC,Y2,UV,Y_WITH_DC} to a typed enum, and use this typed enum consistently across all places where relevant. In places where the type is implied (e.g. in functions that only handle second order planes or chroma planes), remove it as a function argument and instead hardcode the proper enum in the code directly. Change-Id: I93652b4a36aa43163d49c732b0bf5c4442738c47
2012-10-15Merge duplicate loops in tokenization code.Ronald S. Bultje
Also merge the three occurrences of 4x4 chroma block writing into a single function, and call that function instead of duplicating the 4x4 chroma tokenization code in 3 places. Change-Id: I7913538d1029f709b0e3ae49fff1148d3be9eeb9
2012-10-14Minor refactoring in encodeintra.c.Ronald S. Bultje
Merge code blocks for different transform sizes; use MACROBLOCKD as a temp variable where that leads to smaller overall source code; remove duplicate code under #if CONFIG_HYBRIDTRANSFORM/#else blocks. Some style changes to make it follow the style guide a little better. Change-Id: I1870a06dae298243db46e14c6729c96c66196525
2012-10-14Make optimize_mb_16x16() static.Ronald S. Bultje
It is not used outside this file. Change-Id: Id0947180edab4c5f1f50589863350b21e97c25f1
2012-10-14Remove assignment in vp8_optimize_mby_16x16().Ronald S. Bultje
The result of the assignment is unused. Change-Id: Id94b790baa6451f24e9aeda6f036390388d93921
2012-10-14Remove duplicate code in optimize_mb_NxN vs. vp8_optimize_mby/uv_NxN().Ronald S. Bultje
Instead, just call vp8_optimize_mby/uv_NxN() inside optimize_mb_NxN(). Change-Id: Ief941064d86db4277ba02e990a26819144b55f7c
2012-10-14Remove duplicate or unused code in encoder/encodemb.c.Ronald S. Bultje
Also make some minor stylistic changes to bring the code closer to the style guide. Remove distinction between inter and intra transform functions, since both do exactly the same thing except for the check against SPLITMV for the second-order transform. Remove some commented out debug code. Remove 8x8/16x16 transform code in encode_inter16x16y(), since the first-pass only uses 4x4 anyway. Change-Id: Ife54816ff759825b9141f95dc2ba43c253c14dba
2012-10-14Remove duplicate or unused code in encoder/quantize.c.Ronald S. Bultje
Also make some minor stylistic changes to bring the code closer to the style guide. Remove checks against i8x8/bpred in the mb-codepath, since these do individual block reconstruction and thus don't go through this codepath. Change-Id: I4dfcf8f78746f4647a206475acf731837aa4fd47
2012-10-14Remove duplicate or unused code in common/invtrans.c.Ronald S. Bultje
Also make some minor stylistic changes to bring the code closer to the style guide. Change-Id: Idafa280724f49868ed6b7d65b7fd9227d781994b
2012-10-14Add a _4x4 sufix to all 4x4-transform-related functions.Ronald S. Bultje
This includes trellis optimization, forward/inverse transform, quantization, tokenization and stuffing functions. Change-Id: Ibd34132e1bf0cd667671a57b3f25b3d361b9bf8a
2012-10-14Replace vp8_inverse_htransform_b() by direct call to vp8_ihtllm_c().Ronald S. Bultje
Change-Id: I55ee7566341b04d7e7340fdab95189f82231a792
2012-10-13Revert "removed the recon. rtcd invoke macro code"Jim Bankoski
This reverts commit 2113a831575d81faeadd9966e256d58b6b2b1633
2012-10-13removed the recon. rtcd invoke macro codeJim Bankoski
Code clean up - removed rtcd Change-Id: Id963ecf53c370b1d99484ef18d6befeed7e0c748
2012-10-12Merge "Remove reverting of tx-select if only a single txfm-size is used." ↵Adrian Grange
into experimental
2012-10-12removed the dependency cross key frame boundaryYaowu Xu
This allowed decoding from any key frame after skipping the frames prior. Change-Id: I096fccad5346d75ae50a017c94eb8d772a7e8b00
2012-10-12Merge "added the ablility to skip first n frames for vpxdec" into experimentalYaowu Xu