+
+static list<string> notes;
+
+static void
+note_handler (dcp::NoteType n, string s)
+{
+ BOOST_REQUIRE_EQUAL (n, dcp::NoteType::NOTE);
+ notes.push_back (s);
+}
+
+/** Check that xyz_to_rgb clamps XYZ values correctly */
+BOOST_AUTO_TEST_CASE (xyz_rgb_range_test)
+{
+ auto xyz = make_shared<dcp::OpenJPEGImage>(dcp::Size(2, 2));
+
+ xyz->data(0)[0] = -4;
+ xyz->data(0)[1] = 6901;
+ xyz->data(0)[2] = 0;
+ xyz->data(0)[3] = 4095;
+ xyz->data(1)[0] = -4;
+ xyz->data(1)[1] = 6901;
+ xyz->data(1)[2] = 0;
+ xyz->data(1)[3] = 4095;
+ xyz->data(2)[0] = -4;
+ xyz->data(2)[1] = 6901;
+ xyz->data(2)[2] = 0;
+ xyz->data(2)[3] = 4095;
+
+ scoped_array<uint8_t> rgb (new uint8_t[2 * 2 * 6]);
+
+ notes.clear ();
+ dcp::xyz_to_rgb (
+ xyz, dcp::ColourConversion::srgb_to_xyz (), rgb.get(), 2 * 6, boost::optional<dcp::NoteHandler> (boost::bind (¬e_handler, _1, _2))
+ );
+
+ /* The 6 out-of-range samples should have been noted */
+ BOOST_REQUIRE_EQUAL (notes.size(), 6);
+ auto i = notes.begin ();
+ BOOST_REQUIRE_EQUAL (*i++, "XYZ value -4 out of range");
+ BOOST_REQUIRE_EQUAL (*i++, "XYZ value -4 out of range");
+ BOOST_REQUIRE_EQUAL (*i++, "XYZ value -4 out of range");
+ BOOST_REQUIRE_EQUAL (*i++, "XYZ value 6901 out of range");
+ BOOST_REQUIRE_EQUAL (*i++, "XYZ value 6901 out of range");
+ BOOST_REQUIRE_EQUAL (*i++, "XYZ value 6901 out of range");
+
+ /* And those samples should have been clamped, so check that they give the same result
+ as inputs at the extremes (0 and 4095).
+ */
+
+ auto buffer = reinterpret_cast<uint16_t*> (rgb.get ());
+ BOOST_REQUIRE_EQUAL (buffer[0 * 3 + 0], buffer[2 * 3 + 1]);
+ BOOST_REQUIRE_EQUAL (buffer[0 * 3 + 1], buffer[2 * 3 + 1]);
+ BOOST_REQUIRE_EQUAL (buffer[0 * 3 + 2], buffer[2 * 3 + 2]);
+
+ BOOST_REQUIRE_EQUAL (buffer[1 * 3 + 0], buffer[3 * 3 + 0]);
+ BOOST_REQUIRE_EQUAL (buffer[1 * 3 + 1], buffer[3 * 3 + 1]);
+ BOOST_REQUIRE_EQUAL (buffer[1 * 3 + 2], buffer[3 * 3 + 2]);
+}
+
+/** Convert an image from RGB to XYZ and back again */
+BOOST_AUTO_TEST_CASE (rgb_xyz_round_trip_test)
+{
+ srand (0);
+ dcp::Size const size (640, 480);
+
+ scoped_array<uint8_t> rgb (new uint8_t[size.width * size.height * 6]);
+ for (int y = 0; y < size.height; ++y) {
+ uint16_t* p = reinterpret_cast<uint16_t*> (rgb.get() + y * size.width * 6);
+ for (int x = 0; x < size.width; ++x) {
+ /* Write a 12-bit random number for each component */
+ for (int c = 0; c < 3; ++c) {
+ *p = (rand () & 0xfff) << 4;
+ ++p;
+ }
+ }
+ }
+
+ shared_ptr<dcp::OpenJPEGImage> xyz = dcp::rgb_to_xyz (rgb.get(), size, size.width * 6, dcp::ColourConversion::srgb_to_xyz ());
+ scoped_array<uint8_t> back (new uint8_t[size.width * size.height * 6]);
+ dcp::xyz_to_rgb (xyz, dcp::ColourConversion::srgb_to_xyz (), back.get(), size.width * 6);
+
+#if 0
+ uint16_t* p = reinterpret_cast<uint16_t*> (rgb.get ());
+ uint16_t* q = reinterpret_cast<uint16_t*> (back.get ());
+ for (int i = 0; i < (size.width * size.height); ++i) {
+ /* XXX: doesn't quite work */
+ // BOOST_REQUIRE_EQUAL (*p++, *q++);
+ }
+#endif
+}