Remove configuration option for minimum frame size.

Replace it with a fixed 16KB as per #1902.

3 files changed, 2 insertions, 14 deletions

diff --git a/src/lib/config.cc b/src/lib/config.cc
index 4404c8314..ce02b046e 100644
--- a/src/lib/config.cc
+++ b/src/lib/config.cc
@@ -175,7 +175,6 @@ Config::set_defaults ()
 	_player_playlist_directory = boost::none;
 	_player_kdm_directory = boost::none;
 	_audio_mapping = boost::none;
-	_minimum_frame_size = 65536;
 	_custom_languages.clear ();
 
 	_allowed_dcp_frame_rates.clear ();
@@ -545,8 +544,6 @@ try
 		_audio_mapping = AudioMapping (f.node_child("AudioMapping"), Film::current_state_version);
 	}
 
-	_minimum_frame_size = f.optional_number_child<int>("MinimumFrameSize").get_value_or(65536);
-
 	for (auto i: f.node_children("CustomLanguage")) {
 		try {
 			/* This will fail if it's called before dcp::init() as it won't recognise the
@@ -976,7 +973,6 @@ Config::write_config () const
 	if (_audio_mapping) {
 		_audio_mapping->as_xml (root->add_child("AudioMapping"));
 	}
-	root->add_child("MinimumFrameSize")->add_child_text(raw_convert<string>(_minimum_frame_size));
 	for (auto const& i: _custom_languages) {
 		root->add_child("CustomLanguage")->add_child_text(i.to_string());
 	}
diff --git a/src/lib/config.h b/src/lib/config.h
index eb8f88da9..da29a8955 100644
--- a/src/lib/config.h
+++ b/src/lib/config.h
@@ -536,10 +536,6 @@ public:
 
 	AudioMapping audio_mapping (int output_channels);
 
-	int minimum_frame_size () const {
-		return _minimum_frame_size;
-	}
-
 	std::vector<dcp::LanguageTag> custom_languages () const {
 		return _custom_languages;
 	}
@@ -1051,10 +1047,6 @@ public:
 	void set_audio_mapping (AudioMapping m);
 	void set_audio_mapping_to_default ();
 
-	void set_minimum_frame_size (int size) {
-		maybe_set (_minimum_frame_size, size);
-	}
-
 	void add_custom_language (dcp::LanguageTag tag);
 
 	void changed (Property p = OTHER);
@@ -1268,7 +1260,6 @@ private:
 	boost::optional<boost::filesystem::path> _player_playlist_directory;
 	boost::optional<boost::filesystem::path> _player_kdm_directory;
 	boost::optional<AudioMapping> _audio_mapping;
-	int _minimum_frame_size;
 	std::vector<dcp::LanguageTag> _custom_languages;
 
 	static int const _current_version;
diff --git a/src/lib/dcp_video.cc b/src/lib/dcp_video.cc
index ed68299bb..4a505a7e1 100644
--- a/src/lib/dcp_video.cc
+++ b/src/lib/dcp_video.cc
@@ -128,7 +128,8 @@ DCPVideo::encode_locally ()
 	auto const comment = Config::instance()->dcp_j2k_comment();
 
 	ArrayData enc = {};
-	int const minimum_size = Config::instance()->minimum_frame_size();
+	/* This was empirically derived by a user: see #1902 */
+	int const minimum_size = 16384;
 	LOG_GENERAL ("Using minimum frame size %1", minimum_size);
 
 	auto xyz = convert_to_xyz (_frame, boost::bind(&Log::dcp_log, dcpomatic_log.get(), _1, _2));