Hide warnings triggered by Ubuntu 20.04's gcc.
[dcpomatic.git] / src / wx / audio_mapping_view.cc
index df2759ef6b97021dadd2802b82fe035ec961ac13..e1c96fdebe962d35784f5940c63aa03a52ed9a33 100644 (file)
 #include "audio_gain_dialog.h"
 #include "lib/audio_mapping.h"
 #include "lib/util.h"
+#include "lib/warnings.h"
 #include <dcp/locale_convert.h>
 #include <dcp/types.h>
+DCPOMATIC_DISABLE_WARNINGS
 #include <wx/wx.h>
 #include <wx/renderer.h>
 #include <wx/grid.h>
 #include <wx/graphics.h>
+DCPOMATIC_ENABLE_WARNINGS
 #include <boost/foreach.hpp>
 #include <iostream>
 
@@ -55,9 +58,10 @@ using dcp::locale_convert;
 
 enum {
        ID_off = 1,
-       ID_full = 2,
-       ID_minus6dB = 3,
-       ID_edit = 4
+       ID_minus6dB = 2,
+       ID_0dB = 3,
+       ID_plus3dB = 4,
+       ID_edit = 5
 };
 
 AudioMappingView::AudioMappingView (wxWindow* parent, wxString left_label, wxString from, wxString top_label, wxString to)
@@ -71,8 +75,9 @@ AudioMappingView::AudioMappingView (wxWindow* parent, wxString left_label, wxStr
 {
        _menu = new wxMenu;
        _menu->Append (ID_off, _("Off"));
-       _menu->Append (ID_full, _("Full"));
        _menu->Append (ID_minus6dB, _("-6dB"));
+       _menu->Append (ID_0dB, _("0dB (unchanged)"));
+       _menu->Append (ID_plus3dB, _("+3dB"));
        _menu->Append (ID_edit, _("Edit..."));
 
        _body = new wxPanel (this, wxID_ANY);
@@ -85,8 +90,9 @@ AudioMappingView::AudioMappingView (wxWindow* parent, wxString left_label, wxStr
 
        Bind (wxEVT_SIZE, boost::bind(&AudioMappingView::size, this, _1));
        Bind (wxEVT_MENU, boost::bind(&AudioMappingView::set_gain_from_menu, this, 0), ID_off);
-       Bind (wxEVT_MENU, boost::bind(&AudioMappingView::set_gain_from_menu, this, 1), ID_full);
        Bind (wxEVT_MENU, boost::bind(&AudioMappingView::set_gain_from_menu, this, db_to_linear(-6)), ID_minus6dB);
+       Bind (wxEVT_MENU, boost::bind(&AudioMappingView::set_gain_from_menu, this, 1), ID_0dB);
+       Bind (wxEVT_MENU, boost::bind(&AudioMappingView::set_gain_from_menu, this, db_to_linear(3)), ID_plus3dB);
        Bind (wxEVT_MENU, boost::bind(&AudioMappingView::edit, this), ID_edit);
        Bind (wxEVT_MOUSEWHEEL, boost::bind(&AudioMappingView::mouse_wheel, this, _1));
        _body->Bind (wxEVT_PAINT, boost::bind(&AudioMappingView::paint, this));
@@ -156,9 +162,8 @@ AudioMappingView::scroll ()
 }
 
 void
-AudioMappingView::paint_static (wxDC& dc, wxGraphicsContext* gc)
+AudioMappingView::paint_static (wxDC& dc)
 {
-       gc->SetAntialiasMode (wxANTIALIAS_DEFAULT);
        dc.SetFont (wxSWISS_FONT->Bold());
        wxCoord label_width;
        wxCoord label_height;
@@ -175,11 +180,10 @@ AudioMappingView::paint_static (wxDC& dc, wxGraphicsContext* gc)
                );
 
        dc.SetFont (*wxSWISS_FONT);
-       gc->SetPen (*wxBLACK_PEN);
 }
 
 void
-AudioMappingView::paint_column_labels (wxDC& dc, wxGraphicsContext* gc)
+AudioMappingView::paint_column_labels (wxDC& dc)
 {
        wxCoord label_width;
        wxCoord label_height;
@@ -190,33 +194,31 @@ AudioMappingView::paint_column_labels (wxDC& dc, wxGraphicsContext* gc)
                ++N;
        }
 
-       wxGraphicsPath lines = gc->CreatePath ();
-       lines.MoveToPoint (LEFT_WIDTH, GRID_SPACING);
-       lines.AddLineToPoint (LEFT_WIDTH + _output_channels.size() * GRID_SPACING, GRID_SPACING);
-       lines.MoveToPoint (LEFT_WIDTH, GRID_SPACING * 2);
-       lines.AddLineToPoint (LEFT_WIDTH + _output_channels.size() * GRID_SPACING, GRID_SPACING * 2);
-       gc->StrokePath (lines);
+       dc.DrawLine(wxPoint(LEFT_WIDTH, GRID_SPACING), wxPoint(LEFT_WIDTH + _output_channels.size() * GRID_SPACING, GRID_SPACING));
+       dc.DrawLine(wxPoint(LEFT_WIDTH, GRID_SPACING * 2), wxPoint(LEFT_WIDTH + _output_channels.size() * GRID_SPACING, GRID_SPACING * 2));
 }
 
 void
-AudioMappingView::paint_column_lines (wxGraphicsContext* gc)
+AudioMappingView::paint_column_lines (wxDC& dc)
 {
-       wxGraphicsPath lines = gc->CreatePath ();
        for (size_t i = 0; i < _output_channels.size(); ++i) {
-               lines.MoveToPoint    (LEFT_WIDTH + GRID_SPACING * i, GRID_SPACING);
-               lines.AddLineToPoint (LEFT_WIDTH + GRID_SPACING * i, TOP_HEIGHT + _input_channels.size() * GRID_SPACING);
+               dc.DrawLine (
+                       wxPoint(LEFT_WIDTH + GRID_SPACING * i, GRID_SPACING),
+                       wxPoint(LEFT_WIDTH + GRID_SPACING * i, TOP_HEIGHT + _input_channels.size() * GRID_SPACING)
+                       );
        }
-       lines.MoveToPoint    (LEFT_WIDTH + GRID_SPACING * _output_channels.size(), GRID_SPACING);
-       lines.AddLineToPoint (LEFT_WIDTH + GRID_SPACING * _output_channels.size(), TOP_HEIGHT + _input_channels.size() * GRID_SPACING);
-       gc->StrokePath (lines);
+
+       dc.DrawLine (
+               wxPoint(LEFT_WIDTH + GRID_SPACING * _output_channels.size(), GRID_SPACING),
+               wxPoint(LEFT_WIDTH + GRID_SPACING * _output_channels.size(), TOP_HEIGHT + _input_channels.size() * GRID_SPACING)
+               );
 }
 
 void
-AudioMappingView::paint_row_labels (wxDC& dc, wxGraphicsContext* gc)
+AudioMappingView::paint_row_labels (wxDC& dc)
 {
        wxCoord label_width;
        wxCoord label_height;
-       wxGraphicsPath lines = gc->CreatePath ();
 
        /* Row channel labels */
 
@@ -230,8 +232,10 @@ AudioMappingView::paint_row_labels (wxDC& dc, wxGraphicsContext* gc)
        /* Vertical lines on the left */
 
        for (int i = 1; i < 3; ++i) {
-               lines.MoveToPoint    (GRID_SPACING * i, TOP_HEIGHT);
-               lines.AddLineToPoint (GRID_SPACING * i, TOP_HEIGHT + _input_channels.size() * GRID_SPACING);
+               dc.DrawLine (
+                       wxPoint(GRID_SPACING * i, TOP_HEIGHT),
+                       wxPoint(GRID_SPACING * i, TOP_HEIGHT + _input_channels.size() * GRID_SPACING)
+                       );
        }
 
        /* Group labels and lines */
@@ -250,44 +254,34 @@ AudioMappingView::paint_row_labels (wxDC& dc, wxGraphicsContext* gc)
                                yp += dc.GetLogicalOrigin().y;
                        }
 
-                       wxCoord old_x, old_y, old_width, old_height;
-                       dc.GetClippingBox (&old_x, &old_y, &old_width, &old_height);
-                       dc.DestroyClippingRegion ();
-                       dc.SetClippingRegion (GRID_SPACING, yp + 4, GRID_SPACING, height - 8);
-
                        dc.DrawRotatedText (
                                std_to_wx(i.name),
                                GRID_SPACING + (GRID_SPACING - label_height) / 2,
                                y + (height + label_width) / 2,
                                90
                                );
-
-                       dc.DestroyClippingRegion ();
-                       dc.SetClippingRegion (old_x, old_y, old_width, old_height);
                }
 
-               lines.MoveToPoint    (GRID_SPACING,     y);
-               lines.AddLineToPoint (GRID_SPACING * 2, y);
+               dc.DrawLine (wxPoint(GRID_SPACING, y), wxPoint(GRID_SPACING * 2, y));
                y += height;
        }
 
-       lines.MoveToPoint    (GRID_SPACING,     y);
-       lines.AddLineToPoint (GRID_SPACING * 2, y);
-
-       gc->StrokePath (lines);
-}
+       dc.DrawLine (wxPoint(GRID_SPACING, y), wxPoint(GRID_SPACING * 2, y));
+       }
 
 void
-AudioMappingView::paint_row_lines (wxGraphicsContext* gc)
+AudioMappingView::paint_row_lines (wxDC& dc)
 {
-       wxGraphicsPath lines = gc->CreatePath ();
        for (size_t i = 0; i < _input_channels.size(); ++i) {
-               lines.MoveToPoint (GRID_SPACING * 2, TOP_HEIGHT + GRID_SPACING * i);
-               lines.AddLineToPoint (LEFT_WIDTH + _output_channels.size() * GRID_SPACING, TOP_HEIGHT + GRID_SPACING * i);
+               dc.DrawLine (
+                       wxPoint(GRID_SPACING * 2, TOP_HEIGHT + GRID_SPACING * i),
+                       wxPoint(LEFT_WIDTH + _output_channels.size() * GRID_SPACING, TOP_HEIGHT + GRID_SPACING * i)
+                       );
        }
-       lines.MoveToPoint (GRID_SPACING * 2, TOP_HEIGHT + GRID_SPACING * _input_channels.size());
-       lines.AddLineToPoint (LEFT_WIDTH + _output_channels.size() * GRID_SPACING, TOP_HEIGHT + GRID_SPACING * _input_channels.size());
-       gc->StrokePath (lines);
+       dc.DrawLine (
+               wxPoint(GRID_SPACING * 2, TOP_HEIGHT + GRID_SPACING * _input_channels.size()),
+               wxPoint(LEFT_WIDTH + _output_channels.size() * GRID_SPACING, TOP_HEIGHT + GRID_SPACING * _input_channels.size())
+               );
 }
 
 void
@@ -309,13 +303,14 @@ AudioMappingView::paint_indicators (wxDC& dc)
                                );
 
                        float const value_dB = linear_to_db(_map.get(y, x));
+                       wxColour const colour = value_dB <= 0 ? wxColour(0, 255, 0) : wxColour(255, 150, 0);
                        int const range = 18;
                        int height = 0;
                        if (value_dB > -range) {
-                               height = INDICATOR_SIZE * (1 + value_dB / range);
+                               height = min(INDICATOR_SIZE, static_cast<int>(INDICATOR_SIZE * (1 + value_dB / range)));
                        }
 
-                       dc.SetBrush (*wxTheBrushList->FindOrCreateBrush(wxColour (0, 255, 0), wxBRUSHSTYLE_SOLID));
+                       dc.SetBrush (*wxTheBrushList->FindOrCreateBrush(colour, wxBRUSHSTYLE_SOLID));
                        dc.DrawRectangle (
                                wxRect(
                                        LEFT_WIDTH + x * GRID_SPACING + (GRID_SPACING - INDICATOR_SIZE) / 2,
@@ -328,27 +323,22 @@ AudioMappingView::paint_indicators (wxDC& dc)
 }
 
 static
-void clip (wxDC& dc, wxGraphicsContext* gc, int x, int y, int w, int h)
+void clip (wxDC& dc, int x, int y, int w, int h)
 {
        dc.SetClippingRegion (x, y, w, h);
-       gc->Clip (x, y, w, h);
 }
 
 static
-void translate (wxDC& dc, wxGraphicsContext* gc, int x, int y)
+void translate (wxDC& dc, int x, int y)
 {
-       gc->PushState ();
-       gc->Translate (-x, -y);
        dc.SetLogicalOrigin (x, y);
 }
 
 static
-void restore (wxDC& dc, wxGraphicsContext* gc)
+void restore (wxDC& dc)
 {
        dc.SetLogicalOrigin (0, 0);
-       gc->PopState ();
        dc.DestroyClippingRegion ();
-       gc->ResetClip ();
 }
 
 void
@@ -356,42 +346,65 @@ AudioMappingView::paint ()
 {
        wxPaintDC dc (_body);
 
-       wxGraphicsContext* gc = wxGraphicsContext::Create (dc);
-       if (!gc) {
-               return;
-       }
-
        int const hs = _horizontal_scroll->GetThumbPosition ();
        int const vs = _vertical_scroll->GetThumbPosition ();
 
-       paint_static (dc, gc);
-
-       clip (dc, gc, LEFT_WIDTH, 0, GRID_SPACING * _output_channels.size(), GRID_SPACING * (2 + _input_channels.size()));
-       translate (dc, gc, hs, 0);
-       paint_column_labels (dc, gc);
-       restore (dc, gc);
-
-       clip (dc, gc, 0, TOP_HEIGHT, GRID_SPACING * (3 + _output_channels.size()), GRID_SPACING * _input_channels.size() + 1);
-       translate (dc, gc, 0, vs);
-       paint_row_labels (dc, gc);
-       restore (dc, gc);
-
-       clip (dc, gc, GRID_SPACING * 2, TOP_HEIGHT, GRID_SPACING * (1 + _output_channels.size()), GRID_SPACING * _input_channels.size() + 1);
-       translate (dc, gc, hs, vs);
-       paint_row_lines (gc);
-       restore (dc, gc);
-
-       clip (dc, gc, LEFT_WIDTH, GRID_SPACING, GRID_SPACING * (1 + _output_channels.size()), GRID_SPACING * (1 + _input_channels.size()));
-       translate (dc, gc, hs, vs);
-       paint_column_lines (gc);
-       restore (dc, gc);
-
-       clip (dc, gc, LEFT_WIDTH, TOP_HEIGHT, GRID_SPACING * _output_channels.size(), GRID_SPACING * _input_channels.size());
-       translate (dc, gc, hs, vs);
+       paint_static (dc);
+
+       clip (
+               dc,
+               LEFT_WIDTH,
+               0,
+               GRID_SPACING * _output_channels.size(),
+               GRID_SPACING * (2 + _input_channels.size())
+            );
+       translate (dc, hs, 0);
+       paint_column_labels (dc);
+       restore (dc);
+
+       clip (
+               dc,
+               0,
+               TOP_HEIGHT,
+               GRID_SPACING * 3,
+               min(int(GRID_SPACING * _input_channels.size()), GetSize().GetHeight() - TOP_HEIGHT)
+            );
+       translate (dc, 0, vs);
+       paint_row_labels (dc);
+       restore (dc);
+
+       clip (
+               dc,
+               GRID_SPACING * 2,
+               TOP_HEIGHT,
+               GRID_SPACING * (1 + _output_channels.size()),
+               min(int(GRID_SPACING * (2 + _input_channels.size())), GetSize().GetHeight() - TOP_HEIGHT)
+            );
+       translate (dc, hs, vs);
+       paint_row_lines (dc);
+       restore (dc);
+
+       clip (
+               dc,
+               LEFT_WIDTH,
+               GRID_SPACING,
+               GRID_SPACING * (1 + _output_channels.size()),
+               min(int(GRID_SPACING * (1 + _input_channels.size())), GetSize().GetHeight() - GRID_SPACING)
+            );
+       translate (dc, hs, vs);
+       paint_column_lines (dc);
+       restore (dc);
+
+       clip (
+               dc,
+               LEFT_WIDTH,
+               TOP_HEIGHT,
+               GRID_SPACING * _output_channels.size(),
+               min(int(GRID_SPACING * _input_channels.size()), GetSize().GetHeight() - TOP_HEIGHT)
+            );
+       translate (dc, hs, vs);
        paint_indicators (dc);
-       restore (dc, gc);
-
-       delete gc;
+       restore (dc);
 }
 
 optional<pair<int, int> >