Tidy up test film naming.

[dcpomatic.git] / src / lib / examine_content_job.cc

diff --git a/src/lib/examine_content_job.cc b/src/lib/examine_content_job.cc
index 5800c6f24dd643e40d2bf4db26f426be08681e04..f0fb29812ffc99bf012a116e6216fa97ec0febae 100644 (file)
--- a/src/lib/examine_content_job.cc
+++ b/src/lib/examine_content_job.cc
@@ -28,12 +28,13 @@
 #include "decoder.h"
 #include "imagemagick_encoder.h"
 #include "transcoder.h"
+#include "log.h"
 
 using namespace std;
 using namespace boost;
 
 ExamineContentJob::ExamineContentJob (shared_ptr<const FilmState> fs, Log* l, shared_ptr<Job> req)
-       : Job (fs, shared_ptr<Options> (), l, req)
+       : Job (fs, l, req)
 {
 
 }
@@ -45,6 +46,10 @@ ExamineContentJob::~ExamineContentJob ()
 string
 ExamineContentJob::name () const
 {
+       if (_fs->name().empty ()) {
+               return "Examine content";
+       }
+       
        return String::compose ("Examine content of %1", _fs->name());
 }
 
@@ -63,7 +68,10 @@ ExamineContentJob::run ()
 
        _decoder = decoder_factory (fs, o, this, _log, true, true);
        _decoder->go ();
-       fs->set_length (last_video_frame ());
+
+       fs->set_length (_decoder->last_video_frame ());
+
+       _log->log (String::compose ("Video length is %1 frames", _decoder->last_video_frame()));
 
        ascend ();
 
@@ -81,8 +89,6 @@ ExamineContentJob::run ()
                shared_ptr<ImageMagickEncoder> e (new ImageMagickEncoder (fs, o, _log));
                Transcoder w (fs, o, this, _log, e);
                w.go ();
-               set_progress (1);
-               set_state (FINISHED_OK);
                
        } catch (std::exception& e) {
 
@@ -90,6 +96,7 @@ ExamineContentJob::run ()
                set_progress (1);
                set_error (e.what ());
                set_state (FINISHED_ERROR);
+               return;
                
        }