From 3e18ca4cdb27b088a53732e0b3940b112eadd187 Mon Sep 17 00:00:00 2001 From: "fbarchard@google.com" Date: Wed, 21 Jan 2015 22:41:58 +0000 Subject: [PATCH] make 'full' test do multiples of 3 which includes 0 and 255. BUG=324 TESTED=libyuvTest.TestFullYUV R=brucedawson@google.com Review URL: https://webrtc-codereview.appspot.com/35829004 git-svn-id: http://libyuv.googlecode.com/svn/trunk@1242 16f28f9a-4ce2-e073-06de-1de4eb20be90 --- README.chromium | 2 +- include/libyuv/version.h | 2 +- unit_test/color_test.cc | 11 +++++++---- 3 files changed, 9 insertions(+), 6 deletions(-) diff --git a/README.chromium b/README.chromium index a0ce3c89b..21ed29c89 100644 --- a/README.chromium +++ b/README.chromium @@ -1,6 +1,6 @@ Name: libyuv URL: http://code.google.com/p/libyuv/ -Version: 1242 +Version: 1243 License: BSD License File: LICENSE diff --git a/include/libyuv/version.h b/include/libyuv/version.h index 6122fc3cc..72c6d70bf 100644 --- a/include/libyuv/version.h +++ b/include/libyuv/version.h @@ -11,6 +11,6 @@ #ifndef INCLUDE_LIBYUV_VERSION_H_ // NOLINT #define INCLUDE_LIBYUV_VERSION_H_ -#define LIBYUV_VERSION 1242 +#define LIBYUV_VERSION 1243 #endif // INCLUDE_LIBYUV_VERSION_H_ NOLINT diff --git a/unit_test/color_test.cc b/unit_test/color_test.cc index ee6bfe27c..e46791609 100644 --- a/unit_test/color_test.cc +++ b/unit_test/color_test.cc @@ -269,12 +269,15 @@ TEST_F(libyuvTest, TestGreyYUV) { } } -// TODO(fbarchard): Speed up this test or disable it. +// This full test should be run occassionally to test all values are accurate. +// TODO(fbarchard): Determine error distribution. TEST_F(libyuvTest, TestFullYUV) { + // If using small image, step faster. + int step = benchmark_width_ <= 128 ? 3 : 1; int r0, g0, b0, r1, g1, b1; - for (int y = 0; y < 256; ++y) { - for (int u = 0; u < 256; ++u) { - for (int v = 0; v < 256; ++v) { + for (int y = 0; y < 256; y += step) { + for (int u = 0; u < 256; u += step) { + for (int v = 0; v < 256; v += step) { YUVToRGBReference(y, u, v, &r0, &g0, &b0); YUVToRGB(y, u, v, &r1, &g1, &b1); EXPECT_NEAR(r0, r1, ERROR_R);