]> git.xonotic.org Git - xonotic/netradiant.git/blobdiff - radiant/dialog.cpp
Group import/export callbacks
[xonotic/netradiant.git] / radiant / dialog.cpp
index 2a6e4138fe548c639f4f93efd34d432d3859b31e..9de0a3332df2c4be780ac03e3a3f7b97705c59b4 100644 (file)
@@ -50,7 +50,7 @@
 ui::Entry DialogEntry_new(){
        auto entry = ui::Entry(ui::New);
        entry.show();
-       gtk_widget_set_size_request( entry , 64, -1 );
+       entry.dimensions(64, -1);
        return entry;
 }
 
@@ -83,14 +83,14 @@ ui::SpinButton DialogSpinner_new( double value, double lower, double upper, int
        }
        auto spin = ui::SpinButton( ui::Adjustment( value, lower, upper, step, 10, 0 ), step, digits );
        spin.show();
-       gtk_widget_set_size_request( spin , 64, -1 );
+       spin.dimensions(64, -1);
        return spin;
 }
 
 class DialogSpinnerRow
 {
 public:
-DialogSpinnerRow( ui::Widget row, GtkSpinButton* spin ) : m_row( row ), m_spin( spin ){
+DialogSpinnerRow( ui::Widget row, ui::SpinButton spin ) : m_row( row ), m_spin( spin ){
 }
 ui::Widget m_row;
 ui::SpinButton m_spin;
@@ -107,193 +107,206 @@ DialogSpinnerRow DialogSpinnerRow_new( const char* name, double value, double lo
 }
 
 
-
 template<
-       typename Type_,
-       typename Other_,
-       void( *Import ) ( Type_&, Other_ ),
-       void( *Export ) ( Type_&, const Callback1<Other_>& )
-       >
-class ImportExport
-{
+               typename Type_,
+               typename Other_ = Type_,
+               class T = impexp<Type_, Other_>
+>
+class ImportExport {
 public:
-typedef Type_ Type;
-typedef Other_ Other;
+       using Type = Type_;
+       using Other = Other_;
 
-typedef ReferenceCaller1<Type, Other, Import> ImportCaller;
-typedef ReferenceCaller1<Type, const Callback1<Other>&, Export> ExportCaller;
+       using ImportCaller = ReferenceCaller<Type, void(Other), T::Import>;
+       using ExportCaller = ReferenceCaller<Type, void(const Callback<void(Other)> &), T::Export>;
 };
 
-typedef ImportExport<bool, bool, BoolImport, BoolExport> BoolImportExport;
-typedef ImportExport<int, int, IntImport, IntExport> IntImportExport;
-typedef ImportExport<std::size_t, std::size_t, SizeImport, SizeExport> SizeImportExport;
-typedef ImportExport<float, float, FloatImport, FloatExport> FloatImportExport;
-typedef ImportExport<CopiedString, const char*, StringImport, StringExport> StringImportExport;
 
+using BoolImportExport = ImportExport<bool>;
 
+struct BoolToggle {
+       static void Import(GtkToggleButton &widget, bool value) {
+               gtk_toggle_button_set_active(&widget, value);
+       }
 
-void BoolToggleImport( GtkToggleButton& widget, bool value ){
-       gtk_toggle_button_set_active( &widget, value );
-}
-void BoolToggleExport( GtkToggleButton& widget, const BoolImportCallback& importCallback ){
-       importCallback( gtk_toggle_button_get_active( &widget ) != FALSE );
-}
-typedef ImportExport<GtkToggleButton, bool, BoolToggleImport, BoolToggleExport> BoolToggleImportExport;
+       static void Export(GtkToggleButton &widget, const ImportExportCallback<bool>::Import_t &importCallback) {
+               importCallback(gtk_toggle_button_get_active(&widget) != FALSE);
+       }
+};
 
+using BoolToggleImportExport = ImportExport<GtkToggleButton, bool, BoolToggle>;
 
-void IntRadioImport( GtkRadioButton& widget, int index ){
-       radio_button_set_active( ui::RadioButton(&widget), index );
-}
-void IntRadioExport( GtkRadioButton& widget, const IntImportCallback& importCallback ){
-       importCallback( radio_button_get_active( ui::RadioButton(&widget) ) );
-}
-typedef ImportExport<GtkRadioButton, int, IntRadioImport, IntRadioExport> IntRadioImportExport;
+using IntImportExport = ImportExport<int>;
 
-void TextEntryImport( GtkEntry& widget, const char* text ){
-       ui::Entry(&widget).text(text);
-}
-void TextEntryExport( GtkEntry& widget, const StringImportCallback& importCallback ){
-       importCallback( gtk_entry_get_text( &widget ) );
-}
-typedef ImportExport<GtkEntry, const char*, TextEntryImport, TextEntryExport> TextEntryImportExport;
+struct IntEntry {
+       static void Import(GtkEntry &widget, int value) {
+               entry_set_int(ui::Entry(&widget), value);
+       }
 
+       static void Export(GtkEntry &widget, const ImportExportCallback<int>::Import_t &importCallback) {
+               importCallback(atoi(gtk_entry_get_text(&widget)));
+       }
+};
 
-void IntEntryImport( GtkEntry& widget, int value ){
-       entry_set_int( ui::Entry(&widget), value );
-}
-void IntEntryExport( GtkEntry& widget, const IntImportCallback& importCallback ){
-       importCallback( atoi( gtk_entry_get_text( &widget ) ) );
-}
-typedef ImportExport<GtkEntry, int, IntEntryImport, IntEntryExport> IntEntryImportExport;
+using IntEntryImportExport = ImportExport<GtkEntry, int, IntEntry>;
 
+struct IntRadio {
+       static void Import(GtkRadioButton &widget, int index) {
+               radio_button_set_active(ui::RadioButton(&widget), index);
+       }
 
-void SizeEntryImport( GtkEntry& widget, std::size_t value ){
-       entry_set_int( ui::Entry(&widget), int(value) );
-}
-void SizeEntryExport( GtkEntry& widget, const SizeImportCallback& importCallback ){
-       int value = atoi( gtk_entry_get_text( &widget ) );
-       if ( value < 0 ) {
-               value = 0;
+       static void Export(GtkRadioButton &widget, const ImportExportCallback<int>::Import_t &importCallback) {
+               importCallback(radio_button_get_active(ui::RadioButton(&widget)));
        }
-       importCallback( value );
-}
-typedef ImportExport<GtkEntry, std::size_t, SizeEntryImport, SizeEntryExport> SizeEntryImportExport;
+};
 
+using IntRadioImportExport = ImportExport<GtkRadioButton, int, IntRadio>;
 
-void FloatEntryImport( GtkEntry& widget, float value ){
-       entry_set_float( ui::Entry(&widget), value );
-}
-void FloatEntryExport( GtkEntry& widget, const FloatImportCallback& importCallback ){
-       importCallback( (float)atof( gtk_entry_get_text( &widget ) ) );
-}
-typedef ImportExport<GtkEntry, float, FloatEntryImport, FloatEntryExport> FloatEntryImportExport;
+struct IntCombo {
+       static void Import(GtkComboBox &widget, int value) {
+               gtk_combo_box_set_active(&widget, value);
+       }
 
+       static void Export(GtkComboBox &widget, const ImportExportCallback<int>::Import_t &importCallback) {
+               importCallback(gtk_combo_box_get_active(&widget));
+       }
+};
 
-void FloatSpinnerImport( GtkSpinButton& widget, float value ){
-       gtk_spin_button_set_value( &widget, value );
-}
-void FloatSpinnerExport( GtkSpinButton& widget, const FloatImportCallback& importCallback ){
-       importCallback( float(gtk_spin_button_get_value( &widget ) ) );
-}
-typedef ImportExport<GtkSpinButton, float, FloatSpinnerImport, FloatSpinnerExport> FloatSpinnerImportExport;
+using IntComboImportExport = ImportExport<GtkComboBox, int, IntCombo>;
 
+struct IntAdjustment {
+       static void Import(GtkAdjustment &widget, int value) {
+               gtk_adjustment_set_value(&widget, value);
+       }
 
-void IntSpinnerImport( GtkSpinButton& widget, int value ){
-       gtk_spin_button_set_value( &widget, value );
-}
-void IntSpinnerExport( GtkSpinButton& widget, const IntImportCallback& importCallback ){
-       importCallback( gtk_spin_button_get_value_as_int( &widget ) );
-}
-typedef ImportExport<GtkSpinButton, int, IntSpinnerImport, IntSpinnerExport> IntSpinnerImportExport;
+       static void Export(GtkAdjustment &widget, const ImportExportCallback<int>::Import_t &importCallback) {
+               importCallback((int) gtk_adjustment_get_value(&widget));
+       }
+};
 
+using IntAdjustmentImportExport = ImportExport<GtkAdjustment, int, IntAdjustment>;
 
-void IntAdjustmentImport( GtkAdjustment& widget, int value ){
-       gtk_adjustment_set_value( &widget, value );
-}
-void IntAdjustmentExport( GtkAdjustment& widget, const IntImportCallback& importCallback ){
-       importCallback( (int)gtk_adjustment_get_value( &widget ) );
-}
-typedef ImportExport<GtkAdjustment, int, IntAdjustmentImport, IntAdjustmentExport> IntAdjustmentImportExport;
+struct IntSpinner {
+       static void Import(GtkSpinButton &widget, int value) {
+               gtk_spin_button_set_value(&widget, value);
+       }
 
+       static void Export(GtkSpinButton &widget, const ImportExportCallback<int>::Import_t &importCallback) {
+               importCallback(gtk_spin_button_get_value_as_int(&widget));
+       }
+};
 
-void IntComboImport( GtkComboBox& widget, int value ){
-       gtk_combo_box_set_active( &widget, value );
-}
-void IntComboExport( GtkComboBox& widget, const IntImportCallback& importCallback ){
-       importCallback( gtk_combo_box_get_active( &widget ) );
-}
-typedef ImportExport<GtkComboBox, int, IntComboImport, IntComboExport> IntComboImportExport;
+using IntSpinnerImportExport = ImportExport<GtkSpinButton, int, IntSpinner>;
 
+using StringImportExport = ImportExport<CopiedString, const char *>;
 
-template<typename FirstArgument>
-class CallbackDialogData : public DLG_DATA
-{
-public:
-typedef Callback1<FirstArgument> ImportCallback;
-typedef Callback1<const ImportCallback&> ExportCallback;
+struct TextEntry {
+       static void Import(GtkEntry &widget, const char *text) {
+               ui::Entry(&widget).text(text);
+       }
+
+       static void Export(GtkEntry &widget, const ImportExportCallback<const char *>::Import_t &importCallback) {
+               importCallback(gtk_entry_get_text(&widget));
+       }
+};
+
+using TextEntryImportExport = ImportExport<GtkEntry, const char *, TextEntry>;
+
+using SizeImportExport = ImportExport<std::size_t>;
+
+struct SizeEntry {
+       static void Import(GtkEntry &widget, std::size_t value) {
+               entry_set_int(ui::Entry(&widget), int(value));
+       }
+
+       static void Export(GtkEntry &widget, const ImportExportCallback<std::size_t>::Import_t &importCallback) {
+               int value = atoi(gtk_entry_get_text(&widget));
+               if (value < 0) {
+                       value = 0;
+               }
+               importCallback(value);
+       }
+};
+
+using SizeEntryImportExport = ImportExport<GtkEntry, std::size_t, SizeEntry>;
 
-private:
-ImportCallback m_importWidget;
-ExportCallback m_exportWidget;
-ImportCallback m_importViewer;
-ExportCallback m_exportViewer;
+using FloatImportExport = ImportExport<float>;
+
+struct FloatEntry {
+       static void Import(GtkEntry &widget, float value) {
+               entry_set_float(ui::Entry(&widget), value);
+       }
+
+       static void Export(GtkEntry &widget, const ImportExportCallback<float>::Import_t &importCallback) {
+               importCallback((float) atof(gtk_entry_get_text(&widget)));
+       }
+};
+
+using FloatEntryImportExport = ImportExport<GtkEntry, float, FloatEntry>;
+
+struct FloatSpinner {
+       static void Import(GtkSpinButton &widget, float value) {
+               gtk_spin_button_set_value(&widget, value);
+       }
+
+       static void Export(GtkSpinButton &widget, const ImportExportCallback<float>::Import_t &importCallback) {
+               importCallback(float(gtk_spin_button_get_value(&widget)));
+       }
+};
+
+using FloatSpinnerImportExport = ImportExport<GtkSpinButton, float, FloatSpinner>;
+
+
+
+template<typename T>
+class CallbackDialogData : public DLG_DATA {
+       ImportExportCallback<T> m_cbWidget;
+       ImportExportCallback<T> m_cbViewer;
 
 public:
-CallbackDialogData( const ImportCallback& importWidget, const ExportCallback& exportWidget, const ImportCallback& importViewer, const ExportCallback& exportViewer )
-       : m_importWidget( importWidget ), m_exportWidget( exportWidget ), m_importViewer( importViewer ), m_exportViewer( exportViewer ){
-}
-void release(){
-       delete this;
-}
-void importData() const {
-       m_exportViewer( m_importWidget );
-}
-void exportData() const {
-       m_exportWidget( m_importViewer );
-}
+       CallbackDialogData(const ImportExportCallback<T> &cbWidget, const ImportExportCallback<T> &cbViewer)
+                       : m_cbWidget(cbWidget), m_cbViewer(cbViewer) {
+       }
+
+       void release() {
+               delete this;
+       }
+
+       void importData() const {
+               m_cbViewer.Export(m_cbWidget.Import);
+       }
+
+       void exportData() const {
+               m_cbWidget.Export(m_cbViewer.Import);
+       }
 };
 
 template<typename Widget, typename Viewer>
-class AddData
-{
-DialogDataList& m_data;
-public:
-AddData( DialogDataList& data ) : m_data( data ){
-}
-void apply( typename Widget::Type& widget, typename Viewer::Type& viewer ) const {
-       m_data.push_back(
-               new CallbackDialogData<typename Widget::Other>(
-                       typename Widget::ImportCaller( widget ),
-                       typename Widget::ExportCaller( widget ),
-                       typename Viewer::ImportCaller( viewer ),
-                       typename Viewer::ExportCaller( viewer )
+void AddData(DialogDataList &data, typename Widget::Type &widget, typename Viewer::Type &viewer) {
+       data.push_back(
+                       new CallbackDialogData<typename Widget::Other>(
+                                       {typename Widget::ImportCaller(widget),
+                                        typename Widget::ExportCaller(widget)},
+                                       {typename Viewer::ImportCaller(viewer),
+                                        typename Viewer::ExportCaller(viewer)}
                        )
-               );
+       );
 }
-};
 
 template<typename Widget>
-class AddCustomData
-{
-DialogDataList& m_data;
-public:
-AddCustomData( DialogDataList& data ) : m_data( data ){
-}
-void apply(
-       typename Widget::Type& widget,
-       const Callback1<typename Widget::Other>& importViewer,
-       const Callback1<const Callback1<typename Widget::Other>&>& exportViewer
-       ) const {
-       m_data.push_back(
-               new CallbackDialogData<typename Widget::Other>(
-                       typename Widget::ImportCaller( widget ),
-                       typename Widget::ExportCaller( widget ),
-                       importViewer,
-                       exportViewer
+void AddCustomData(
+               DialogDataList &data,
+               typename Widget::Type &widget,
+               ImportExportCallback<typename Widget::Other> const &cbViewer
+) {
+       data.push_back(
+                       new CallbackDialogData<typename Widget::Other>(
+                                       {typename Widget::ImportCaller(widget),
+                                        typename Widget::ExportCaller(widget)},
+                                       cbViewer
                        )
-               );
+       );
 }
-};
 
 // =============================================================================
 // Dialog class
@@ -343,76 +356,76 @@ void Dialog::Destroy(){
 }
 
 
-void Dialog::AddBoolToggleData( GtkToggleButton& widget, const BoolImportCallback& importViewer, const BoolExportCallback& exportViewer ){
-       AddCustomData<BoolToggleImportExport>( m_data ).apply( widget, importViewer, exportViewer );
+void Dialog::AddBoolToggleData( GtkToggleButton& widget, ImportExportCallback<bool> const &cb ){
+       AddCustomData<BoolToggleImportExport>( m_data, widget, cb );
 }
 
-void Dialog::AddIntRadioData( GtkRadioButton& widget, const IntImportCallback& importViewer, const IntExportCallback& exportViewer ){
-       AddCustomData<IntRadioImportExport>( m_data ).apply( widget, importViewer, exportViewer );
+void Dialog::AddIntRadioData( GtkRadioButton& widget, ImportExportCallback<int> const &cb ){
+       AddCustomData<IntRadioImportExport>( m_data, widget, cb );
 }
 
-void Dialog::AddTextEntryData( GtkEntry& widget, const StringImportCallback& importViewer, const StringExportCallback& exportViewer ){
-       AddCustomData<TextEntryImportExport>( m_data ).apply( widget, importViewer, exportViewer );
+void Dialog::AddTextEntryData( GtkEntry& widget, ImportExportCallback<const char *> const &cb ){
+       AddCustomData<TextEntryImportExport>( m_data, widget, cb );
 }
 
-void Dialog::AddIntEntryData( GtkEntry& widget, const IntImportCallback& importViewer, const IntExportCallback& exportViewer ){
-       AddCustomData<IntEntryImportExport>( m_data ).apply( widget, importViewer, exportViewer );
+void Dialog::AddIntEntryData( GtkEntry& widget, ImportExportCallback<int> const &cb ){
+       AddCustomData<IntEntryImportExport>( m_data, widget, cb );
 }
 
-void Dialog::AddSizeEntryData( GtkEntry& widget, const SizeImportCallback& importViewer, const SizeExportCallback& exportViewer ){
-       AddCustomData<SizeEntryImportExport>( m_data ).apply( widget, importViewer, exportViewer );
+void Dialog::AddSizeEntryData( GtkEntry& widget, ImportExportCallback<std::size_t> const &cb ){
+       AddCustomData<SizeEntryImportExport>( m_data, widget, cb );
 }
 
-void Dialog::AddFloatEntryData( GtkEntry& widget, const FloatImportCallback& importViewer, const FloatExportCallback& exportViewer ){
-       AddCustomData<FloatEntryImportExport>( m_data ).apply( widget, importViewer, exportViewer );
+void Dialog::AddFloatEntryData( GtkEntry& widget, ImportExportCallback<float> const &cb ){
+       AddCustomData<FloatEntryImportExport>( m_data, widget, cb );
 }
 
-void Dialog::AddFloatSpinnerData( GtkSpinButton& widget, const FloatImportCallback& importViewer, const FloatExportCallback& exportViewer ){
-       AddCustomData<FloatSpinnerImportExport>( m_data ).apply( widget, importViewer, exportViewer );
+void Dialog::AddFloatSpinnerData( GtkSpinButton& widget, ImportExportCallback<float> const &cb ){
+       AddCustomData<FloatSpinnerImportExport>( m_data, widget, cb );
 }
 
-void Dialog::AddIntSpinnerData( GtkSpinButton& widget, const IntImportCallback& importViewer, const IntExportCallback& exportViewer ){
-       AddCustomData<IntSpinnerImportExport>( m_data ).apply( widget, importViewer, exportViewer );
+void Dialog::AddIntSpinnerData( GtkSpinButton& widget, ImportExportCallback<int> const &cb ){
+       AddCustomData<IntSpinnerImportExport>( m_data, widget, cb );
 }
 
-void Dialog::AddIntAdjustmentData( GtkAdjustment& widget, const IntImportCallback& importViewer, const IntExportCallback& exportViewer ){
-       AddCustomData<IntAdjustmentImportExport>( m_data ).apply( widget, importViewer, exportViewer );
+void Dialog::AddIntAdjustmentData( GtkAdjustment& widget, ImportExportCallback<int> const &cb ){
+       AddCustomData<IntAdjustmentImportExport>( m_data, widget, cb );
 }
 
-void Dialog::AddIntComboData( GtkComboBox& widget, const IntImportCallback& importViewer, const IntExportCallback& exportViewer ){
-       AddCustomData<IntComboImportExport>( m_data ).apply( widget, importViewer, exportViewer );
+void Dialog::AddIntComboData( GtkComboBox& widget, ImportExportCallback<int> const &cb ){
+       AddCustomData<IntComboImportExport>( m_data, widget, cb );
 }
 
 
 void Dialog::AddDialogData( GtkToggleButton& widget, bool& data ){
-       AddData<BoolToggleImportExport, BoolImportExport>( m_data ).apply( widget, data );
+       AddData<BoolToggleImportExport, BoolImportExport>( m_data, widget, data );
 }
 void Dialog::AddDialogData( GtkRadioButton& widget, int& data ){
-       AddData<IntRadioImportExport, IntImportExport>( m_data ).apply( widget, data );
+       AddData<IntRadioImportExport, IntImportExport>( m_data, widget, data );
 }
 void Dialog::AddDialogData( GtkEntry& widget, CopiedString& data ){
-       AddData<TextEntryImportExport, StringImportExport>( m_data ).apply( widget, data );
+       AddData<TextEntryImportExport, StringImportExport>( m_data, widget, data );
 }
 void Dialog::AddDialogData( GtkEntry& widget, int& data ){
-       AddData<IntEntryImportExport, IntImportExport>( m_data ).apply( widget, data );
+       AddData<IntEntryImportExport, IntImportExport>( m_data, widget, data );
 }
 void Dialog::AddDialogData( GtkEntry& widget, std::size_t& data ){
-       AddData<SizeEntryImportExport, SizeImportExport>( m_data ).apply( widget, data );
+       AddData<SizeEntryImportExport, SizeImportExport>( m_data, widget, data );
 }
 void Dialog::AddDialogData( GtkEntry& widget, float& data ){
-       AddData<FloatEntryImportExport, FloatImportExport>( m_data ).apply( widget, data );
+       AddData<FloatEntryImportExport, FloatImportExport>( m_data, widget, data );
 }
 void Dialog::AddDialogData( GtkSpinButton& widget, float& data ){
-       AddData<FloatSpinnerImportExport, FloatImportExport>( m_data ).apply( widget, data );
+       AddData<FloatSpinnerImportExport, FloatImportExport>( m_data, widget, data );
 }
 void Dialog::AddDialogData( GtkSpinButton& widget, int& data ){
-       AddData<IntSpinnerImportExport, IntImportExport>( m_data ).apply( widget, data );
+       AddData<IntSpinnerImportExport, IntImportExport>( m_data, widget, data );
 }
 void Dialog::AddDialogData( GtkAdjustment& widget, int& data ){
-       AddData<IntAdjustmentImportExport, IntImportExport>( m_data ).apply( widget, data );
+       AddData<IntAdjustmentImportExport, IntImportExport>( m_data, widget, data );
 }
 void Dialog::AddDialogData( GtkComboBox& widget, int& data ){
-       AddData<IntComboImportExport, IntImportExport>( m_data ).apply( widget, data );
+       AddData<IntComboImportExport, IntImportExport>( m_data, widget, data );
 }
 
 void Dialog::exportData(){
@@ -453,20 +466,20 @@ EMessageBoxReturn Dialog::DoModal(){
 }
 
 
-ui::CheckButton Dialog::addCheckBox( ui::VBox vbox, const char* name, const char* flag, const BoolImportCallback& importViewer, const BoolExportCallback& exportViewer ){
+ui::CheckButton Dialog::addCheckBox( ui::VBox vbox, const char* name, const char* flag, ImportExportCallback<bool> const &cb ){
        auto check = ui::CheckButton( flag );
        check.show();
-       AddBoolToggleData( *GTK_TOGGLE_BUTTON( check ), importViewer, exportViewer );
+       AddBoolToggleData( *GTK_TOGGLE_BUTTON( check ), cb );
 
        DialogVBox_packRow( vbox, ui::Widget(DialogRow_new( name, check  ) ));
        return check;
 }
 
 ui::CheckButton Dialog::addCheckBox( ui::VBox vbox, const char* name, const char* flag, bool& data ){
-       return addCheckBox( vbox, name, flag, BoolImportCaller( data ), BoolExportCaller( data ) );
+       return addCheckBox(vbox, name, flag, mkImportExportCallback(data));
 }
 
-void Dialog::addCombo( ui::VBox vbox, const char* name, StringArrayRange values, const IntImportCallback& importViewer, const IntExportCallback& exportViewer ){
+void Dialog::addCombo( ui::VBox vbox, const char* name, StringArrayRange values, ImportExportCallback<int> const &cb ){
        auto alignment = ui::Alignment( 0.0, 0.5, 0.0, 0.0 );
        alignment.show();
        {
@@ -477,7 +490,7 @@ void Dialog::addCombo( ui::VBox vbox, const char* name, StringArrayRange values,
                        gtk_combo_box_text_append_text( GTK_COMBO_BOX_TEXT( combo ), *i );
                }
 
-               AddIntComboData( *GTK_COMBO_BOX( combo ), importViewer, exportViewer );
+               AddIntComboData( *GTK_COMBO_BOX( combo ), cb );
 
                combo.show();
                alignment.add(combo);
@@ -488,7 +501,7 @@ void Dialog::addCombo( ui::VBox vbox, const char* name, StringArrayRange values,
 }
 
 void Dialog::addCombo( ui::VBox vbox, const char* name, int& data, StringArrayRange values ){
-       addCombo( vbox, name, values, IntImportCaller( data ), IntExportCaller( data ) );
+       addCombo(vbox, name, values, mkImportExportCallback(data));
 }
 
 void Dialog::addSlider( ui::VBox vbox, const char* name, int& data, gboolean draw_value, const char* low, const char* high, double value, double lower, double upper, double step_increment, double page_increment ){
@@ -512,7 +525,7 @@ void Dialog::addSlider( ui::VBox vbox, const char* name, int& data, gboolean dra
 
        // adjustment
        auto adj = ui::Adjustment( value, lower, upper, step_increment, page_increment, 0 );
-       AddIntAdjustmentData( *GTK_ADJUSTMENT(adj), IntImportCaller( data ), IntExportCaller( data ) );
+       AddIntAdjustmentData(*GTK_ADJUSTMENT(adj), mkImportExportCallback(data));
 
        // scale
        auto alignment = ui::Alignment( 0.0, 0.5, 1.0, 0.0 );
@@ -530,13 +543,13 @@ void Dialog::addSlider( ui::VBox vbox, const char* name, int& data, gboolean dra
        DialogVBox_packRow( vbox, row );
 }
 
-void Dialog::addRadio( ui::VBox vbox, const char* name, StringArrayRange names, const IntImportCallback& importViewer, const IntExportCallback& exportViewer ){
+void Dialog::addRadio( ui::VBox vbox, const char* name, StringArrayRange names, ImportExportCallback<int> const &cb ){
        auto alignment = ui::Alignment( 0.0, 0.5, 0.0, 0.0 );
        alignment.show();;
        {
                RadioHBox radioBox = RadioHBox_new( names );
                alignment.add(radioBox.m_hbox);
-               AddIntRadioData( *GTK_RADIO_BUTTON( radioBox.m_radio ), importViewer, exportViewer );
+               AddIntRadioData( *GTK_RADIO_BUTTON( radioBox.m_radio ), cb );
        }
 
        auto row = DialogRow_new( name, alignment );
@@ -544,15 +557,15 @@ void Dialog::addRadio( ui::VBox vbox, const char* name, StringArrayRange names,
 }
 
 void Dialog::addRadio( ui::VBox vbox, const char* name, int& data, StringArrayRange names ){
-       addRadio( vbox, name, names, IntImportCaller( data ), IntExportCaller( data ) );
+       addRadio(vbox, name, names, mkImportExportCallback(data));
 }
 
-void Dialog::addRadioIcons( ui::VBox vbox, const char* name, StringArrayRange icons, const IntImportCallback& importViewer, const IntExportCallback& exportViewer ){
-       ui::Widget table = ui::Table( 2, icons.last - icons.first, FALSE );
-       table.show();;
+void Dialog::addRadioIcons( ui::VBox vbox, const char* name, StringArrayRange icons, ImportExportCallback<int> const &cb ){
+    auto table = ui::Table(2, icons.last - icons.first, FALSE);
+    table.show();
 
-       gtk_table_set_row_spacings( GTK_TABLE( table ), 5 );
-       gtk_table_set_col_spacings( GTK_TABLE( table ), 5 );
+    gtk_table_set_row_spacings(table, 5);
+    gtk_table_set_col_spacings(table, 5);
 
        GSList* group = 0;
        ui::RadioButton radio{ui::null};
@@ -561,79 +574,75 @@ void Dialog::addRadioIcons( ui::VBox vbox, const char* name, StringArrayRange ic
                guint pos = static_cast<guint>( icon - icons.first );
                auto image = new_local_image( *icon );
                image.show();
-               gtk_table_attach( GTK_TABLE( table ), image , pos, pos + 1, 0, 1,
-                                                 (GtkAttachOptions) ( 0 ),
-                                                 (GtkAttachOptions) ( 0 ), 0, 0 );
+        table.attach(image, {pos, pos + 1, 0, 1}, {0, 0});
 
                radio = ui::RadioButton(GTK_RADIO_BUTTON(gtk_radio_button_new( group )));
                radio.show();
-               gtk_table_attach( GTK_TABLE( table ), radio, pos, pos + 1, 1, 2,
-                                                 (GtkAttachOptions) ( 0 ),
-                                                 (GtkAttachOptions) ( 0 ), 0, 0 );
+        table.attach(radio, {pos, pos + 1, 1, 2}, {0, 0});
 
                group = gtk_radio_button_get_group( GTK_RADIO_BUTTON( radio ) );
        }
 
-       AddIntRadioData( *GTK_RADIO_BUTTON( radio ), importViewer, exportViewer );
+       AddIntRadioData( *GTK_RADIO_BUTTON( radio ), cb );
 
        DialogVBox_packRow( vbox, DialogRow_new( name, table ) );
 }
 
 void Dialog::addRadioIcons( ui::VBox vbox, const char* name, int& data, StringArrayRange icons ){
-       addRadioIcons( vbox, name, icons, IntImportCaller( data ), IntExportCaller( data ) );
+       addRadioIcons(vbox, name, icons, mkImportExportCallback(data));
 }
 
-ui::Widget Dialog::addIntEntry( ui::VBox vbox, const char* name, const IntImportCallback& importViewer, const IntExportCallback& exportViewer ){
+ui::Widget Dialog::addIntEntry( ui::VBox vbox, const char* name, ImportExportCallback<int> const &cb ){
        DialogEntryRow row( DialogEntryRow_new( name ) );
-       AddIntEntryData( *GTK_ENTRY(row.m_entry), importViewer, exportViewer );
+       AddIntEntryData( *GTK_ENTRY(row.m_entry), cb );
        DialogVBox_packRow( vbox, row.m_row );
        return row.m_row;
 }
 
-ui::Widget Dialog::addSizeEntry( ui::VBox vbox, const char* name, const SizeImportCallback& importViewer, const SizeExportCallback& exportViewer ){
+ui::Widget Dialog::addSizeEntry( ui::VBox vbox, const char* name, ImportExportCallback<std::size_t> const &cb ){
        DialogEntryRow row( DialogEntryRow_new( name ) );
-       AddSizeEntryData( *GTK_ENTRY(row.m_entry), importViewer, exportViewer );
+       AddSizeEntryData( *GTK_ENTRY(row.m_entry), cb );
        DialogVBox_packRow( vbox, row.m_row );
        return row.m_row;
 }
 
-ui::Widget Dialog::addFloatEntry( ui::VBox vbox, const char* name, const FloatImportCallback& importViewer, const FloatExportCallback& exportViewer ){
+ui::Widget Dialog::addFloatEntry( ui::VBox vbox, const char* name, ImportExportCallback<float> const &cb ){
        DialogEntryRow row( DialogEntryRow_new( name ) );
-       AddFloatEntryData( *GTK_ENTRY(row.m_entry), importViewer, exportViewer );
+       AddFloatEntryData( *GTK_ENTRY(row.m_entry), cb );
        DialogVBox_packRow( vbox, row.m_row );
        return row.m_row;
 }
 
-ui::Widget Dialog::addPathEntry( ui::VBox vbox, const char* name, bool browse_directory, const StringImportCallback& importViewer, const StringExportCallback& exportViewer ){
+ui::Widget Dialog::addPathEntry( ui::VBox vbox, const char* name, bool browse_directory, ImportExportCallback<const char *> const &cb ){
        PathEntry pathEntry = PathEntry_new();
        pathEntry.m_button.connect( "clicked", G_CALLBACK( browse_directory ? button_clicked_entry_browse_directory : button_clicked_entry_browse_file ), pathEntry.m_entry );
 
-       AddTextEntryData( *GTK_ENTRY(pathEntry.m_entry), importViewer, exportViewer );
+       AddTextEntryData( *GTK_ENTRY(pathEntry.m_entry), cb );
 
        auto row = DialogRow_new( name, ui::Widget(pathEntry.m_frame ) );
        DialogVBox_packRow( vbox, row );
 
-       return ui::Widget(row );
+       return row;
 }
 
 ui::Widget Dialog::addPathEntry( ui::VBox vbox, const char* name, CopiedString& data, bool browse_directory ){
-       return addPathEntry( vbox, name, browse_directory, StringImportCallback( StringImportCaller( data ) ), StringExportCallback( StringExportCaller( data ) ) );
+    return addPathEntry(vbox, name, browse_directory, mkImportExportCallback<CopiedString, const char *>(data));
 }
 
-ui::SpinButton Dialog::addSpinner( ui::VBox vbox, const char* name, double value, double lower, double upper, const IntImportCallback& importViewer, const IntExportCallback& exportViewer ){
+ui::SpinButton Dialog::addSpinner( ui::VBox vbox, const char* name, double value, double lower, double upper, ImportExportCallback<int> const &cb ){
        DialogSpinnerRow row( DialogSpinnerRow_new( name, value, lower, upper, 1 ) );
-       AddIntSpinnerData( *GTK_SPIN_BUTTON(row.m_spin), importViewer, exportViewer );
+       AddIntSpinnerData( *GTK_SPIN_BUTTON(row.m_spin), cb );
        DialogVBox_packRow( vbox, row.m_row );
-       return ui::SpinButton(row.m_spin);
+       return row.m_spin;
 }
 
 ui::SpinButton Dialog::addSpinner( ui::VBox vbox, const char* name, int& data, double value, double lower, double upper ){
-       return addSpinner( vbox, name, value, lower, upper, IntImportCallback( IntImportCaller( data ) ), IntExportCallback( IntExportCaller( data ) ) );
+       return addSpinner(vbox, name, value, lower, upper, mkImportExportCallback(data));
 }
 
-ui::SpinButton Dialog::addSpinner( ui::VBox vbox, const char* name, double value, double lower, double upper, const FloatImportCallback& importViewer, const FloatExportCallback& exportViewer ){
+ui::SpinButton Dialog::addSpinner( ui::VBox vbox, const char* name, double value, double lower, double upper, ImportExportCallback<float> const &cb ){
        DialogSpinnerRow row( DialogSpinnerRow_new( name, value, lower, upper, 10 ) );
-       AddFloatSpinnerData( *GTK_SPIN_BUTTON(row.m_spin), importViewer, exportViewer );
+       AddFloatSpinnerData( *GTK_SPIN_BUTTON(row.m_spin), cb );
        DialogVBox_packRow( vbox, row.m_row );
-       return ui::SpinButton(row.m_spin);
+       return row.m_spin;
 }