projects
/
dcpomatic.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Rename TYPE_DEBUG_PLAYER to TYPE_DEBUG_VIDEO_VIEW.
[dcpomatic.git]
/
src
/
lib
/
transcode_job.cc
diff --git
a/src/lib/transcode_job.cc
b/src/lib/transcode_job.cc
index 981816fd528f24edc19e071cf37526c88f37b714..fb586b0825631da9e9a9efc79038103733735791 100644
(file)
--- a/
src/lib/transcode_job.cc
+++ b/
src/lib/transcode_job.cc
@@
-42,6
+42,7
@@
using std::fixed;
using std::setprecision;
using std::cout;
using boost::shared_ptr;
using std::setprecision;
using std::cout;
using boost::shared_ptr;
+using boost::optional;
using boost::dynamic_pointer_cast;
/** @param film Film to use */
using boost::dynamic_pointer_cast;
/** @param film Film to use */
@@
-53,9
+54,6
@@
TranscodeJob::TranscodeJob (shared_ptr<const Film> film)
TranscodeJob::~TranscodeJob ()
{
TranscodeJob::~TranscodeJob ()
{
- /* We have to stop the job thread here as we're about to start tearing down
- the Encoder, which is bad news if the job thread is still feeding it data.
- */
stop_thread ();
}
stop_thread ();
}
@@
-87,8
+85,6
@@
TranscodeJob::run ()
DCPOMATIC_ASSERT (_encoder);
_encoder->go ();
DCPOMATIC_ASSERT (_encoder);
_encoder->go ();
- set_progress (1);
- set_state (FINISHED_OK);
struct timeval finish;
gettimeofday (&finish, 0);
struct timeval finish;
gettimeofday (&finish, 0);
@@
-112,6
+108,9
@@
TranscodeJob::run ()
_encoder.reset ();
_encoder.reset ();
+ set_progress (1);
+ set_state (FINISHED_OK);
+
} catch (...) {
_encoder.reset ();
throw;
} catch (...) {
_encoder.reset ();
throw;
@@
-138,11
+137,11
@@
TranscodeJob::status () const
_film->length().frames_round (_film->video_frame_rate ())
);
_film->length().frames_round (_film->video_frame_rate ())
);
-
float
const fps = _encoder->current_rate ();
+
optional<float>
const fps = _encoder->current_rate ();
if (fps) {
char fps_buffer[64];
/// TRANSLATORS: fps here is an abbreviation for frames per second
if (fps) {
char fps_buffer[64];
/// TRANSLATORS: fps here is an abbreviation for frames per second
- snprintf (fps_buffer, sizeof(fps_buffer), _("; %.1f fps"), fps);
+ snprintf (fps_buffer, sizeof(fps_buffer), _("; %.1f fps"),
*
fps);
strncat (buffer, fps_buffer, strlen(buffer) - 1);
}
}
strncat (buffer, fps_buffer, strlen(buffer) - 1);
}
}
@@
-164,12
+163,12
@@
TranscodeJob::remaining_time () const
/* We're encoding so guess based on the current encoding rate */
/* We're encoding so guess based on the current encoding rate */
-
float
fps = e->current_rate ();
+
optional<float>
fps = e->current_rate ();
- if (
fps == 0
) {
+ if (
!fps
) {
return 0;
}
/* Compute approximate proposed length here, as it's only here that we need it */
return 0;
}
/* Compute approximate proposed length here, as it's only here that we need it */
- return (_film->length().frames_round
(_film->video_frame_rate ()) - e->frames_done()) /
fps;
+ return (_film->length().frames_round
(_film->video_frame_rate()) - e->frames_done()) / *
fps;
}
}