From d7cff287964cd1b37dd108bc0f39251f03779562 Mon Sep 17 00:00:00 2001 From: James Zern Date: Thu, 27 Feb 2014 12:49:02 -0800 Subject: [PATCH] cosmetics: sad_test: add a few comments - break the platform sections a bit more visibly - add #endif comments on long blocks Change-Id: I8612f19755ea37efd6dac2918f51deee4a2c483e --- test/sad_test.cc | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/test/sad_test.cc b/test/sad_test.cc index 4a91b0b60..9c156e020 100644 --- a/test/sad_test.cc +++ b/test/sad_test.cc @@ -296,6 +296,8 @@ TEST_P(SADTest, MaxSAD) { using std::tr1::make_tuple; +//------------------------------------------------------------------------------ +// C functions #if CONFIG_VP8_ENCODER const sad_m_by_n_fn_t sad_16x16_c = vp8_sad16x16_c; const sad_m_by_n_fn_t sad_8x16_c = vp8_sad8x16_c; @@ -364,9 +366,10 @@ INSTANTIATE_TEST_CASE_P(C, SADx4Test, ::testing::Values( make_tuple(8, 4, sad_8x4x4d_c), make_tuple(4, 8, sad_4x8x4d_c), make_tuple(4, 4, sad_4x4x4d_c))); -#endif +#endif // CONFIG_VP9_ENCODER -// ARM tests +//------------------------------------------------------------------------------ +// ARM functions #if HAVE_MEDIA const sad_m_by_n_fn_t sad_16x16_armv6 = vp8_sad16x16_armv6; INSTANTIATE_TEST_CASE_P(MEDIA, SADTest, ::testing::Values( @@ -387,7 +390,8 @@ INSTANTIATE_TEST_CASE_P(NEON, SADTest, ::testing::Values( make_tuple(4, 4, sad_4x4_neon))); #endif -// X86 tests +//------------------------------------------------------------------------------ +// x86 functions #if HAVE_MMX #if CONFIG_VP8_ENCODER const sad_m_by_n_fn_t sad_16x16_mmx = vp8_sad16x16_mmx; @@ -437,9 +441,9 @@ const sad_n_by_n_by_4_fn_t sad_4x4x4d_sse = vp9_sad4x4x4d_sse; INSTANTIATE_TEST_CASE_P(SSE, SADx4Test, ::testing::Values( make_tuple(4, 8, sad_4x8x4d_sse), make_tuple(4, 4, sad_4x4x4d_sse))); -#endif -#endif -#endif +#endif // CONFIG_USE_X86INC +#endif // CONFIG_VP9_ENCODER +#endif // HAVE_SSE #if HAVE_SSE2 #if CONFIG_VP8_ENCODER -- 2.40.0