From: James Zern Date: Tue, 22 Jul 2014 22:48:25 +0000 (-0700) Subject: frame_size_tests: reduce default 'large' frame size X-Git-Tag: v1.4.0~1167^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2528051f41161fbe092cb773d66fe07aac1c73b7;p=platform%2Fupstream%2Flibvpx.git frame_size_tests: reduce default 'large' frame size VP9FrameSizeTestsLarge exposed an integer overflow in the VP9 encoder, for now reduce the size to allow the tests to clear and prevent further regressions. 4096x4096 -> 4096x2160 this should be restored after the bug is fixed: https://code.google.com/p/webm/issues/detail?id=828 Change-Id: I47fdf0648f1d9a3951f731bbf0b727f85ada4fa1 --- diff --git a/test/frame_size_tests.cc b/test/frame_size_tests.cc index d1ea096..b05d12e 100644 --- a/test/frame_size_tests.cc +++ b/test/frame_size_tests.cc @@ -89,7 +89,8 @@ TEST_F(VP9FrameSizeTestsLarge, ValidSizes) { // one for each lag in frames (for 2 pass), and then one for each possible // reference buffer (8) - we can end up with up to 30 buffers of roughly this // size or almost 1 gig of memory. - video.SetSize(4096, 4096); + // TODO(jzern): restore this to at least 4096x4096 after issue #828 is fixed. + video.SetSize(4096, 2160); video.set_limit(2); expected_res_ = VPX_CODEC_OK; ASSERT_NO_FATAL_FAILURE(RunLoop(&video));