X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fframe_rate_change.cc;h=8ad6de9425aa2675158ed5b7a3421658e4c0aedf;hb=001ba1644fc6aa54f91fcaaa62ae7e5de2313bc1;hp=d327f8e31ad151d8cdd7e7c517b836c28394b78f;hpb=e60bb3e51bd1508b149e6b8f6608f09b5196ae26;p=dcpomatic.git diff --git a/src/lib/frame_rate_change.cc b/src/lib/frame_rate_change.cc index d327f8e31..8ad6de942 100644 --- a/src/lib/frame_rate_change.cc +++ b/src/lib/frame_rate_change.cc @@ -1,65 +1,66 @@ /* - Copyright (C) 2012-2014 Carl Hetherington + Copyright (C) 2012-2016 Carl Hetherington - This program is free software; you can redistribute it and/or modify + This file is part of DCP-o-matic. + + DCP-o-matic 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, + DCP-o-matic 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. + along with DCP-o-matic. If not, see . */ -#include #include "frame_rate_change.h" +#include "types.h" +#include "content.h" +#include "film.h" #include "compose.hpp" +#include #include "i18n.h" using std::string; +using boost::shared_ptr; static bool -about_equal (float a, float b) +about_equal (double a, double b) { - /* A film of F seconds at f FPS will be Ff frames; - Consider some delta FPS d, so if we run the same - film at (f + d) FPS it will last F(f + d) seconds. - - Hence the difference in length over the length of the film will - be F(f + d) - Ff frames - = Ff + Fd - Ff frames - = Fd frames - = Fd/f seconds - - So if we accept a difference of 1 frame, ie 1/f seconds, we can - say that - - 1/f = Fd/f - ie 1 = Fd - ie d = 1/F + return (fabs (a - b) < VIDEO_FRAME_RATE_EPSILON); +} - So for a 3hr film, ie F = 3 * 60 * 60 = 10800, the acceptable - FPS error is 1/F ~= 0.0001 ~= 10-e4 - */ +FrameRateChange::FrameRateChange () + : source (24) + , dcp (24) + , skip (false) + , repeat (1) + , change_speed (false) + , speed_up (1) +{ - return (fabs (a - b) < 1e-4); } - -FrameRateChange::FrameRateChange (float source_, int dcp_) - : source (source_) - , dcp (dcp_) - , skip (false) +FrameRateChange::FrameRateChange (double source_, int dcp_) + : skip (false) , repeat (1) , change_speed (false) { + construct (source_, dcp_); +} + +void +FrameRateChange::construct (double source_, int dcp_) +{ + source = source_; + dcp = dcp_; + if (fabs (source / 2.0 - dcp) < fabs (source - dcp)) { /* The difference between source and DCP frame rate will be lower (i.e. better) if we skip. @@ -77,6 +78,20 @@ FrameRateChange::FrameRateChange (float source_, int dcp_) change_speed = !about_equal (speed_up, 1.0); } +FrameRateChange::FrameRateChange (shared_ptr film, shared_ptr content) + : skip (false) + , repeat (1) +{ + construct (content->active_video_frame_rate(film), film->video_frame_rate()); +} + +FrameRateChange::FrameRateChange (shared_ptr film, Content const * content) + : skip (false) + , repeat (1) +{ + construct (content->active_video_frame_rate(film), film->video_frame_rate()); +} + string FrameRateChange::description () const { @@ -94,8 +109,10 @@ FrameRateChange::description () const } if (change_speed) { - float const pc = dcp * 100 / (source * factor()); - description += String::compose (_("DCP will run at %1%% of the content speed.\n"), pc); + double const pc = dcp * 100 / (source * factor()); + char buffer[256]; + snprintf (buffer, sizeof(buffer), _("DCP will run at %.1f%% of the content speed.\n"), pc); + description += buffer; } }