X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Fexport_dialog.cc;h=5fc4036feddfce2df9b14bb97692dee26a5cf7a7;hb=0b15f493754d1b630338b17f2a6dfc66ccc4f6b7;hp=52aa03c20028841d0d08f3936fc1b834f21ca80b;hpb=d8ea1796f34ff894b148a0af78c0a547e0496ee1;p=dcpomatic.git

diff --git a/src/wx/export_dialog.cc b/src/wx/export_dialog.cc
index 52aa03c20..5fc4036fe 100644
--- a/src/wx/export_dialog.cc
+++ b/src/wx/export_dialog.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2017 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2017-2018 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -21,35 +21,70 @@
 #include "export_dialog.h"
 #include "file_picker_ctrl.h"
 #include "wx_util.h"
+#include "check_box.h"
+#include <wx/filepicker.h>
 #include <boost/bind.hpp>
 
+using std::string;
 using boost::bind;
 
-#define FORMATS 2
+#define FORMATS 3
 
 wxString format_names[] = {
 	_("ProRes"),
-	_("H.264")
+	_("MP4 / H.264"),
+	_("DCP subtitles")
 };
 
 wxString format_filters[] = {
 	_("MOV files (*.mov)|*.mov"),
 	_("MP4 files (*.mp4)|*.mp4"),
+	_("Subtitle files (*.xml)|*.xml"),
 };
 
-FFmpegEncoder::Format formats[] = {
-	FFmpegEncoder::FORMAT_PRORES,
-	FFmpegEncoder::FORMAT_H264,
+wxString format_extensions[] = {
+	"mov",
+	"mp4",
+	"xml",
 };
 
-ExportDialog::ExportDialog (wxWindow* parent)
+ExportFormat formats[] = {
+	EXPORT_FORMAT_PRORES,
+	EXPORT_FORMAT_H264_AAC,
+	EXPORT_FORMAT_SUBTITLES_DCP
+};
+
+ExportDialog::ExportDialog (wxWindow* parent, string name)
 	: TableDialog (parent, _("Export film"), 2, 1, true)
+	, _initial_name (name)
 {
 	add (_("Format"), true);
 	_format = new wxChoice (this, wxID_ANY);
 	add (_format);
+	add_spacer ();
+	_mixdown = new CheckBox (this, _("Mix audio down to stereo"));
+	add (_mixdown, false);
+	add_spacer ();
+	_split_reels = new CheckBox (this, _("Write reels into separate files"));
+	add (_split_reels, false);
+	_x264_crf_label[0] = add (_("Quality"), true);
+	_x264_crf = new wxSlider (this, wxID_ANY, 23, 0, 51, wxDefaultPosition, wxDefaultSize, wxSL_HORIZONTAL | wxSL_LABELS);
+	add (_x264_crf, false);
+	add_spacer ();
+	_x264_crf_label[1] = add (_("0 is best, 51 is worst"), false);
+	wxFont font = _x264_crf_label[1]->GetFont();
+	font.SetStyle(wxFONTSTYLE_ITALIC);
+	font.SetPointSize(font.GetPointSize() - 1);
+	_x264_crf_label[1]->SetFont(font);
+
 	add (_("Output file"), true);
-	_file = new FilePickerCtrl (this, _("Select output file"), format_filters[0], false);
+	/* Don't warn overwrite here, because on Linux (at least) if we specify a filename like foo
+	   the wxFileDialog will check that foo exists, but we will add an extension so we actually
+	   need to check if foo.mov (or similar) exists.  I can't find a way to make wxWidgets do this,
+	   so disable its check and the caller will have to do it themselves.
+	*/
+	_file = new FilePickerCtrl (this, _("Select output file"), format_filters[0], false, false);
+	_file->SetPath (_initial_name);
 	add (_file);
 
 	for (int i = 0; i < FORMATS; ++i) {
@@ -57,9 +92,18 @@ ExportDialog::ExportDialog (wxWindow* parent)
 	}
 	_format->SetSelection (0);
 
+	_x264_crf->Enable (false);
+	for (int i = 0; i < 2; ++i) {
+		_x264_crf_label[i]->Enable (false);
+	}
+
 	_format->Bind (wxEVT_CHOICE, bind (&ExportDialog::format_changed, this));
+	_file->Bind (wxEVT_FILEPICKER_CHANGED, bind (&ExportDialog::file_changed, this));
 
 	layout ();
+
+	wxButton* ok = dynamic_cast<wxButton *> (FindWindowById (wxID_OK, this));
+	ok->Enable (false);
 }
 
 void
@@ -67,18 +111,50 @@ ExportDialog::format_changed ()
 {
 	DCPOMATIC_ASSERT (_format->GetSelection() >= 0 && _format->GetSelection() < FORMATS);
 	_file->SetWildcard (format_filters[_format->GetSelection()]);
-	_file->SetPath ("");
+	_file->SetPath (_initial_name);
+	_x264_crf->Enable (_format->GetSelection() == 1);
+	for (int i = 0; i < 2; ++i) {
+		_x264_crf_label[i]->Enable (_format->GetSelection() == 1);
+	}
+	_mixdown->Enable (_format->GetSelection() != 2);
 }
 
 boost::filesystem::path
 ExportDialog::path () const
 {
-	return wx_to_std (_file->GetPath ());
+	wxFileName fn (_file->GetPath());
+	fn.SetExt (format_extensions[_format->GetSelection()]);
+	return wx_to_std (fn.GetFullPath());
 }
 
-FFmpegEncoder::Format
+ExportFormat
 ExportDialog::format () const
 {
 	DCPOMATIC_ASSERT (_format->GetSelection() >= 0 && _format->GetSelection() < FORMATS);
 	return formats[_format->GetSelection()];
 }
+
+bool
+ExportDialog::mixdown_to_stereo () const
+{
+	return _mixdown->GetValue ();
+}
+
+bool
+ExportDialog::split_reels () const
+{
+	return _split_reels->GetValue ();
+}
+
+int
+ExportDialog::x264_crf () const
+{
+	return _x264_crf->GetValue ();
+}
+
+void
+ExportDialog::file_changed ()
+{
+	wxButton* ok = dynamic_cast<wxButton *> (FindWindowById (wxID_OK, this));
+	ok->Enable (_file->GetPath().Length() > 0);
+}