Lots of #include <iostream>s for Arch.
[dcpomatic.git] / src / wx / colour_conversion_editor.cc
index 482e403095ab94879660e89c8415b3c327cf0b1a..1d164cb334e4ada2029af231d7511cdb08cb72dd 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2013-2014 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2013-2015 Carl Hetherington <cth@carlh.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -27,6 +27,7 @@
 #include <wx/spinctrl.h>
 #include <wx/gbsizer.h>
 #include <boost/lexical_cast.hpp>
+#include <iostream>
 
 using std::string;
 using std::cout;
@@ -40,14 +41,14 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
        wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL);
        SetSizer (overall_sizer);
 
-       wxGridBagSizer* table = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
+       wxGridBagSizer* table = new wxGridBagSizer (DCPOMATIC_SIZER_Y_GAP - 3, DCPOMATIC_SIZER_X_GAP);
        overall_sizer->Add (table, 1, wxEXPAND | wxALL, DCPOMATIC_DIALOG_BORDER);
 
        int r = 0;
 
        subhead (table, this, _("Input gamma correction"), r);
 
-       _input_gamma_linearised = new wxCheckBox (this, wxID_ANY, _("Linearise input gamma curve for low values"));
+       _input_gamma_linearised = new wxCheckBox (this, wxID_ANY, _("Linearise input gamma curve for small values"));
        table->Add (_input_gamma_linearised, wxGBPosition (r, 0), wxGBSpan (1, 2));
        ++r;
 
@@ -57,25 +58,23 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
        ++r;
 
        add_label_to_grid_bag_sizer (table, this, _("Input power"), true, wxGBPosition (r, 0));
-       _input_power = new wxSpinCtrlDouble (this);
-       table->Add (_input_power, wxGBPosition (r, 1));
-       ++r;
-
-       add_label_to_grid_bag_sizer (table, this, _("Input threshold"), true, wxGBPosition (r, 0));
-       _input_threshold = new wxTextCtrl (this, wxID_ANY, wxT (""));
-       table->Add (_input_threshold, wxGBPosition (r, 1));
-       ++r;
-
-       add_label_to_grid_bag_sizer (table, this, _("Input A value"), true, wxGBPosition (r, 0));
-       _input_A = new wxTextCtrl (this, wxID_ANY, wxT (""));
-       table->Add (_input_A, wxGBPosition (r, 1));
+       {
+               wxBoxSizer* s = new wxBoxSizer (wxHORIZONTAL);
+               _input_power = new wxSpinCtrlDouble (this);
+               s->Add (_input_power, 1, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_GAP);
+               add_label_to_sizer (s, this, _("threshold"), true);
+               _input_threshold = new wxTextCtrl (this, wxID_ANY, wxT (""));
+               s->Add (_input_threshold, 1, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_GAP);
+               add_label_to_sizer (s, this, _("A"), true);
+               _input_A = new wxTextCtrl (this, wxID_ANY, wxT (""));
+               s->Add (_input_A, 1, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_GAP);
+               add_label_to_sizer (s, this, _("B"), true);
+               _input_B = new wxTextCtrl (this, wxID_ANY, wxT (""));
+               s->Add (_input_B, 1, wxEXPAND | wxRIGHT, DCPOMATIC_SIZER_GAP);
+               table->Add (s, wxGBPosition (r, 1), wxGBSpan (1, 3));
+       }
        ++r;
 
-       add_label_to_grid_bag_sizer (table, this, _("Input B value"), true, wxGBPosition (r, 0));
-       _input_B = new wxTextCtrl (this, wxID_ANY, wxT (""));
-       table->Add (_input_B, wxGBPosition (r, 1));
-       ++r;
-       
         wxClientDC dc (parent);
         wxSize size = dc.GetTextExtent (wxT ("-0.12345678901"));
         size.SetHeight (-1);
@@ -94,7 +93,7 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
        /* YUV to RGB conversion */
 
        subhead (table, this, _("YUV to RGB conversion"), r);
-       
+
        add_label_to_grid_bag_sizer (table, this, _("YUV to RGB matrix"), true, wxGBPosition (r, 0));
        _yuv_to_rgb = new wxChoice (this, wxID_ANY);
        _yuv_to_rgb->Append (_("Rec. 601"));
@@ -141,7 +140,7 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
 
         size = dc.GetTextExtent (wxT ("0.12345678"));
         size.SetHeight (-1);
-       
+
        wxFlexGridSizer* rgb_to_xyz_sizer = new wxFlexGridSizer (3, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
        for (int i = 0; i < 3; ++i) {
                for (int j = 0; j < 3; ++j) {
@@ -171,7 +170,7 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
 
         size = dc.GetTextExtent (wxT ("0.12345678"));
         size.SetHeight (-1);
-       
+
        wxFlexGridSizer* bradford_sizer = new wxFlexGridSizer (3, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
        for (int i = 0; i < 3; ++i) {
                for (int j = 0; j < 3; ++j) {
@@ -181,27 +180,12 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
        }
        table->Add (bradford_sizer, wxGBPosition (r - 2, 3), wxGBSpan (2, 1));
 
-       subhead (table, this, _("Output gamma correction"), r);
-       
-       add_label_to_grid_bag_sizer (table, this, _("Output gamma"), true, wxGBPosition (r, 0));
-       wxBoxSizer* output_sizer = new wxBoxSizer (wxHORIZONTAL);
-       /// TRANSLATORS: this means the mathematical reciprocal operation, i.e. we are dividing 1 by the control that
-       /// comes after it.
-       add_label_to_sizer (output_sizer, this, _("1 / "), false);
-       _output_gamma = new wxSpinCtrlDouble (this);
-       output_sizer->Add (_output_gamma);
-       table->Add (output_sizer, wxGBPosition (r, 1), wxGBSpan (1, 2));
-       ++r;
-
        _input_gamma->SetRange (0.1, 4.0);
        _input_gamma->SetDigits (2);
        _input_gamma->SetIncrement (0.1);
        _input_power->SetRange (0.1, 4.0);
-       _input_power->SetDigits (2);
+       _input_power->SetDigits (6);
        _input_power->SetIncrement (0.1);
-       _output_gamma->SetRange (0.1, 4.0);
-       _output_gamma->SetDigits (2);
-       _output_gamma->SetIncrement (0.1);
 
        _input_gamma->Bind (wxEVT_COMMAND_SPINCTRLDOUBLE_UPDATED, boost::bind (&ColourConversionEditor::changed, this, _input_gamma));
        _input_gamma_linearised->Bind (wxEVT_COMMAND_CHECKBOX_CLICKED, boost::bind (&ColourConversionEditor::changed, this));
@@ -221,7 +205,6 @@ ColourConversionEditor::ColourConversionEditor (wxWindow* parent)
        _adjusted_white_x->Bind (wxEVT_COMMAND_TEXT_UPDATED, boost::bind (&ColourConversionEditor::adjusted_white_changed, this));
        _adjusted_white_y->Bind (wxEVT_COMMAND_TEXT_UPDATED, boost::bind (&ColourConversionEditor::adjusted_white_changed, this));
        _yuv_to_rgb->Bind (wxEVT_COMMAND_CHOICE_SELECTED, boost::bind (&ColourConversionEditor::changed, this));
-       _output_gamma->Bind (wxEVT_COMMAND_SPINCTRLDOUBLE_UPDATED, boost::bind (&ColourConversionEditor::changed, this, _output_gamma));
 }
 
 void
@@ -302,8 +285,6 @@ ColourConversionEditor::set (ColourConversion conversion)
 
        update_rgb_to_xyz ();
        update_bradford ();
-       
-       set_spin_ctrl (_output_gamma, dynamic_pointer_cast<const dcp::GammaTransferFunction> (conversion.out ())->gamma ());
 }
 
 ColourConversion
@@ -354,7 +335,7 @@ ColourConversionEditor::get () const
                conversion.unset_adjusted_white ();
        }
 
-       conversion.set_out (shared_ptr<dcp::GammaTransferFunction> (new dcp::GammaTransferFunction (_output_gamma->GetValue ())));
+       conversion.set_out (shared_ptr<dcp::GammaTransferFunction> (new dcp::GammaTransferFunction (2.6)));
 
        return conversion;
 }
@@ -368,7 +349,7 @@ ColourConversionEditor::changed ()
        _input_threshold->Enable (lin);
        _input_A->Enable (lin);
        _input_B->Enable (lin);
-       
+
        Changed ();
 }
 
@@ -391,7 +372,7 @@ ColourConversionEditor::update_bradford ()
 {
        _adjusted_white_x->Enable (_adjust_white->GetValue ());
        _adjusted_white_y->Enable (_adjust_white->GetValue ());
-       
+
        boost::numeric::ublas::matrix<double> m = get().bradford ();
        for (int i = 0; i < 3; ++i) {
                for (int j = 0; j < 3; ++j) {
@@ -429,7 +410,7 @@ ColourConversionEditor::changed (wxSpinCtrlDouble* sc)
        if (fabs (_last_spin_ctrl_value[sc] - sc->GetValue()) < 1e-3) {
                return;
        }
-       
+
        Changed ();
 }