Cleanup

- Removing temporary code segments that were used to output new target values in the CPU tests.

Signed-off-by: cuneyt.ozdas <cuneyt.ozdas@autodesk.com>

Author: cozdas

tests/cpu/ops/fixedfunction/FixedFunctionOpCPU_tests.cpp

@@ -523,17 +523,6 @@ OCIO_ADD_TEST(FixedFunctionOpCPU, aces_output_transform_20)
                        1e-5f,
                        __LINE__);
 
-#if DUMP_RESULT
-    std::cout << "aces_output_transform_20 results: \n" << std::setprecision(9) << std::fixed;
-    for (unsigned i = 0; i < num_samples; ++i) 
-    {
-        std::cout   << input2_32f[i * 4 + 0] << "f, "
-                    << input2_32f[i * 4 + 1] << "f, " 
-                    << input2_32f[i * 4 + 2] << "f, "
-                    << input2_32f[i * 4 + 3] << "f,\n";
-    }
-#endif
-
     OCIO::ConstFixedFunctionOpDataRcPtr funcData2
         = std::make_shared<OCIO::FixedFunctionOpData>(OCIO::FixedFunctionOpData::ACES_OUTPUT_TRANSFORM_20_INV,
                                                       params);
@@ -835,17 +824,6 @@ OCIO_ADD_TEST(FixedFunctionOpCPU, aces_tonescale_compress_20)
                        1e-5f,
                        __LINE__);
 
-#if DUMP_RESULTS
-    std::cout << "aces_tonescale_compress_20 results: \n" << std::setprecision(9) << std::fixed;
-    for (unsigned i = 0; i < num_samples; ++i)
-    {
-        std::cout   << input2_32f[i * 4 + 0] << "f, "
-                    << input2_32f[i * 4 + 1] << "f, "
-                    << input2_32f[i * 4 + 2] << "f, "
-                    << input2_32f[i * 4 + 3] << "f,\n";
-    }
-#endif
-
     OCIO::ConstFixedFunctionOpDataRcPtr funcData2
         = std::make_shared<OCIO::FixedFunctionOpData>(OCIO::FixedFunctionOpData::ACES_TONESCALE_COMPRESS_20_INV,
                                                       params);
@@ -943,16 +921,6 @@ OCIO_ADD_TEST(FixedFunctionOpCPU, aces_gamut_map_20)
                        funcData,
                        1e-5f,
                        __LINE__);
-#if DUMP_RESULTS
-    std::cout << "Results: \n" << std::setprecision(9) << std::fixed;
-    for (unsigned i = 0; i < num_samples; ++i)
-    {
-        std::cout   << input2_32f[i * 4 + 0] << "f, "
-                    << input2_32f[i * 4 + 1] << "f, "
-                    << input2_32f[i * 4 + 2] << "f, "
-                    << input2_32f[i * 4 + 3] << "f,\n";
-    }
-#endif
 
     OCIO::ConstFixedFunctionOpDataRcPtr funcData2
         = std::make_shared<OCIO::FixedFunctionOpData>(OCIO::FixedFunctionOpData::ACES_GAMUT_COMPRESS_20_INV,

tests/cpu/transforms/BuiltinTransform_tests.cpp

@@ -17,8 +17,6 @@
 #include "UnitTestUtils.h"
 
 namespace OCIO = OCIO_NAMESPACE;
-#define DUMP_RESULTS 0
-
 
 OCIO_ADD_TEST(BuiltinTransform, creation)
 {
@@ -705,10 +703,6 @@ OCIO_ADD_TEST(Builtins, validate)
 {
     OCIO::ConstBuiltinTransformRegistryRcPtr reg = OCIO::BuiltinTransformRegistry::Get();
 
-#if DUMP_RESULTS
-    std::ostringstream ss;
-#endif
-
     for (size_t index = 0; index < reg->getNumBuiltins(); ++index)
     {
         const char * name = reg->getBuiltinStyle(index);
@@ -738,33 +732,9 @@ OCIO_ADD_TEST(Builtins, validate)
         {
             Values results;
             ValidateBuiltinTransform(name, std::get<1>(values), std::get<2>(values), std::get<0>(values), __LINE__, results);
-
-#if DUMP_RESULTS
-            ss << std::defaultfloat;
-            ss << "    { \"" << name << "\",\n";
-            ss << "        { " << std::get<0>(values) << "f,\n        { ";
-            int comma = 0;
-            for (auto val : std::get<1>(values)) 
-            {
-                ss << (comma++ ? ", " : "") << val << "f";
-                comma = true;
-            }
-            ss << " },\n        { ";
-            ss << std::setprecision(8) << std::fixed;
-            comma = 0;
-            for (auto val : results) 
-            {
-                ss << (comma++ ? ", " : "") << val << "f";
-            }
-            ss << " } } },\n ";
-#endif
         }
     }
 
-#if DUMP_RESULTS
-    std::cout << ss.str() << std::endl;
-#endif
-
     // The above checks if a test values is missing, but not if there are test values
     // that don't have an associated built-in.
     OCIO_CHECK_EQUAL(UnitTestValues.size(), reg->getNumBuiltins());