projects
/
dcpomatic.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
pot/merge.
[dcpomatic.git]
/
src
/
wx
/
content_colour_conversion_dialog.cc
diff --git
a/src/wx/content_colour_conversion_dialog.cc
b/src/wx/content_colour_conversion_dialog.cc
index 3fa8e120a8cff1d41284529cf4cc38d2aa52989c..b970618a7b973126311be88ce30ab4392c5dcf6a 100644
(file)
--- a/
src/wx/content_colour_conversion_dialog.cc
+++ b/
src/wx/content_colour_conversion_dialog.cc
@@
-1,45
+1,51
@@
/*
/*
- Copyright (C) 2013 Carl Hetherington <cth@carlh.net>
+ Copyright (C) 2013
-2021
Carl Hetherington <cth@carlh.net>
- 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.
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
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 <http://www.gnu.org/licenses/>.
*/
*/
-#include <wx/statline.h>
+
+#include "check_box.h"
+#include "colour_conversion_editor.h"
+#include "content_colour_conversion_dialog.h"
+#include "wx_util.h"
#include "lib/colour_conversion.h"
#include "lib/config.h"
#include "lib/util.h"
#include "lib/colour_conversion.h"
#include "lib/config.h"
#include "lib/util.h"
-#include
"wx_util.h"
-#include
"content_colour_conversion_dialog.h"
-#include "colour_conversion_editor.h"
+#include
<wx/statline.h>
+#include
<iostream>
+
+using std::cout;
using std::string;
using std::vector;
using std::string;
using std::vector;
-using std::cout;
using boost::optional;
using boost::optional;
-ContentColourConversionDialog::ContentColourConversionDialog (wxWindow* parent)
+
+ContentColourConversionDialog::ContentColourConversionDialog (wxWindow* parent, bool yuv)
: wxDialog (parent, wxID_ANY, _("Colour conversion"))
: wxDialog (parent, wxID_ANY, _("Colour conversion"))
- , _editor (new ColourConversionEditor
(this
))
+ , _editor (new ColourConversionEditor
(this, yuv
))
, _setting (false)
{
, _setting (false)
{
-
wxBoxSizer*
overall_sizer = new wxBoxSizer (wxVERTICAL);
+
auto
overall_sizer = new wxBoxSizer (wxVERTICAL);
SetSizer (overall_sizer);
SetSizer (overall_sizer);
-
wxFlexGridSizer* table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y
_GAP);
- _preset_check = new
wxCheckBox (this, wxID_ANY
, _("Use preset"));
+
auto table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_Y_GAP - 2, DCPOMATIC_SIZER_X
_GAP);
+ _preset_check = new
CheckBox (this
, _("Use preset"));
table->Add (_preset_check, 0, wxALIGN_CENTER_VERTICAL);
_preset_choice = new wxChoice (this, wxID_ANY);
table->Add (_preset_choice);
table->Add (_preset_check, 0, wxALIGN_CENTER_VERTICAL);
_preset_choice = new wxChoice (this, wxID_ANY);
table->Add (_preset_choice);
@@
-48,7
+54,7
@@
ContentColourConversionDialog::ContentColourConversionDialog (wxWindow* parent)
overall_sizer->Add (new wxStaticLine (this, wxID_ANY), 0, wxEXPAND);
overall_sizer->Add (_editor);
overall_sizer->Add (new wxStaticLine (this, wxID_ANY), 0, wxEXPAND);
overall_sizer->Add (_editor);
-
wxSizer* buttons = CreateSeparatedButtonSizer (wxOK
);
+
auto buttons = CreateSeparatedButtonSizer (wxOK | wxCANCEL
);
if (buttons) {
overall_sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder());
}
if (buttons) {
overall_sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder());
}
@@
-56,23
+62,24
@@
ContentColourConversionDialog::ContentColourConversionDialog (wxWindow* parent)
overall_sizer->Layout ();
overall_sizer->SetSizeHints (this);
overall_sizer->Layout ();
overall_sizer->SetSizeHints (this);
- _preset_check->Bind (wxEVT_C
OMMAND_CHECKBOX_CLICKED
, boost::bind (&ContentColourConversionDialog::preset_check_clicked, this));
- _preset_choice->Bind (wxEVT_C
OMMAND_CHOICE_SELECTED
, boost::bind (&ContentColourConversionDialog::preset_choice_changed, this));
+ _preset_check->Bind (wxEVT_C
HECKBOX
, boost::bind (&ContentColourConversionDialog::preset_check_clicked, this));
+ _preset_choice->Bind (wxEVT_C
HOICE
, boost::bind (&ContentColourConversionDialog::preset_choice_changed, this));
- _editor->Changed.connect (boost::bind (&ContentColourConversionDialog::check_for_preset, this));
+ _editor
_connection = _editor
->Changed.connect (boost::bind (&ContentColourConversionDialog::check_for_preset, this));
- vector<PresetColourConversion> presets = Config::instance()->colour_conversions ();
- for (vector<PresetColourConversion>::const_iterator i = presets.begin(); i != presets.end(); ++i) {
- _preset_choice->Append (std_to_wx (i->name));
+ for (auto const& i: PresetColourConversion::all ()) {
+ _preset_choice->Append (std_to_wx (i.name));
}
}
}
}
+
ColourConversion
ContentColourConversionDialog::get () const
{
return _editor->get ();
}
ColourConversion
ContentColourConversionDialog::get () const
{
return _editor->get ();
}
+
void
ContentColourConversionDialog::set (ColourConversion c)
{
void
ContentColourConversionDialog::set (ColourConversion c)
{
@@
-83,17
+90,18
@@
ContentColourConversionDialog::set (ColourConversion c)
check_for_preset ();
}
check_for_preset ();
}
+
void
ContentColourConversionDialog::check_for_preset ()
{
if (_setting) {
return;
}
void
ContentColourConversionDialog::check_for_preset ()
{
if (_setting) {
return;
}
-
- optional<size_t> preset = _editor->get().preset ();
- _preset_check->SetValue (preset);
- _preset_choice->Enable (preset);
+ auto preset = _editor->get().preset ();
+
+ _preset_check->SetValue (static_cast<bool>(preset));
+ _preset_choice->Enable (static_cast<bool>(preset));
if (preset) {
_preset_choice->SetSelection (preset.get ());
} else {
if (preset) {
_preset_choice->SetSelection (preset.get ());
} else {
@@
-101,6
+109,7
@@
ContentColourConversionDialog::check_for_preset ()
}
}
}
}
+
void
ContentColourConversionDialog::preset_check_clicked ()
{
void
ContentColourConversionDialog::preset_check_clicked ()
{
@@
-113,14
+122,13
@@
ContentColourConversionDialog::preset_check_clicked ()
}
}
}
}
+
void
ContentColourConversionDialog::preset_choice_changed ()
{
void
ContentColourConversionDialog::preset_choice_changed ()
{
-
vector<PresetColourConversion> presets = Config::instance()->colour_conversions
();
+
auto presets = PresetColourConversion::all
();
int const s = _preset_choice->GetCurrentSelection();
if (s != -1) {
set (presets[s].conversion);
}
}
int const s = _preset_choice->GetCurrentSelection();
if (s != -1) {
set (presets[s].conversion);
}
}
-
-