A few missing checks on the return value of fopen_boost.

[dcpomatic.git] / src / wx / editable_list.h

diff --git a/src/wx/editable_list.h b/src/wx/editable_list.h
index a282bba695734ba88815ac86847afe6430184270..c6cb28a452e8df35fe339d5498a0987812163705 100644 (file)
--- a/src/wx/editable_list.h
+++ b/src/wx/editable_list.h
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-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
@@ -61,6 +61,8 @@ public:
                        wxSizer* s = new wxBoxSizer (wxVERTICAL);
                        _add = new wxButton (this, wxID_ANY, _("Add..."));
                        s->Add (_add, 0, wxTOP | wxBOTTOM, 2);
+                       _copy = new wxButton (this, wxID_ANY, _("Copy..."));
+                       s->Add (_copy, 0, wxTOP | wxBOTTOM, 2);
                        _edit = new wxButton (this, wxID_ANY, _("Edit..."));
                        s->Add (_edit, 0, wxTOP | wxBOTTOM, 2);
                        _remove = new wxButton (this, wxID_ANY, _("Remove"));
@@ -68,12 +70,8 @@ public:
                        table->Add (s, 0);
                }
 
-               std::vector<T> current = _get ();
-               for (typename std::vector<T>::iterator i = current.begin (); i != current.end(); ++i) {
-                       add_to_control (*i);
-               }
-
                _add->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&EditableList::add_clicked, this));
+               _copy->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&EditableList::copy_clicked, this));
                _edit->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&EditableList::edit_clicked, this));
                _remove->Bind (wxEVT_COMMAND_BUTTON_CLICKED, boost::bind (&EditableList::remove_clicked, this));
 
@@ -84,6 +82,16 @@ public:
 
        }
 
+       void refresh ()
+       {
+               _list->DeleteAllItems ();
+               
+               std::vector<T> current = _get ();
+               for (typename std::vector<T>::iterator i = current.begin (); i != current.end(); ++i) {
+                       add_to_control (*i);
+               }
+       }               
+
 private:       
 
        void add_to_control (T item)
@@ -121,6 +129,23 @@ private:
                dialog->Destroy ();
        }
 
+       void copy_clicked ()
+       {
+               int item = _list->GetNextItem (-1, wxLIST_NEXT_ALL, wxLIST_STATE_SELECTED);
+               if (item == -1) {
+                       return;
+               }
+
+               std::vector<T> all = _get ();
+               DCPOMATIC_ASSERT (item >= 0 && item < int (all.size ()));
+
+               T copy (all[item]);
+               add_to_control (copy);
+               
+               all.push_back (copy);
+               _set (all);
+       }
+
        void edit_clicked ()
        {
                int item = _list->GetNextItem (-1, wxLIST_NEXT_ALL, wxLIST_STATE_SELECTED);
@@ -129,7 +154,7 @@ private:
                }
 
                std::vector<T> all = _get ();
-               assert (item >= 0 && item < int (all.size ()));
+               DCPOMATIC_ASSERT (item >= 0 && item < int (all.size ()));
 
                S* dialog = new S (this);
                dialog->set (all[item]);
@@ -174,6 +199,7 @@ private:
        boost::function<std::string (T, int)> _column;
 
        wxButton* _add;
+       wxButton* _copy;
        wxButton* _edit;
        wxButton* _remove;
        wxListCtrl* _list;