Assorted test fixes.
authorCarl Hetherington <cth@carlh.net>
Sat, 3 May 2014 19:18:07 +0000 (20:18 +0100)
committerCarl Hetherington <cth@carlh.net>
Sat, 3 May 2014 19:18:07 +0000 (20:18 +0100)
run/tests
src/dcp.cc
src/dcp.h
tools/common.cc [new file with mode: 0644]
tools/common.h [new file with mode: 0644]
tools/dcpdiff.cc
tools/dcpinfo.cc
tools/wscript

index df4ed594a146e0f2f1c3798429cdc8af54843585..5113b3afab3cfadafe969698823b2981bfc2d96d 100755 (executable)
--- a/run/tests
+++ b/run/tests
@@ -55,7 +55,7 @@ fi
 rm -f $work/info.log
 for d in `find $private/metadata -mindepth 1 -maxdepth 1 -type d | sort`; do
     if [ `basename $d` != ".git" ]; then
-        $dcpinfo -k -s $d 2> /dev/null >> $work/info.log
+        $dcpinfo --ignore-missing-assets -k -s $d 2> /dev/null >> $work/info.log
         if [ "$?" != "0" ]; then
             echo "FAIL: dcpinfo failed for $d"
             exit 1
@@ -79,7 +79,7 @@ cp -r $private/* $work/private
 for d in `find $work/private/metadata -mindepth 1 -maxdepth 1 -type d | sort`; do
     if [ `basename $d` != ".git" ]; then
         $work/rewrite_subs $d
-        $dcpinfo -k -s $d 2>&1 >> $work/info2.log
+        $dcpinfo --ignore-missing-assets -k -s $d >> $work/info2.log
     fi
 done
 
index dc0ffeaa3e3c146e856e0d7b772771869e682e74..044e1f537840638d85e4040b7db8a57373fecca7 100644 (file)
@@ -69,7 +69,7 @@ DCP::DCP (boost::filesystem::path directory)
 }
 
 template<class T> void
-survivable_error (bool keep_going, list<shared_ptr<DCPReadError> >* errors, T const & e)
+survivable_error (bool keep_going, dcp::DCP::ReadErrors* errors, T const & e)
 {
        if (keep_going) {
                if (errors) {
@@ -81,7 +81,7 @@ survivable_error (bool keep_going, list<shared_ptr<DCPReadError> >* errors, T co
 }
 
 void
-DCP::read (bool keep_going, list<shared_ptr<DCPReadError> >* errors)
+DCP::read (bool keep_going, ReadErrors* errors)
 {
        /* Read the ASSETMAP */
        
index 66486ca047b72eabc31ee691247e79d29b22239c..1fa7ae0c65e71740fb1cb82ccbe0700edc9fe1b8 100644 (file)
--- a/src/dcp.h
+++ b/src/dcp.h
@@ -69,11 +69,13 @@ public:
         */
        DCP (boost::filesystem::path directory);
 
+       typedef std::list<boost::shared_ptr<DCPReadError> > ReadErrors;
+       
        /** Read the DCP's structure into this object.
         *  @param keep_going true to try to keep going in the face of (some) errors.
         *  @param errors List of errors that will be added to if keep_going is true.
         */
-       void read (bool keep_going = false, std::list<boost::shared_ptr<DCPReadError> >* errors = 0);
+       void read (bool keep_going = false, ReadErrors* errors = 0);
 
        /** Compare this DCP with another, according to various options.
         *  @param other DCP to compare this one to.
diff --git a/tools/common.cc b/tools/common.cc
new file mode 100644 (file)
index 0000000..1c0c1df
--- /dev/null
@@ -0,0 +1,42 @@
+/*
+    Copyright (C) 2014 Carl Hetherington <cth@carlh.net>
+
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with this program; if not, write to the Free Software
+    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+
+*/
+
+#include "common.h"
+#include "dcp.h"
+
+using std::list;
+using boost::shared_ptr;
+using boost::dynamic_pointer_cast;
+
+void
+dcp::filter_errors (dcp::DCP::ReadErrors& errors, bool ignore_missing_assets)
+{
+       for (DCP::ReadErrors::iterator i = errors.begin(); i != errors.end(); ) {
+
+               DCP::ReadErrors::iterator tmp = i;
+               ++tmp;
+               
+               if (ignore_missing_assets && dynamic_pointer_cast<MissingAssetError> (*i)) {
+                       errors.erase (i);
+               }
+
+               i = tmp;
+       }
+}
+
diff --git a/tools/common.h b/tools/common.h
new file mode 100644 (file)
index 0000000..a9c657c
--- /dev/null
@@ -0,0 +1,26 @@
+/*
+    Copyright (C) 2014 Carl Hetherington <cth@carlh.net>
+
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with this program; if not, write to the Free Software
+    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+
+*/
+
+#include "exceptions.h"
+
+namespace dcp {
+
+extern void filter_errors (std::list<boost::shared_ptr<DCPReadError> >& errors, bool ignore_missing_assets);
+
+}
index 666fc1b2c4a15448a8e5a3cc8b6d7b0899e332ef..75664810ee6059b88ac24a9bb8976951a596faee 100644 (file)
@@ -1,8 +1,28 @@
+/*
+    Copyright (C) 2012-2014 Carl Hetherington <cth@carlh.net>
+
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with this program; if not, write to the Free Software
+    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+
+*/
+
 #include <iostream>
 #include <boost/filesystem.hpp>
 #include <getopt.h>
 #include "dcp.h"
 #include "exceptions.h"
+#include "common.h"
 
 using namespace std;
 using namespace boost;
@@ -14,13 +34,14 @@ static void
 help (string n)
 {
        cerr << "Syntax: " << n << " [OPTION] <DCP> <DCP>\n"
-            << "  -V, --version        show libdcp version\n"
-            << "  -h, --help           show this help\n"
-            << "  -v, --verbose        be verbose\n"
-            << "  -n, --names          allow differing MXF names\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"
-            << "  -k, --keep-going     carry on in the event of errors, if possible\n"
+            << "  -V, --version                show libdcp version\n"
+            << "  -h, --help                   show this help\n"
+            << "  -v, --verbose                be verbose\n"
+            << "  -n, --names                  allow differing MXF names\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"
+            << "  -k, --keep-going             carry on in the event of errors, if possible\n"
+            << "      --ignore-missing-assets  ignore missing asset files\n"
             << "\n"
             << "The <DCP>s are the DCP directories to compare.\n"
             << "Comparison is of metadata and content, ignoring timestamps\n"
@@ -35,6 +56,26 @@ note (NoteType t, string n)
        }
 }
 
+DCP *
+load_dcp (boost::filesystem::path path, bool keep_going, bool ignore_missing_assets)
+{
+       DCP* dcp = 0;
+       try {
+               dcp = new DCP (path);
+               DCP::ReadErrors errors;
+               dcp->read (keep_going, &errors);
+               filter_errors (errors, ignore_missing_assets);
+               for (DCP::ReadErrors::const_iterator i = errors.begin(); i != errors.end(); ++i) {
+                       cerr << (*i)->what() << "\n";
+               }
+       } catch (FileError& e) {
+               cerr << "Could not read DCP " << path.string() << "; " << e.what() << " " << e.filename() << "\n";
+               exit (EXIT_FAILURE);
+       }
+
+       return dcp;
+}
+
 int
 main (int argc, char* argv[])
 {
@@ -42,6 +83,7 @@ main (int argc, char* argv[])
        options.max_mean_pixel_error = 5;
        options.max_std_dev_pixel_error = 5;
        bool keep_going = false;
+       bool ignore_missing_assets = false;
        
        int option_index = 0;
        while (1) {
@@ -53,10 +95,11 @@ main (int argc, char* argv[])
                        { "mean-pixel", required_argument, 0, 'm'},
                        { "std-dev-pixel", required_argument, 0, 's'},
                        { "keep-going", no_argument, 0, 'k'},
+                       { "ignore-missing-assets", no_argument, 0, 'A'},
                        { 0, 0, 0, 0 }
                };
 
-               int c = getopt_long (argc, argv, "Vhvnm:s:k", long_options, &option_index);
+               int c = getopt_long (argc, argv, "Vhvnm:s:kA", long_options, &option_index);
 
                if (c == -1) {
                        break;
@@ -84,6 +127,9 @@ main (int argc, char* argv[])
                case 'k':
                        keep_going = true;
                        break;
+               case 'A':
+                       ignore_missing_assets = true;
+                       break;
                }
        }
 
@@ -102,31 +148,8 @@ main (int argc, char* argv[])
                exit (EXIT_FAILURE);
        }
 
-       DCP* a = 0;
-       try {
-               a = new DCP (argv[optind]);
-               list<shared_ptr<DCPReadError> > errors;
-               a->read (keep_going, &errors);
-               for (list<shared_ptr<DCPReadError> >::const_iterator i = errors.begin(); i != errors.end(); ++i) {
-                       cerr << (*i)->what() << "\n";
-               }
-       } catch (FileError& e) {
-               cerr << "Could not read DCP " << argv[optind] << "; " << e.what() << " " << e.filename() << "\n";
-               exit (EXIT_FAILURE);
-       }
-
-       DCP* b = 0;
-       try {
-               b = new DCP (argv[optind + 1]);
-               list<shared_ptr<DCPReadError> > errors;
-               b->read (keep_going, &errors);
-               for (list<shared_ptr<DCPReadError> >::const_iterator i = errors.begin(); i != errors.end(); ++i) {
-                       cerr << (*i)->what() << "\n";
-               }
-       } catch (FileError& e) {
-               cerr << "Could not read DCP " << argv[optind + 1] << "; " << e.what() << " " << e.filename() << "\n";
-               exit (EXIT_FAILURE);
-       }
+       DCP* a = load_dcp (argv[optind], keep_going, ignore_missing_assets);
+       DCP* b = load_dcp (argv[optind + 1], keep_going, ignore_missing_assets);
 
        /* I think this is just below the LSB at 16-bits (ie the 8th most significant bit at 24-bit) */
        options.max_audio_sample_error = 255;
index 8f4c1b7d8cc419bdaa3553d7d8e2868ac81b5434..0c41b0c3a00fddbf7eb472b2818b05006792ca7b 100644 (file)
@@ -32,6 +32,7 @@
 #include "reel_subtitle_asset.h"
 #include "subtitle_string.h"
 #include "cpl.h"
+#include "common.h"
 
 using std::string;
 using std::cerr;
@@ -44,8 +45,9 @@ static void
 help (string n)
 {
        cerr << "Syntax: " << n << " [options] <DCP>\n"
-            << "  -s, --subtitles   list all subtitles\n"
-            << "  -k, --keep-going  carry on in the event of errors, if possible\n";
+            << "  -s, --subtitles              list all subtitles\n"
+            << "  -k, --keep-going             carry on in the event of errors, if possible\n"
+            << "      --ignore-missing-assets  ignore missing asset files\n";
 }
 
 static void
@@ -104,18 +106,20 @@ main (int argc, char* argv[])
 {
        bool subtitles = false;
        bool keep_going = false;
+       bool ignore_missing_assets = false;
        
        int option_index = 0;
        while (1) {
                static struct option long_options[] = {
-                       { "version", no_argument, 0, 'v'},
-                       { "help", no_argument, 0, 'h'},
-                       { "subtitles", no_argument, 0, 's'},
-                       { "keep-going", no_argument, 0, 'k'},
+                       { "version", no_argument, 0, 'v' },
+                       { "help", no_argument, 0, 'h' },
+                       { "subtitles", no_argument, 0, 's' },
+                       { "keep-going", no_argument, 0, 'k' },
+                       { "ignore-missing-assets", no_argument, 0, 'A' },
                        { 0, 0, 0, 0 }
                };
 
-               int c = getopt_long (argc, argv, "vhsk", long_options, &option_index);
+               int c = getopt_long (argc, argv, "vhskA", long_options, &option_index);
 
                if (c == -1) {
                        break;
@@ -134,6 +138,9 @@ main (int argc, char* argv[])
                case 'k':
                        keep_going = true;
                        break;
+               case 'A':
+                       ignore_missing_assets = true;
+                       break;
                }
        }
 
@@ -148,7 +155,7 @@ main (int argc, char* argv[])
        }
 
        DCP* dcp = 0;
-       list<shared_ptr<DCPReadError> > errors;
+       DCP::ReadErrors errors;
        try {
                dcp = new DCP (argv[optind]);
                dcp->read (keep_going, &errors);
@@ -162,7 +169,8 @@ main (int argc, char* argv[])
        
        cout << "DCP: " << boost::filesystem::path(argv[optind]).filename().string() << "\n";
 
-       for (list<shared_ptr<DCPReadError> >::const_iterator i = errors.begin(); i != errors.end(); ++i) {
+       dcp::filter_errors (errors, ignore_missing_assets);
+       for (DCP::ReadErrors::const_iterator i = errors.begin(); i != errors.end(); ++i) {
                cerr << "Error: " << (*i)->what() << "\n";
        }
 
@@ -181,7 +189,9 @@ main (int argc, char* argv[])
                                main_picture (*j);
                        } catch (UnresolvedRefError& e) {
                                if (keep_going) {
-                                       cerr << e.what() << " (for main picture)\n";
+                                       if (!ignore_missing_assets) {
+                                               cerr << e.what() << " (for main picture)\n";
+                                       }
                                } else {
                                        throw;
                                }
@@ -191,7 +201,9 @@ main (int argc, char* argv[])
                                main_sound (*j);
                        } catch (UnresolvedRefError& e) {
                                if (keep_going) {
-                                       cerr << e.what() << " (for main sound)\n";
+                                       if (!ignore_missing_assets) {
+                                               cerr << e.what() << " (for main sound)\n";
+                                       }
                                } else {
                                        throw;
                                }
@@ -201,7 +213,9 @@ main (int argc, char* argv[])
                                main_subtitle (*j, subtitles);
                        } catch (UnresolvedRefError& e) {
                                if (keep_going) {
-                                       cerr << e.what() << " (for main subtitle)\n";
+                                       if (!ignore_missing_assets) {
+                                               cerr << e.what() << " (for main subtitle)\n";
+                                       }
                                } else {
                                        throw;
                                }
index 518f00261b493923e42f78bb2d0666aca56c05f8..10389eaccc1bf9e491c987a895346af0c93b2cab 100644 (file)
@@ -2,12 +2,12 @@ def build(bld):
     obj = bld(features = 'cxx cxxprogram')
     obj.use = ['libdcp%s' % bld.env.API_VERSION]
     obj.uselib = 'OPENJPEG CXML'
-    obj.source = 'dcpdiff.cc'
+    obj.source = 'dcpdiff.cc common.cc'
     obj.target = 'dcpdiff'
 
     obj = bld(features = 'cxx cxxprogram')
     obj.use = ['libdcp%s' % bld.env.API_VERSION]
     obj.uselib = 'OPENJPEG CXML'
-    obj.source = 'dcpinfo.cc'
+    obj.source = 'dcpinfo.cc common.cc'
     obj.target = 'dcpinfo'