Module: synfig
Branch: nikitakit_master
Commit: 6ac9d50b4178c3a836175d04173f7bee93a3dac7
URL:    
http://synfig.git.sourceforge.net/git/gitweb.cgi?p=synfig;a=commit;h=6ac9d50b4178c3a836175d04173f7bee93a3dac7

Author: Nikita Kitaev <[email protected]>
Date:   Sat Apr 10 19:57:18 2010 -0700

Cleanup dead code

Also rename gradient dialog button from "grab" to "set_default"

---

 synfig-studio/src/gtkmm/app.cpp                |   58 +-----------------------
 synfig-studio/src/gtkmm/autorecover.cpp        |    1 -
 synfig-studio/src/gtkmm/cellrenderer_value.cpp |    2 +-
 synfig-studio/src/gtkmm/childrentree.cpp       |    5 ++-
 synfig-studio/src/gtkmm/childrentree.h         |    1 +
 synfig-studio/src/gtkmm/dialog_gradient.cpp    |   26 ++---------
 synfig-studio/src/gtkmm/dialog_gradient.h      |   10 ++---
 synfig-studio/src/gtkmm/widget_defaults.cpp    |    2 +-
 8 files changed, 18 insertions(+), 87 deletions(-)

diff --git a/synfig-studio/src/gtkmm/app.cpp b/synfig-studio/src/gtkmm/app.cpp
index 874ee35..95452d3 100644
--- a/synfig-studio/src/gtkmm/app.cpp
+++ b/synfig-studio/src/gtkmm/app.cpp
@@ -74,7 +74,6 @@
 #include "dialog_gradient.h"
 #include "dialog_color.h"
 #include "toolbox.h"
-#include "compview.h"
 #include "onemoment.h"
 
 #include "dockmanager.h"
@@ -1430,14 +1429,8 @@ App::~App()
 
        toolbox->hide();
 
-//     studio::App::iteration(false);
-
        delete toolbox;
 
-//     studio::App::iteration(false);
-
-//     studio::App::iteration(false);
-
        delete dialog_setup;
 
        delete dialog_gradient;
@@ -1449,8 +1442,6 @@ App::~App()
        delete dock_manager;
 
        instance_list.clear();
-
-//     studio::App::iteration(false);
 }
 
 String
@@ -1958,7 +1949,7 @@ App::dialog_open_file(const std::string &title, 
std::string &filename, std::stri
        }
        return false;
 
-#else
+#else   // not USE_WIN32_FILE_DIALOGS
        synfig::String prev_path;
 
        if(!_preferences.get_value(preference, prev_path))
@@ -1989,52 +1980,7 @@ App::dialog_open_file(const std::string &title, 
std::string &filename, std::stri
 
     delete dialog;
     return false;
-
-    /*
-
-       GtkWidget *ok;
-       GtkWidget *cancel;
-       int val=0;
-
-       GtkWidget *fileselection;
-       fileselection = gtk_file_selection_new(title.c_str());
-
-
-       if(basename(filename)==filename)
-       {
-               
gtk_file_selection_set_filename(GTK_FILE_SELECTION(fileselection),(prev_path+ETL_DIRECTORY_SEPARATOR).c_str());
-       }
-       else
-               
gtk_file_selection_set_filename(GTK_FILE_SELECTION(fileselection),dirname(filename).c_str());
-
-       
gtk_file_selection_complete(GTK_FILE_SELECTION(fileselection),basename(filename).c_str());
-
-       ok=GTK_FILE_SELECTION(fileselection)->ok_button;
-       cancel=GTK_FILE_SELECTION(fileselection)->cancel_button;
-
-       
gtk_signal_connect(GTK_OBJECT(ok),"clicked",GTK_SIGNAL_FUNC(Signal_Open_Ok),&val);
-       
gtk_signal_connect(GTK_OBJECT(cancel),"clicked",GTK_SIGNAL_FUNC(Signal_Open_Cancel),&val);
-
-       gtk_widget_show(fileselection);
-
-       while(!val)
-               iteration();
-
-
-       if(val==1)
-       {
-               
filename=gtk_file_selection_get_filename(GTK_FILE_SELECTION(fileselection));
-               _preferences.set_value(preference,dirname(filename));
-       }
-       else
-       {
-               gtk_widget_destroy(fileselection);
-               return false;
-       }
-       gtk_widget_destroy(fileselection);
-       return true;
-    */
-#endif
+#endif   // not USE_WIN32_FILE_DIALOGS
 }
 
 bool
diff --git a/synfig-studio/src/gtkmm/autorecover.cpp 
b/synfig-studio/src/gtkmm/autorecover.cpp
index 1b55eb4..4e9eeb8 100644
--- a/synfig-studio/src/gtkmm/autorecover.cpp
+++ b/synfig-studio/src/gtkmm/autorecover.cpp
@@ -126,7 +126,6 @@ AutoRecover::set_timeout(int milliseconds)
        auto_backup_connect.disconnect();
        if(timeout)
                
auto_backup_connect=Glib::signal_timeout().connect(sigc::ptr_fun(&AutoRecover::auto_backup),timeout);
-//             
auto_backup_connect=App::main.get_context()->signal_timeout().connect(sigc::mem_fun(&AutoRecover::auto_backup),timeout);
 }
 
 synfig::String
diff --git a/synfig-studio/src/gtkmm/cellrenderer_value.cpp 
b/synfig-studio/src/gtkmm/cellrenderer_value.cpp
index 456dee7..f0403fd 100644
--- a/synfig-studio/src/gtkmm/cellrenderer_value.cpp
+++ b/synfig-studio/src/gtkmm/cellrenderer_value.cpp
@@ -534,7 +534,7 @@ CellRenderer_ValueBase::start_editing_vfunc(
                                path
                        )
                );
-               App::dialog_gradient->grab_button_set_sensitive(true);
+               App::dialog_gradient->set_default_button_set_sensitive(true);
                App::dialog_gradient->present();
 
                return NULL;
diff --git a/synfig-studio/src/gtkmm/childrentree.cpp 
b/synfig-studio/src/gtkmm/childrentree.cpp
index aa47850..01ef95e 100644
--- a/synfig-studio/src/gtkmm/childrentree.cpp
+++ b/synfig-studio/src/gtkmm/childrentree.cpp
@@ -180,7 +180,10 @@ ChildrenTree::ChildrenTree()
        tree_view.set_enable_search(true);
        tree_view.set_search_column(model.label);
 
-/*
+/*  // Buttons to raise/lower/duplicate/delete children valuenodes
+       //   Commented out because these functions are not implemented
+    //   and children sort themselves alphabetically
+
        Gtk::Image *icon;
        //Gtk::IconSize iconsize(Gtk::IconSize::from_name("synfig-small_icon"));
        Gtk::IconSize iconsize(Gtk::ICON_SIZE_SMALL_TOOLBAR);
diff --git a/synfig-studio/src/gtkmm/childrentree.h 
b/synfig-studio/src/gtkmm/childrentree.h
index 34a216f..d307fd0 100644
--- a/synfig-studio/src/gtkmm/childrentree.h
+++ b/synfig-studio/src/gtkmm/childrentree.h
@@ -130,6 +130,7 @@ private:
 
        void on_dirty_preview();
 
+       //! \todo Implement raise/lower/duplicate/delete functions
        void on_raise_pressed();
 
        void on_lower_pressed();
diff --git a/synfig-studio/src/gtkmm/dialog_gradient.cpp 
b/synfig-studio/src/gtkmm/dialog_gradient.cpp
index 5a3edca..fe060b9 100644
--- a/synfig-studio/src/gtkmm/dialog_gradient.cpp
+++ b/synfig-studio/src/gtkmm/dialog_gradient.cpp
@@ -72,17 +72,10 @@ Dialog_Gradient::Dialog_Gradient():
        set_role("gradient_editor");
 
        // Setup the buttons
-       grab_button = manage(new class Gtk::Button(Gtk::StockID(_("Set as 
Default"))));
-       grab_button->show();
-       add_action_widget(*grab_button,2);
-       grab_button->signal_clicked().connect(sigc::mem_fun(*this, 
&Dialog_Gradient::on_grab_pressed));
-
-       /*
-       Gtk::Button *apply_button(manage(new class 
Gtk::Button(Gtk::StockID("gtk-apply"))));
-       apply_button->show();
-       add_action_widget(*apply_button,1);
-       apply_button->signal_clicked().connect(sigc::mem_fun(*this, 
&Dialog_Gradient::on_apply_pressed));
-       */
+       set_default_button = manage(new class Gtk::Button(Gtk::StockID(_("Set 
as Default"))));
+       set_default_button->show();
+       add_action_widget(*set_default_button,2);
+       set_default_button->signal_clicked().connect(sigc::mem_fun(*this, 
&Dialog_Gradient::on_set_default_pressed));
 
        Gtk::Button *cancel_button(manage(new class 
Gtk::Button(Gtk::StockID("gtk-close"))));
        cancel_button->show();
@@ -136,18 +129,9 @@ Dialog_Gradient::reset()
 }
 
 void
-Dialog_Gradient::on_grab_pressed()
+Dialog_Gradient::on_set_default_pressed()
 {
        synfigapp::Main::set_gradient(get_gradient());
-//     signal_edited_(get_gradient());
-//     reset();
-//     hide();
-}
-
-void
-Dialog_Gradient::on_apply_pressed()
-{
-       signal_edited_(get_gradient());
 }
 
 void
diff --git a/synfig-studio/src/gtkmm/dialog_gradient.h 
b/synfig-studio/src/gtkmm/dialog_gradient.h
index f9d674c..698ba3c 100644
--- a/synfig-studio/src/gtkmm/dialog_gradient.h
+++ b/synfig-studio/src/gtkmm/dialog_gradient.h
@@ -77,16 +77,14 @@ class Dialog_Gradient : public Gtk::Dialog
 
        sigc::connection value_changed_connection;
 
-       void on_ok_pressed();
-       void on_apply_pressed();
-       void on_grab_pressed();
+       void on_set_default_pressed();
 
        void on_cpoint_selected(synfig::Gradient::CPoint x);
        void on_values_adjusted();
 
        Widget_Gradient* widget_gradient;
        Widget_ColorEdit* widget_color;
-       Gtk::Button *grab_button;
+       Gtk::Button *set_default_button;
 
        void on_changed();
 
@@ -98,14 +96,14 @@ public:
 
        const synfig::Gradient& get_gradient()const { return 
widget_gradient->get_value(); }
 
-       void grab_button_set_sensitive(bool sensitive) { 
grab_button->set_sensitive(sensitive); }
+       void set_default_button_set_sensitive(bool sensitive) { 
set_default_button->set_sensitive(sensitive); }
 
        void reset();
 
 
        Dialog_Gradient();
        ~Dialog_Gradient();
-       //! Inerface to external calls to fill in the Gradient Editor Dialog
+       //! Interface to external calls to fill in the Gradient Editor Dialog
        //! when a Constant ValueNode or a Animated ValueNode is double cliked.
        void edit(const synfigapp::ValueDesc &x, 
etl::handle<synfigapp::CanvasInterface> canvas_interface, synfig::Time time=0);
 }; // END of Dialog_Gradient
diff --git a/synfig-studio/src/gtkmm/widget_defaults.cpp 
b/synfig-studio/src/gtkmm/widget_defaults.cpp
index e672b8f..20cabfe 100644
--- a/synfig-studio/src/gtkmm/widget_defaults.cpp
+++ b/synfig-studio/src/gtkmm/widget_defaults.cpp
@@ -464,7 +464,7 @@ Widget_Defaults::on_gradient_clicked()
        App::dialog_gradient->set_gradient(synfigapp::Main::get_gradient());
        App::dialog_gradient->reset();
        
App::dialog_gradient->signal_edited().connect(sigc::ptr_fun(synfigapp::Main::set_gradient));
-       App::dialog_gradient->grab_button_set_sensitive(false);
+       App::dialog_gradient->set_default_button_set_sensitive(false);
        App::dialog_gradient->present();
 }
 


------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Synfig-devl mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/synfig-devl

Reply via email to