dcp_trim -> trim

5 files changed, 33 insertions, 33 deletions

diff --git a/src/lib/check_hashes_job.cc b/src/lib/check_hashes_job.cc
index 2ad97c382..cf4e86b79 100644
--- a/src/lib/check_hashes_job.cc
+++ b/src/lib/check_hashes_job.cc
@@ -57,12 +57,12 @@ CheckHashesJob::run ()
 		throw EncodeError ("cannot check hashes of a DCP with unknown length");
 	}
 	
-	SourceFrame const N = _film->dcp_trim_start() + _film->dcp_length().get();
+	SourceFrame const N = _film->trim_start() + _film->dcp_length().get();
 	DCPFrameRate const dfr (_film->frames_per_second ());
 
 	int const inc = dfr.skip ? 2 : 1;
 	
-	for (SourceFrame i = _film->dcp_trim_start(); i < N; i += inc) {
+	for (SourceFrame i = _film->trim_start(); i < N; i += inc) {
 		string const j2k_file = _film->frame_out_path (i, false);
 		string const hash_file = _film->hash_out_path (i, false);
 
diff --git a/src/lib/film.cc b/src/lib/film.cc
index 2934595ab..8fe03184d 100644
--- a/src/lib/film.cc
+++ b/src/lib/film.cc
@@ -73,7 +73,7 @@ using boost::ends_with;
 using boost::starts_with;
 using boost::optional;
 
-int const Film::state_version = 1;
+int const Film::state_version = 2;
 
 /** Construct a Film object in a given directory, reading any metadata
  *  file that exists in that directory.  An exception will be thrown if
@@ -89,8 +89,8 @@ Film::Film (string d, bool must_exist)
 	, _dcp_content_type (0)
 	, _format (0)
 	, _scaler (Scaler::from_id ("bicubic"))
-	, _dcp_trim_start (0)
-	, _dcp_trim_end (0)
+	, _trim_start (0)
+	, _trim_end (0)
 	, _dcp_ab (false)
 	, _use_content_audio (true)
 	, _audio_gain (0)
@@ -155,8 +155,8 @@ Film::Film (Film const & o)
 	, _crop              (o._crop)
 	, _filters           (o._filters)
 	, _scaler            (o._scaler)
-	, _dcp_trim_start    (o._dcp_trim_start)
-	, _dcp_trim_end      (o._dcp_trim_end)
+	, _trim_start        (o._trim_start)
+	, _trim_end          (o._trim_end)
 	, _reel_size         (o._reel_size)
 	, _dcp_ab            (o._dcp_ab)
 	, _content_audio_stream (o._content_audio_stream)
@@ -388,8 +388,8 @@ Film::write_metadata () const
 		f << "filter " << (*i)->id () << "\n";
 	}
 	f << "scaler " << _scaler->id () << "\n";
-	f << "dcp_trim_start " << _dcp_trim_start << "\n";
-	f << "dcp_trim_end " << _dcp_trim_end << "\n";
+	f << "trim_start " << _trim_start << "\n";
+	f << "trim_end " << _trim_end << "\n";
 	if (_reel_size) {
 		f << "reel_size " << _reel_size.get() << "\n";
 	}
@@ -503,10 +503,10 @@ Film::read_metadata ()
 			_filters.push_back (Filter::from_id (v));
 		} else if (k == "scaler") {
 			_scaler = Scaler::from_id (v);
-		} else if (k == "dcp_trim_start") {
-			_dcp_trim_start = atoi (v.c_str ());
-		} else if (k == "dcp_trim_end") {
-			_dcp_trim_end = atoi (v.c_str ());
+		} else if ( ((!version || version < 2) && k == "trim_start") || k == "trim_start") {
+			_trim_start = atoi (v.c_str ());
+		} else if ( ((!version || version < 2) && k == "trim_end") || k == "trim_end") {
+			_trim_end = atoi (v.c_str ());
 		} else if (k == "reel_size") {
 			_reel_size = boost::lexical_cast<uint64_t> (v);
 		} else if (k == "dcp_ab") {
@@ -706,7 +706,7 @@ Film::dcp_length () const
 		return boost::optional<int> ();
 	}
 
-	return length().get() - dcp_trim_start() - dcp_trim_end();
+	return length().get() - trim_start() - trim_end();
 }
 
 /** @return a DCI-compliant name for a DCP of this film */
@@ -1053,23 +1053,23 @@ Film::set_scaler (Scaler const * s)
 }
 
 void
-Film::set_dcp_trim_start (int t)
+Film::set_trim_start (int t)
 {
 	{
 		boost::mutex::scoped_lock lm (_state_mutex);
-		_dcp_trim_start = t;
+		_trim_start = t;
 	}
-	signal_changed (DCP_TRIM_START);
+	signal_changed (TRIM_START);
 }
 
 void
-Film::set_dcp_trim_end (int t)
+Film::set_trim_end (int t)
 {
 	{
 		boost::mutex::scoped_lock lm (_state_mutex);
-		_dcp_trim_end = t;
+		_trim_end = t;
 	}
-	signal_changed (DCP_TRIM_END);
+	signal_changed (TRIM_END);
 }
 
 void
@@ -1457,7 +1457,7 @@ Film::video_range () const
 		return boost::optional<pair<SourceFrame, SourceFrame> > ();
 	}
 	
-	return make_pair (dcp_trim_start(), dcp_trim_start() + dcp_length().get());
+	return make_pair (trim_start(), trim_start() + dcp_length().get());
 }
 
 boost::optional<pair<int64_t, int64_t> >
diff --git a/src/lib/film.h b/src/lib/film.h
index 6c27af3ab..4cecf663c 100644
--- a/src/lib/film.h
+++ b/src/lib/film.h
@@ -120,8 +120,8 @@ public:
 		CROP,
 		FILTERS,
 		SCALER,
-		DCP_TRIM_START,
-		DCP_TRIM_END,
+		TRIM_START,
+		TRIM_END,
 		REEL_SIZE,
 		DCP_AB,
 		CONTENT_AUDIO_STREAM,
@@ -197,14 +197,14 @@ public:
 		return _scaler;
 	}
 
-	SourceFrame dcp_trim_start () const {
+	SourceFrame trim_start () const {
 		boost::mutex::scoped_lock lm (_state_mutex);
-		return _dcp_trim_start;
+		return _trim_start;
 	}
 
-	SourceFrame dcp_trim_end () const {
+	SourceFrame trim_end () const {
 		boost::mutex::scoped_lock lm (_state_mutex);
-		return _dcp_trim_end;
+		return _trim_end;
 	}
 
 	boost::optional<uint64_t> reel_size () const {
@@ -365,8 +365,8 @@ public:
 	void set_bottom_crop (int);
 	void set_filters (std::vector<Filter const *>);
 	void set_scaler (Scaler const *);
-	void set_dcp_trim_start (int);
-	void set_dcp_trim_end (int);
+	void set_trim_start (int);
+	void set_trim_end (int);
 	void set_reel_size (uint64_t);
 	void unset_reel_size ();
 	void set_dcp_ab (bool);
@@ -448,9 +448,9 @@ private:
 	/** Scaler algorithm to use */
 	Scaler const * _scaler;
 	/** Frames to trim off the start of the DCP */
-	int _dcp_trim_start;
+	int _trim_start;
 	/** Frames to trim off the end of the DCP */
-	int _dcp_trim_end;
+	int _trim_end;
 	/** Approximate target reel size in bytes; if not set, use a single reel */
 	boost::optional<uint64_t> _reel_size;
 	/** true to create an A/B comparison DCP, where the left half of the image
diff --git a/src/lib/make_dcp_job.cc b/src/lib/make_dcp_job.cc
index efc2f05d5..fcb10069d 100644
--- a/src/lib/make_dcp_job.cc
+++ b/src/lib/make_dcp_job.cc
@@ -63,7 +63,7 @@ MakeDCPJob::j2c_path (int f, int offset) const
 {
 	DCPFrameRate dfr (_film->frames_per_second());
 	int const mult = dfr.skip ? 2 : 1;
-	SourceFrame const s = ((f + offset) * mult) + _film->dcp_trim_start();
+	SourceFrame const s = ((f + offset) * mult) + _film->trim_start();
 	return _film->frame_out_path (s, false);
 }
 
diff --git a/src/lib/transcode_job.cc b/src/lib/transcode_job.cc
index 23a3d9fe8..c9792ed2e 100644
--- a/src/lib/transcode_job.cc
+++ b/src/lib/transcode_job.cc
@@ -120,6 +120,6 @@ TranscodeJob::remaining_time () const
 	}
 
 	/* We assume that dcp_length() is valid, if it is set */
-	SourceFrame const left = _film->dcp_trim_start() + _film->dcp_length().get() - _encoder->video_frame();
+	SourceFrame const left = _film->trim_start() + _film->dcp_length().get() - _encoder->video_frame();
 	return left / fps;
 }