Merge changes from topic 'clang-tidy'

* changes:
  vp8/{bitstream,rdopt},y4minput: correct break placement
  y4minput.c: correct empty loop formatting
  vp8: simplify a few #if's
  vp8: remove extra semicolons

Change to call vp9_post_proc_frame()

This commit changes the call in vp9 encoder from vp9_deblock() to
vp9_post_proc_frame() to ensure the data structures used in the call
are properly allocated. This fixes an encoder crash when configured
with --enable-internal-stats.

Change-Id: I2393b336c0f566665336df4f1ba91c405eb56764

vp8/{bitstream,rdopt},y4minput: correct break placement

these should be placed within {}s when present

Change-Id: If00e9766fa8cb039cc070467f353a468f99460fb

y4minput.c: correct empty loop formatting

prefer {}s over ';'

Change-Id: I563fc82717e1deb4f42a40e03dca318c6adaa0c1

vp8: simplify a few #if's

bitstream.c: asserts are disabled when CONFIG_DEBUG is unset
vp8_dx_iface.c: split |s into 2 statements across #if bounds

Change-Id: I307d1e969134db5c9c0edd7690589b6b29116cbd

vp8: remove extra semicolons

Change-Id: I84e1a293ee033865f82c244e8aaaadfb2fb27e63

top-level: apply clang-format

Change-Id: Ibd5395bf8956a80f7c0df4d539c7a42c927a1fc7

Merge "vp8: apply clang-tidy google-readability-braces-around-statements"

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

Merge "examples: apply clang-format"

Merge changes Ia6004c08,I1954f9d6

* changes:
  cosmetics: Add a few explanatory comments
  cosmetics: Correct grammar/spelling in comments

cosmetics: Add a few explanatory comments

Change-Id: Ia6004c08e6f5fd269a1bbd4df51ce9b76345150d

Merge "vp9: Allow usage of lookahead for real-time, 1 pass vbr."

Merge "Change 'git cl upload' default to --no-squash"

Merge changes I18982dbf,I15c8976c

* changes:
  build/make/Makefile: add a 'test_*' default target
  build/make/Makefile: remove default suffix rules

Merge "take II: variance_test partial clean-up"

examples: apply clang-format

Change-Id: Icc3bbb07c99a31a70030baec7e51b881902a7b5e

Merge "configure: turn on all unused warnings by default"

build/make/Makefile: add a 'test_*' default target

allows 'make test_libvpx', etc. some reworking of the makefiles would be
needed to avoid hard coding targets here.

Change-Id: I18982dbf691e7d36ab8bcf5934bab9340687b061

build/make/Makefile: remove default suffix rules

Change-Id: I15c8976c6478bf75ec617398f49461b310ab7569

take II: variance_test partial clean-up

remove some (but not all yet!) tuple mis-use, and revamp the code a lot.
Factorize some common chores into MainTestClass.

Change-Id: Id37b7330eebe80d19b9d12a454f24ff9be6b1116

vp9: Allow usage of lookahead for real-time, 1 pass vbr.

Allow usage of lookahead for VBR in real-time mode, for 1 pass vbr.

Current usage is for fast checking of future scene cuts/changes,
and adjusting rate control (gf interval and active_worst/target size).

Added unittests (datarate) for 1 pass vbr mode, with non-zero lag.

Added an experimental option to limit QP based on lookahead.

Overall positive gain in metrics on ytlive set:
avgPNSR/SSIM up on average ~1-3%; several clips up by 5, 7%.

Change-Id: I960d57dfc89de121c4824b9a9bf88d2814e74b56

Change 'git cl upload' default to --no-squash

Chromium changed the upstream default to --squash but this conflicts
with libvpx historical defaults.

Change-Id: I80f2f2b48e2ba08e02184b50e6d5f8f5e76fec24

cosmetics: Correct grammar/spelling in comments

Change-Id: I1954f9d6e33abff9081fe7a5cf59d5497768e0df

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

Merge "Revert "variance_test partial clean-up""

vpx_plane_add_noise_c: normalize int types

quiets signed/unsigned mismatch warning

Change-Id: Iaabd7dfff110ba26056258457541f5635d2e85e6

Revert "variance_test partial clean-up"

This reverts commit f993ed5c86217e31b92ecc7ef58d7d7860b80108.

build warnings under msvc, segfaults with high-bitdepth enabled.

Change-Id: I67502651107830bcadb6ef56d8f2709cccbfdf2b

Merge "sad_test: add some const to methods"

Merge "vp9_error_block_test: simplify fn wrapper generation"

sad_test: add some const to methods

Change-Id: I6f2481509b0aa94338ed6185f80c4a6b65532280

Merge "remove tuple from 'sad_test.cc'"

vp9_error_block_test: simplify fn wrapper generation

Change-Id: I1f1d396b9456e52e863c4c75f23c3d17420668b4

remove tuple from 'sad_test.cc'

+ general clean-up

Change-Id: Ib9dca3d1a3b7f0c1bedef2a26c9ff5ae1c289e8a

vp8: apply clang-format

Change-Id: I7605b6678014a5426ceb45c27b54885e0c4e06ed

add .clang-format, based on Google style

derived from clang-format 3.7.1; same as used in libaom

Change-Id: I8ea915a41d1f2ea3b0d4e4dab9ebc808e9116f11

Merge "addnoise : clear out static size for generated noise"

addnoise : clear out static size for generated noise

Change-Id: I5d4343f2da9cd4b01dd37be7a048d159fec109d1

configure: turn on all unused warnings by default

Change-Id: I7f6cb446cd3ac57ac39835cf065d9501a66acd5b

vp9_postproc.c : unused variable if not vp9_highbitdepth.

Change-Id: Ib89b128f23767934c40b5add3fcf9dbd875e82f9

Merge "postproc : fix function parameters for noise functions."

postproc : fix function parameters for noise functions.

Change-Id: I582b6307f28bfc987dcf8910379a52c6f679173c

Merge "variance_test partial clean-up"

variance_test partial clean-up

remove some (but not all yet!) tuple mis-use, and revamp the code a lot.
Factorize some common chores into MainTestClass.

Change-Id: Ia14f3924140e8545e4f10d0504475681baae8336

Merge "vp9_intrapred_test: follow-up cleanup"

gtest-all.cc: quiet an unused variable warning

under windows / mingw builds

Change-Id: I93f9a5df77cea0c28d4afb272abcde5a9732e355

vp9_intrapred_test: follow-up cleanup

address few comments from ce050afaf3e288895c3bee4160336e2d2133b6ea

Change-Id: I5d8fc9dab35c4ee5ec3671134c4eef4ec241e309

Merge "postproc: noise style fixes."

Merge "Revert "Eliminate isolated and small tail coefficients:""

Merge "clean-up vp9_intrapred_test"

Merge "Fix encoder crashes for odd size input"

postproc: noise style fixes.

Change-Id: Ifdcb36b8e77b65faeeb10644256e175acb32275d

clean-up vp9_intrapred_test

remove tuple and overkill VP9IntraPredBase class.

Change-Id: I85b85bdd33d7fe417895e75f77db219f713dfea3

Revert "Eliminate isolated and small tail coefficients:"

This reverts commit ff19cdafdbb5ee470e4369582b0266f4bc23287d.

Change-Id: I81f68870ca27a1ff683ee22090530b6997815fb2

Merge "Disable trellis optimization when lossless is on"

Merge "postproc - move filling of noise buffer to vpx_dsp."

postproc - move filling of noise buffer to vpx_dsp.

Change-Id: I63ba35dc0ae9286c9812367a531e01d79a4c1635

deblock: missing const on extern const.

Change-Id: I0df08f7c431daf939e266f008bf5158b0c97358b

vp9_postproc.c missing extern.

BUG=webm:1256

Change-Id: I5271e71bc53cce033fb906040643dcdd5ccb2381

Merge changes from topic 'webp-thread-update'

* changes:
  vpx_thread: use CreateThread for windows phone
  vpx_thread: use WaitForSingleObjectEx if available
  vpx_thread: use InitializeCriticalSectionEx if available
  vpx_thread: use native windows cond var if available
  vpx_thread.[hc]: update webp source reference

Fix encoder crashes for odd size input

Change-Id: Id5c30c419282369cc8c3280d9a70b34a859a71d8

Merge "vp9 svc: Reuse scaled_temp in two stage downscaling."

vp9 svc: Reuse scaled_temp in two stage downscaling.

This change eliminates redundant computation in the two stage
downscaling, which saves ~1% encoding time in 3-layer svc encoding.

Change-Id: Ib4b218811b68499a740af1f9b7b5a5445e28d671

Disable trellis optimization when lossless is on

Disable trellis coefficient optimization when the lossless mode
is turned on.

Change-Id: I9001bf626e86dc3c8c32331ede04fd39036e5f7c

deblock filter : moved from vp8 code branch

The deblocking filters used in vp8 have been moved to vpx_dsp for
use by both vp8 and vp9.

Change-Id: I5209d76edafc894b550f751fc76d3aa6799b392d

Merge "remove *debugmodes.c from the default build"

Merge "vp9_rd_pick_intra_mode_sb(): set interp_filter to"

vp9_rd_pick_intra_mode_sb(): set interp_filter to

SWITCHABLE_FILTERS.  This is a partial fix for the build
issues with Change 357240.

Change-Id: I4e507c196175bae729a4f1397878ec8776b0146c

Fix non-highbitdepth coding path for HBD build

Change-Id: I38eb42b8d051924a7cd1ccc3421a4057cf6e170f

Merge "vp9: Adjustment of gfu_boost and af_ratio for 1 pass vbr."

Merge "Enable HBD support in real time encoding path"

vp9: Adjustment of gfu_boost and af_ratio for 1 pass vbr.

Modify the gfu_boost and af_ratio setting based on the
average frame motion level.

Change only affects 1 pass vbr.

Metrics overall positive on ytlive set.
On average up by ~1%, several clips up by 2-4%.

Change-Id: Ic18c49eb2df74cb4986b63cdb11be36d86ab5e8d

Merge "vp9: Adjustment to mv bias for non-rd pickmode."

Enable coeff optimization for intra modes

This further improves the coding performance by
lowres 0.3%
midres 0.5%
hdres  0.6%

Change-Id: I6a03b6da210b9cbc261474bad4a103e0ba021c68

Use precise context to estimate coeff rate cost

Use the precise context to estimate the zero token cost in trellis
optimization process. This improves the speed 0 coding performance
by 0.15% for lowres and 0.1% for midres. It improves the speed 1
coding performance by 0.2% for midres and hdres.

Change-Id: I59c7c08702fc79dc4f8534b64ca594da909e2c91

Enable uniform quantization with trellis optimization in speed 0

This commit allows the inter prediction residual to use uniform
quantization followed by trellis coefficient optimization in
speed 0. It improves the coding performance by

lowres 0.79%
midres 1.07%
hdres  1.44%

Change-Id: I46ef8cfe042a4ccc7a0055515012cd6cbf5c9619

Refactor coeff_cost() function

Move the operations that update the context buffers outside this
function. The coeff_cost() takes all input as const value and returns
the coefficient cost.

This makes preparation for the next coefficient optimization CLs.

Change-Id: I850eec6e5470b91ea84646ff26b9231b09f70a0c

Merge "Support measure distortion in the pixel domain"

vp9: Adjustment to mv bias for non-rd pickmode.

Replace the existing mv bias with a bias only for
NEWMV, and based on the motion vector difference of
its top/left neighbors.

For cbr non-screen-content mode.

Change-Id: I8a8cf56347cfa23e9ffd8ead69eec8746c8f9e09

Merge "Fix Visual Studio build warning"

Fix Visual Studio build warning

Fixed signed/unsigned mismatch warning.

Change-Id: I1634d0634de752f4b8baa8059e8f3e2891fa53b6

Merge "Add experimental spatial de-noise filter on key frames."

Support measure distortion in the pixel domain

Use pixel domain distortion metric in speed 0. This improves the
compression performance by 0.3% for both low and high resolution
test sets.

Change-Id: I5b5b7115960de73f0b5e5d0c69db305e490e6f1d

Enable HBD support in real time encoding path

BUG=webm:1223

Change-Id: If83a613784e3b2a33c9c93f9ad0ba39dd4d23056

Merge "Remove decode asserts from better-hw-compatibility"

Merge "vp9: Choose the scheme for modeling rd for 32x32 based on skin color."

Remove decode asserts from better-hw-compatibility

Safer to have the decoder operate normally and have
better-hw-compatibility only implement encoding changes.
Fixes some test failures.

Change-Id: I0dd70d002e4e893992f0cd59774b9363e6f7fe76

Merge "Modify the name of vp9cx_set_ref example"

vp9: Choose the scheme for modeling rd for 32x32 based on skin color.

For real time CBR mode, use model_rd_for_sb_y for 32x32 if the sb is
a skin sb to avoid visual regression on the slowly moving face.

Refer to the cl: https://chromium-review.googlesource.com/#/c/356020/

Change-Id: I42c36666b2b474ce5ee274239d52ae8ab400fd46

Eliminate isolated and small tail coefficients:

Improve hdres PSNR by 0.696%
Improve midres PSNR by 0.313%
Improve lowres PSNR by 0.142%

Change-Id: Icabde78aa9689f539f6a03ec09f712c20758796c

Modify the name of vp9cx_set_ref example

Modified the name of vp9cx_set_ref example so that the test script
ran correctly.

Change-Id: I0ab2de66220b0a88b7af7ea1633a088ab78dd9ff

Merge "Remove txfrm_block_to_raster_xy() from vp9 encoder"

Merge "Make set_reference control API work in VP9"

Remove txfrm_block_to_raster_xy() from vp9 encoder

The transform block row and column positions are always available
outside the callees. There is no need to re-compute these values
again. This approach has been used by the decoder. This commit
removes txfrm_block_to_raster_xy() function.

Change-Id: I5b90f91a0d8b7c35cfa7d171da9edf8202630108

Merge "improve vpx_filter_block1d* based on replace paddsw+psrlw to pmulhrsw"

Merge "Update vpx subpixel 1d filter ssse3 asm"

vpx_thread: use CreateThread for windows phone

BUG=b/29583578

original webp change:

commit d2afe974f9d751de144ef09d31255aea13b442c0
Author: James Zern <jzern@google.com>
Date:   Mon Nov 23 20:41:26 2015 -0800

    thread: use CreateThread for windows phone

    _beginthreadex is unavailable for winrt/uwp

    Change-Id: Ie7412a568278ac67f0047f1764e2521193d74d4d

100644 blob 93f7622797f05f6acc1126e8296c481d276e4047 src/utils/thread.c
100644 blob 840831185502d42a3246e4b7ff870121c8064791 src/utils/thread.h

Change-Id: Iade8fff6367b45534986c77ebe61abeb45bce0f8

vpx_thread: use WaitForSingleObjectEx if available

BUG=b/29583578

original webp change:

commit 0fd0e12bfe83f16ce4f1c038b251ccbc13c62ac2
Author: James Zern <jzern@google.com>
Date:   Mon Nov 23 20:40:26 2015 -0800

    thread: use WaitForSingleObjectEx if available

    Windows XP and up

    Change-Id: Ie1a46a82722b8624437c8aba0aa4566a4b0b3f57

100644 blob d58f74e5523dbc985fc531cf5f0833f1e9157cf0 src/utils/thread.c
100644 blob 840831185502d42a3246e4b7ff870121c8064791 src/utils/thread.h

Change-Id: If165c38b378c6e0c55e17a1b071efd3ec3e7dcdd

vpx_thread: use InitializeCriticalSectionEx if available

BUG=b/29583578

original webp change:

commit 63fadc9ffacc77d4617526a50c696d21d558a70b
Author: James Zern <jzern@google.com>
Date:   Mon Nov 23 20:38:46 2015 -0800

    thread: use InitializeCriticalSectionEx if available

    Windows Vista / Server 2008 and up

    Change-Id: I32c5b4e5384d614c5a821ef511293ff014c67966

100644 blob f84207d89b3a6bb98bfe8f3fa55cad72dfd061ff src/utils/thread.c
100644 blob 840831185502d42a3246e4b7ff870121c8064791 src/utils/thread.h

Change-Id: I9ce49b3a86857267e504cd8ceab503b7b441d614

vpx_thread: use native windows cond var if available

BUG=b/29583578

original webp change:

commit 110ad5835ecd66995d0e7f66dca1b90dea595f5a
Author: James Zern <jzern@google.com>
Date:   Mon Nov 23 19:49:58 2015 -0800

    thread: use native windows cond var if available

    Vista / Server 2008 and up. no speed difference observed.

    Change-Id: Ice19704777cb679b290dc107a751a0f36dd0c0a9

100644 blob 4fc372b7bc6980a9ed3618c8cce5b67ed7b0f412 src/utils/thread.c
100644 blob 840831185502d42a3246e4b7ff870121c8064791 src/utils/thread.h

Change-Id: Iede7ae8a7184e4b17a4050b33956918fc84e15b5