Use OpenMP when comparing picture assets.

diff --git a/examples/wscript b/examples/wscript
index abb0885c4159141695043146cdbaebc590da5850..01dc9c8ea6dfd3867eebbec1aa53aa5473a409c3 100644 (file)
--- a/examples/wscript
+++ b/examples/wscript
@@ -1,16 +1,16 @@
 def build(bld):
-    obj = bld(features = 'cxx cxxprogram')
+    obj = bld(features='cxx cxxprogram')
     obj.name   = 'make_dcp'
     obj.use    = 'libdcp%s' % bld.env.API_VERSION
-    obj.uselib = 'OPENJPEG CXML'
+    obj.uselib = 'OPENJPEG CXML OPENMP'
     obj.source = 'make_dcp.cc'
     obj.target = 'make_dcp'
     obj.install_path = ''
 
-    obj = bld(features = 'cxx cxxprogram')
+    obj = bld(features='cxx cxxprogram')
     obj.name   = 'read_dcp'
     obj.use    = 'libdcp%s' % bld.env.API_VERSION
-    obj.uselib = 'OPENJPEG CXML MAGICK'
+    obj.uselib = 'OPENJPEG CXML MAGICK OPENMP'
     obj.source = 'read_dcp.cc'
     obj.target = 'read_dcp'
     obj.install_path = ''

diff --git a/src/mono_picture_asset.cc b/src/mono_picture_asset.cc
index f14e50c2191e9f68843ffda2fb9761a18cc1085b..5ab12117d49b5c5ff2087cb2cef35d47a792dc67 100644 (file)
--- a/src/mono_picture_asset.cc
+++ b/src/mono_picture_asset.cc
@@ -105,23 +105,23 @@ MonoPictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt, No
 
        bool result = true;
 
+#pragma omp parallel for
        for (int i = 0; i < _intrinsic_duration; ++i) {
                if (i >= other_picture->intrinsic_duration()) {
-                       return false;
+                       result = false;
                }
 
-               note (DCP_PROGRESS, String::compose ("Comparing video frame %1 of %2", i, _intrinsic_duration));
-               shared_ptr<const MonoPictureFrame> frame_A = get_frame (i);
-               shared_ptr<const MonoPictureFrame> frame_B = other_picture->get_frame (i);
-
-               if (!frame_buffer_equals (
-                           i, opt, note,
-                           frame_A->j2k_data(), frame_A->j2k_size(),
-                           frame_B->j2k_data(), frame_B->j2k_size()
-                           )) {
-                       result = false;
-                       if (!opt.keep_going) {
-                               return result;
+               if (result || opt.keep_going) {
+                       note (DCP_PROGRESS, String::compose ("Comparing video frame %1 of %2", i, _intrinsic_duration));
+                       shared_ptr<const MonoPictureFrame> frame_A = get_frame (i);
+                       shared_ptr<const MonoPictureFrame> frame_B = other_picture->get_frame (i);
+
+                       if (!frame_buffer_equals (
+                                   i, opt, note,
+                                   frame_A->j2k_data(), frame_A->j2k_size(),
+                                   frame_B->j2k_data(), frame_B->j2k_size()
+                                   )) {
+                               result = false;
                        }
                }
        }

diff --git a/test/wscript b/test/wscript
index 84c21a93127bb2bfd411820ee61aad367a0cd6de..7e6e48ed2ccec1835787f09069b47aa44f9ec4cd 100644 (file)
--- a/test/wscript
+++ b/test/wscript
@@ -18,7 +18,7 @@ def configure(conf):
 def build(bld):
     obj = bld(features='cxx cxxprogram')
     obj.name   = 'tests'
-    obj.uselib = 'BOOST_TEST BOOST_FILESYSTEM OPENJPEG CXML XMLSEC1 SNDFILE'
+    obj.uselib = 'BOOST_TEST BOOST_FILESYSTEM OPENJPEG CXML XMLSEC1 SNDFILE OPENMP'
     obj.cppflags = ['-fno-inline', '-fno-default-inline', '-fno-elide-constructors', '-g', '-O0']
     if bld.is_defined('HAVE_GCOV'):
         obj.use = 'libdcp%s_gcov' % bld.env.API_VERSION
@@ -63,7 +63,7 @@ def build(bld):
 
     obj = bld(features='cxx cxxprogram')
     obj.name   = 'subs_in_out'
-    obj.uselib = 'BOOST_TEST BOOST_FILESYSTEM OPENJPEG CXML'
+    obj.uselib = 'BOOST_TEST BOOST_FILESYSTEM OPENJPEG CXML OPENMP'
     obj.cppflags = ['-fno-inline', '-fno-default-inline', '-fno-elide-constructors', '-g', '-O0']
     if bld.is_defined('HAVE_GCOV'):
         obj.use = 'libdcp%s_gcov' % bld.env.API_VERSION
@@ -78,7 +78,7 @@ def build(bld):
 
     obj = bld(features='cxx cxxprogram')
     obj.name   = 'rewrite_subs'
-    obj.uselib = 'BOOST_TEST BOOST_FILESYSTEM OPENJPEG CXML'
+    obj.uselib = 'BOOST_TEST BOOST_FILESYSTEM OPENJPEG CXML OPENMP'
     obj.cppflags = ['-fno-inline', '-fno-default-inline', '-fno-elide-constructors', '-g', '-O0']
     if bld.is_defined('HAVE_GCOV'):
         obj.use = 'libdcp%s_gcov' % bld.env.API_VERSION
@@ -93,7 +93,7 @@ def build(bld):
 
     obj = bld(features='cxx cxxprogram')
     obj.name   = 'bench'
-    obj.uselib = 'BOOST_FILESYSTEM OPENJPEG CXML'
+    obj.uselib = 'BOOST_FILESYSTEM OPENJPEG CXML OPENMP'
     obj.use = 'libdcp%s' % bld.env.API_VERSION
     obj.source = 'bench.cc'
     obj.target = 'bench'

diff --git a/tools/dcpdiff.cc b/tools/dcpdiff.cc
index 473efa45f68ff84efe920b89d5f359aa1a7b4a2f..1085e2893ffcf9bd399bfc3f60523427374b0849 100644 (file)
--- a/tools/dcpdiff.cc
+++ b/tools/dcpdiff.cc
@@ -48,6 +48,7 @@ help (string n)
             << "  -v, --verbose                be verbose\n"
             << "      --cpl-annotation-texts   allow differing CPL annotation texts\n"
             << "      --reel-annotation-texts  allow differing reel annotation texts\n"
+            << "  -a, --annotation-texts       allow different CPL and reel annotation texts\n"
             << "  -m, --mean-pixel             maximum allowed mean pixel error (default 5)\n"
             << "  -s, --std-dev-pixel          maximum allowed standard deviation of pixel error (default 5)\n"
             << "      --key                    hexadecimal key to use to decrypt MXFs\n"
@@ -119,6 +120,7 @@ main (int argc, char* argv[])
                        { "mean-pixel", required_argument, 0, 'm'},
                        { "std-dev-pixel", required_argument, 0, 's'},
                        { "keep-going", no_argument, 0, 'k'},
+                       { "annotation-texts", no_argument, 0, 'a'},
                        /* From here we're using random capital letters for the short option */
                        { "ignore-missing-assets", no_argument, 0, 'A'},
                        { "cpl-annotation-texts", no_argument, 0, 'C'},
@@ -127,7 +129,7 @@ main (int argc, char* argv[])
                        { 0, 0, 0, 0 }
                };
 
-               int c = getopt_long (argc, argv, "Vhvm:s:kACD:E", long_options, &option_index);
+               int c = getopt_long (argc, argv, "Vhvm:s:kaACD:E", long_options, &option_index);
 
                if (c == -1) {
                        break;
@@ -152,6 +154,9 @@ main (int argc, char* argv[])
                case 'k':
                        options.keep_going = true;
                        break;
+               case 'a':
+                       options.cpl_annotation_texts_can_differ = options.reel_annotation_texts_can_differ = true;
+                       break;
                case 'A':
                        ignore_missing_assets = true;
                        break;

diff --git a/tools/wscript b/tools/wscript
index 26474fc8129748fd1ad21f2ceab6e1b445ce68d4..965d1c3001ab9153f0fe8d9e2790290d1b4fbb2c 100644 (file)
--- a/tools/wscript
+++ b/tools/wscript
@@ -1,19 +1,18 @@
 def build(bld):
-    obj = bld(features = 'cxx cxxprogram')
+    obj = bld(features='cxx cxxprogram')
     obj.use = ['libdcp%s' % bld.env.API_VERSION]
-    obj.uselib = 'OPENJPEG CXML'
+    obj.uselib = 'OPENJPEG CXML OPENMP'
     obj.source = 'dcpdiff.cc common.cc'
     obj.target = 'dcpdiff'
 
-    obj = bld(features = 'cxx cxxprogram')
+    obj = bld(features='cxx cxxprogram')
     obj.use = ['libdcp%s' % bld.env.API_VERSION]
-    obj.uselib = 'OPENJPEG CXML'
+    obj.uselib = 'OPENJPEG CXML OPENMP'
     obj.source = 'dcpinfo.cc common.cc'
     obj.target = 'dcpinfo'
 
-    obj = bld(features = 'cxx cxxprogram')
+    obj = bld(features='cxx cxxprogram')
     obj.use = ['libdcp%s' % bld.env.API_VERSION]
-    obj.uselib = 'OPENJPEG CXML'
+    obj.uselib = 'OPENJPEG CXML OPENMP'
     obj.source = 'dcpdumpsub.cc'
     obj.target = 'dcpdumpsub'
-

diff --git a/wscript b/wscript
index 2d58e696c5e4deadafcd927fa18f4980507cd8ea..229efa7bf37609fc7d573ce98252a5c220295698 100644 (file)
--- a/wscript
+++ b/wscript
@@ -16,6 +16,7 @@ def options(opt):
     opt.add_option('--disable-tests', action='store_true', default=False, help='disable building of tests')
     opt.add_option('--disable-gcov', action='store_true', default=False, help='don''t use gcov in tests')
     opt.add_option('--disable-examples', action='store_true', default=False, help='disable building of examples')
+    opt.add_option('--enable-openmp', action='store_true', default=False, help='enable use of OpenMP')
 
 def configure(conf):
     conf.load('compiler_cxx')
@@ -38,6 +39,10 @@ def configure(conf):
     if not conf.env.TARGET_OSX:
         conf.env.append_value('CXXFLAGS', ['-Wno-unused-result', '-Wno-unused-parameter'])
 
+    if conf.options.enable_openmp:
+        conf.env.append_value('CXXFLAGS', '-fopenmp')
+        conf.env.LIB_OPENMP = ['gomp']
+
     conf.check_cfg(package='openssl', args='--cflags --libs', uselib_store='OPENSSL', mandatory=True)
     conf.check_cfg(package='libxml++-2.6', args='--cflags --libs', uselib_store='LIBXML++', mandatory=True)
     conf.check_cfg(package='xmlsec1', args='--cflags --libs', uselib_store='XMLSEC1', mandatory=True)