From: Václav Slavík <vslavik@fastmail.fm>
Date: Sun, 28 Nov 2004 09:33:43 +0000 (+0000)
Subject: fixed to not duplicate code needlessly; clarified why gtk_file_chooser_set_current_na... 
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/e0107cbd45da66d23ad970424e645ff73d37126f

fixed to not duplicate code needlessly; clarified why gtk_file_chooser_set_current_name is called as it is


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@30804 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/gtk/filedlg.cpp b/src/gtk/filedlg.cpp
index 04aa374d79..929c0ccd68 100644
--- a/src/gtk/filedlg.cpp
+++ b/src/gtk/filedlg.cpp
@@ -237,21 +237,9 @@ void wxFileDialog::SetPath(const wxString& path)
 
     wxFileName fn(path);
     m_path = fn.GetFullPath();
-    m_dir = fn.GetPath();
-    m_fileName = fn.GetFullName();
     
-    gtk_file_chooser_set_current_folder(GTK_FILE_CHOOSER(m_widget),
-                                            wxGTK_CONV(m_dir));
-                                            
-    gtk_file_chooser_set_filename(GTK_FILE_CHOOSER(m_widget),
-                                      wxGTK_CONV(m_path));
-                                      
-    // pre-fill the filename, too:
-    if (GetWindowStyle() & wxSAVE) // Why only then??
-    {
-        gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER(m_widget),
-                                              wxGTK_CONV(m_fileName));
-    }
+    SetDirectory(fn.GetPath());
+    SetFilename(fn.GetFullName());
 }
 
 void wxFileDialog::SetDirectory(const wxString& dir)
@@ -275,8 +263,10 @@ void wxFileDialog::SetFilename(const wxString& name)
     gtk_file_chooser_set_filename(GTK_FILE_CHOOSER(m_widget),
                                       wxGTK_CONV(m_path));
 
-    // pre-fill the filename, too:
-    if (GetWindowStyle() & wxSAVE) // Why only then??
+    // pre-fill the filename when saving, too (there's no text entry
+    // control when opening a file, so it doesn't make sense to
+    // do this in when opening files):
+    if (GetWindowStyle() & wxSAVE)
     {
         gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER(m_widget),
                                               wxGTK_CONV(m_fileName));
diff --git a/src/gtk1/filedlg.cpp b/src/gtk1/filedlg.cpp
index 04aa374d79..929c0ccd68 100644
--- a/src/gtk1/filedlg.cpp
+++ b/src/gtk1/filedlg.cpp
@@ -237,21 +237,9 @@ void wxFileDialog::SetPath(const wxString& path)
 
     wxFileName fn(path);
     m_path = fn.GetFullPath();
-    m_dir = fn.GetPath();
-    m_fileName = fn.GetFullName();
     
-    gtk_file_chooser_set_current_folder(GTK_FILE_CHOOSER(m_widget),
-                                            wxGTK_CONV(m_dir));
-                                            
-    gtk_file_chooser_set_filename(GTK_FILE_CHOOSER(m_widget),
-                                      wxGTK_CONV(m_path));
-                                      
-    // pre-fill the filename, too:
-    if (GetWindowStyle() & wxSAVE) // Why only then??
-    {
-        gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER(m_widget),
-                                              wxGTK_CONV(m_fileName));
-    }
+    SetDirectory(fn.GetPath());
+    SetFilename(fn.GetFullName());
 }
 
 void wxFileDialog::SetDirectory(const wxString& dir)
@@ -275,8 +263,10 @@ void wxFileDialog::SetFilename(const wxString& name)
     gtk_file_chooser_set_filename(GTK_FILE_CHOOSER(m_widget),
                                       wxGTK_CONV(m_path));
 
-    // pre-fill the filename, too:
-    if (GetWindowStyle() & wxSAVE) // Why only then??
+    // pre-fill the filename when saving, too (there's no text entry
+    // control when opening a file, so it doesn't make sense to
+    // do this in when opening files):
+    if (GetWindowStyle() & wxSAVE)
     {
         gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER(m_widget),
                                               wxGTK_CONV(m_fileName));