compilation warning removed

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

diff --git a/include/wx/gtk/filedlg.h b/include/wx/gtk/filedlg.h
index 641c056762ed2d8e8d61c7479599ed2811d1ff24..c5d4ae64b1e055aa72124618a49ad9ff2519a1ee 100644 (file)
--- a/include/wx/gtk/filedlg.h
+++ b/include/wx/gtk/filedlg.h
@@ -26,39 +26,38 @@ extern const char *wxFileSelectorDefaultWildcardStr;
 
 class wxFileDialog: public wxDialog
 {
 
 class wxFileDialog: public wxDialog
 {

-
-  DECLARE_DYNAMIC_CLASS(wxFileDialog)
-
-  friend void gtk_filedialog_ok_callback( GtkWidget *widget, gpointer data );
-
-  public:
-
-    wxFileDialog() {};
-
-    wxFileDialog(wxWindow *parent, const wxString& message = wxFileSelectorPromptStr,
-        const wxString& defaultDir = "", const wxString& defaultFile = "",
-    const wxString& wildCard = wxFileSelectorDefaultWildcardStr,
-        long style = 0, const wxPoint& pos = wxDefaultPosition);
-
-    inline void SetMessage(const wxString& message) { m_message = message; }
-    inline void SetPath(const wxString& path) { m_path = path; }
-    inline void SetDirectory(const wxString& dir) { m_dir = dir; }
-    inline void SetFilename(const wxString& name) { m_fileName = name; }
-    inline void SetWildcard(const wxString& wildCard) { m_wildCard = wildCard; }
-    inline void SetStyle(long style) { m_dialogStyle = style; }
-    inline void SetFilterIndex(int filterIndex) { m_filterIndex = filterIndex; }
-
-    inline wxString GetMessage(void) const { return m_message; }
-    inline wxString GetPath(void) const { return m_path; }
-    inline wxString GetDirectory(void) const { return m_dir; }
-    inline wxString GetFilename(void) const { return m_fileName; }
-    inline wxString GetWildcard(void) const { return m_wildCard; }
-    inline long GetStyle(void) const { return m_dialogStyle; }
-    inline int GetFilterIndex(void) const { return m_filterIndex ; }
-
-    int ShowModal(void);
-
-  protected:
+DECLARE_DYNAMIC_CLASS(wxFileDialog)
+
+public:
+    wxFileDialog() { }
+
+    wxFileDialog(wxWindow *parent,
+                 const wxString& message = wxFileSelectorPromptStr,
+                 const wxString& defaultDir = "",
+                 const wxString& defaultFile = "",
+                 const wxString& wildCard = wxFileSelectorDefaultWildcardStr,
+                 long style = 0,
+                 const wxPoint& pos = wxDefaultPosition);
+
+    void SetMessage(const wxString& message) { m_message = message; }
+    void SetPath(const wxString& path) { m_path = path; }
+    void SetDirectory(const wxString& dir) { m_dir = dir; }
+    void SetFilename(const wxString& name) { m_fileName = name; }
+    void SetWildcard(const wxString& wildCard) { m_wildCard = wildCard; }
+    void SetStyle(long style) { m_dialogStyle = style; }
+    void SetFilterIndex(int filterIndex) { m_filterIndex = filterIndex; }
+
+    wxString GetMessage() const { return m_message; }
+    wxString GetPath() const { return m_path; }
+    wxString GetDirectory() const { return m_dir; }
+    wxString GetFilename() const { return m_fileName; }
+    wxString GetWildcard() const { return m_wildCard; }
+    long GetStyle() const { return m_dialogStyle; }
+    int GetFilterIndex() const { return m_filterIndex ; }
+
+    int ShowModal();
+
+    protected:

 
     wxString    m_message;
     long        m_dialogStyle;
 
     wxString    m_message;
     long        m_dialogStyle;

diff --git a/include/wx/gtk1/filedlg.h b/include/wx/gtk1/filedlg.h
index 641c056762ed2d8e8d61c7479599ed2811d1ff24..c5d4ae64b1e055aa72124618a49ad9ff2519a1ee 100644 (file)
--- a/include/wx/gtk1/filedlg.h
+++ b/include/wx/gtk1/filedlg.h
@@ -26,39 +26,38 @@ extern const char *wxFileSelectorDefaultWildcardStr;
 
 class wxFileDialog: public wxDialog
 {
 
 class wxFileDialog: public wxDialog
 {

-
-  DECLARE_DYNAMIC_CLASS(wxFileDialog)
-
-  friend void gtk_filedialog_ok_callback( GtkWidget *widget, gpointer data );
-
-  public:
-
-    wxFileDialog() {};
-
-    wxFileDialog(wxWindow *parent, const wxString& message = wxFileSelectorPromptStr,
-        const wxString& defaultDir = "", const wxString& defaultFile = "",
-    const wxString& wildCard = wxFileSelectorDefaultWildcardStr,
-        long style = 0, const wxPoint& pos = wxDefaultPosition);
-
-    inline void SetMessage(const wxString& message) { m_message = message; }
-    inline void SetPath(const wxString& path) { m_path = path; }
-    inline void SetDirectory(const wxString& dir) { m_dir = dir; }
-    inline void SetFilename(const wxString& name) { m_fileName = name; }
-    inline void SetWildcard(const wxString& wildCard) { m_wildCard = wildCard; }
-    inline void SetStyle(long style) { m_dialogStyle = style; }
-    inline void SetFilterIndex(int filterIndex) { m_filterIndex = filterIndex; }
-
-    inline wxString GetMessage(void) const { return m_message; }
-    inline wxString GetPath(void) const { return m_path; }
-    inline wxString GetDirectory(void) const { return m_dir; }
-    inline wxString GetFilename(void) const { return m_fileName; }
-    inline wxString GetWildcard(void) const { return m_wildCard; }
-    inline long GetStyle(void) const { return m_dialogStyle; }
-    inline int GetFilterIndex(void) const { return m_filterIndex ; }
-
-    int ShowModal(void);
-
-  protected:
+DECLARE_DYNAMIC_CLASS(wxFileDialog)
+
+public:
+    wxFileDialog() { }
+
+    wxFileDialog(wxWindow *parent,
+                 const wxString& message = wxFileSelectorPromptStr,
+                 const wxString& defaultDir = "",
+                 const wxString& defaultFile = "",
+                 const wxString& wildCard = wxFileSelectorDefaultWildcardStr,
+                 long style = 0,
+                 const wxPoint& pos = wxDefaultPosition);
+
+    void SetMessage(const wxString& message) { m_message = message; }
+    void SetPath(const wxString& path) { m_path = path; }
+    void SetDirectory(const wxString& dir) { m_dir = dir; }
+    void SetFilename(const wxString& name) { m_fileName = name; }
+    void SetWildcard(const wxString& wildCard) { m_wildCard = wildCard; }
+    void SetStyle(long style) { m_dialogStyle = style; }
+    void SetFilterIndex(int filterIndex) { m_filterIndex = filterIndex; }
+
+    wxString GetMessage() const { return m_message; }
+    wxString GetPath() const { return m_path; }
+    wxString GetDirectory() const { return m_dir; }
+    wxString GetFilename() const { return m_fileName; }
+    wxString GetWildcard() const { return m_wildCard; }
+    long GetStyle() const { return m_dialogStyle; }
+    int GetFilterIndex() const { return m_filterIndex ; }
+
+    int ShowModal();
+
+    protected:

 
     wxString    m_message;
     long        m_dialogStyle;
 
     wxString    m_message;
     long        m_dialogStyle;

diff --git a/src/gtk/filedlg.cpp b/src/gtk/filedlg.cpp
index a3e60fe420ebcf84c2d433f28ba6d05fa5f66b88..4d314208c57be7c5479a1fff2e7ed4265be846a5 100644 (file)
--- a/src/gtk/filedlg.cpp
+++ b/src/gtk/filedlg.cpp
@@ -22,16 +22,16 @@
 // "delete_event"
 //-----------------------------------------------------------------------------
 
 // "delete_event"
 //-----------------------------------------------------------------------------
 

-static 
+static

 bool gtk_filedialog_delete_callback( GtkWidget *WXUNUSED(widget), GdkEvent *WXUNUSED(event), wxDialog *win )
 bool gtk_filedialog_delete_callback( GtkWidget *WXUNUSED(widget), GdkEvent *WXUNUSED(event), wxDialog *win )

-{ 
+{

 /*
     printf( "OnDelete from " );
     if (win->GetClassInfo() && win->GetClassInfo()->GetClassName())
         printf( win->GetClassInfo()->GetClassName() );
     printf( ".\n" );
 */
 /*
     printf( "OnDelete from " );
     if (win->GetClassInfo() && win->GetClassInfo()->GetClassName())
         printf( win->GetClassInfo()->GetClassName() );
     printf( ".\n" );
 */

-  
+

     win->Close();
 
     return TRUE;
     win->Close();
 
     return TRUE;


@@ -41,7 +41,7 @@ bool gtk_filedialog_delete_callback( GtkWidget *WXUNUSED(widget), GdkEvent *WXUN
 // "clicked" for OK-button
 //-----------------------------------------------------------------------------
 
 // "clicked" for OK-button
 //-----------------------------------------------------------------------------
 

-static 
+static

 void gtk_filedialog_ok_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 {
     wxFileDialog *dialog = (wxFileDialog*)data;
 void gtk_filedialog_ok_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 {
     wxFileDialog *dialog = (wxFileDialog*)data;


@@ -50,14 +50,21 @@ void gtk_filedialog_ok_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 
     style = dialog->GetStyle();
 
 
     style = dialog->GetStyle();
 

-    if((style&wxSAVE)&&(style&wxOVERWRITE_PROMPT))
+    if( (style & wxSAVE ) && ( style&wxOVERWRITE_PROMPT ) )

     {
     {

-       if(wxFileExists(gtk_file_selection_get_filename(GTK_FILE_SELECTION(dialog->m_widget) ))) 
-       {
-         if(wxMessageBox(_("File exists. Overwrite?"),
-                         _("Confirm"), wxYES_NO) != wxYES)
-               return;
-       }
+        char *filename = gtk_file_selection_get_filename(
+                            GTK_FILE_SELECTION(dialog->m_widget)
+                         );
+
+        if(wxFileExists( filename ))
+        {
+            wxString msg;
+            msg.Printf(_("File '%s' already exists, do you really want to "
+                         "overwrite it?"), filename);
+
+            if( wxMessageBox(msg, _("Confirm"), wxYES_NO) != wxYES)
+                return;
+        }

     }
 
     dialog->OnOK( event );
     }
 
     dialog->OnOK( event );


@@ -67,7 +74,7 @@ void gtk_filedialog_ok_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 // "clicked" for Cancel-button
 //-----------------------------------------------------------------------------
 
 // "clicked" for Cancel-button
 //-----------------------------------------------------------------------------
 

-static 
+static

 void gtk_filedialog_cancel_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 {
     wxFileDialog *dialog = (wxFileDialog*)data;
 void gtk_filedialog_cancel_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 {
     wxFileDialog *dialog = (wxFileDialog*)data;


@@ -98,11 +105,11 @@ wxFileDialog::wxFileDialog( wxWindow *parent, const wxString& message,
     m_filterIndex = 1;
 
     m_widget = gtk_file_selection_new( m_message );
     m_filterIndex = 1;
 
     m_widget = gtk_file_selection_new( m_message );

-  
+

     int x = (gdk_screen_width () - 400) / 2;
     int y = (gdk_screen_height () - 400) / 2;
     gtk_widget_set_uposition( m_widget, x, y );
     int x = (gdk_screen_width () - 400) / 2;
     int y = (gdk_screen_height () - 400) / 2;
     gtk_widget_set_uposition( m_widget, x, y );

-  
+

     GtkFileSelection *sel = GTK_FILE_SELECTION(m_widget);
 
     m_path.Append(m_dir);
     GtkFileSelection *sel = GTK_FILE_SELECTION(m_widget);
 
     m_path.Append(m_dir);


@@ -117,9 +124,9 @@ wxFileDialog::wxFileDialog( wxWindow *parent, const wxString& message,
     gtk_signal_connect( GTK_OBJECT(sel->cancel_button), "clicked",
       GTK_SIGNAL_FUNC(gtk_filedialog_cancel_callback), (gpointer*)this );
 
     gtk_signal_connect( GTK_OBJECT(sel->cancel_button), "clicked",
       GTK_SIGNAL_FUNC(gtk_filedialog_cancel_callback), (gpointer*)this );
 

-    gtk_signal_connect( GTK_OBJECT(m_widget), "delete_event", 
+    gtk_signal_connect( GTK_OBJECT(m_widget), "delete_event",

         GTK_SIGNAL_FUNC(gtk_filedialog_delete_callback), (gpointer)this );
         GTK_SIGNAL_FUNC(gtk_filedialog_delete_callback), (gpointer)this );

-    
+

 }
 
 int wxFileDialog::ShowModal(void)
 }
 
 int wxFileDialog::ShowModal(void)


@@ -157,7 +164,7 @@ wxString wxFileSelector( const char *title,
         defaultFilenameString = defaultFileName;
     else
         defaultFilenameString = "";
         defaultFilenameString = defaultFileName;
     else
         defaultFilenameString = "";

- 
+

     wxFileDialog fileDialog( parent, title, defaultDirString, defaultFilenameString, filter2, flags, wxPoint(x, y) );
 
     if ( fileDialog.ShowModal() == wxID_OK )
     wxFileDialog fileDialog( parent, title, defaultDirString, defaultFilenameString, filter2, flags, wxPoint(x, y) );
 
     if ( fileDialog.ShowModal() == wxID_OK )

diff --git a/src/gtk1/filedlg.cpp b/src/gtk1/filedlg.cpp
index a3e60fe420ebcf84c2d433f28ba6d05fa5f66b88..4d314208c57be7c5479a1fff2e7ed4265be846a5 100644 (file)
--- a/src/gtk1/filedlg.cpp
+++ b/src/gtk1/filedlg.cpp
@@ -22,16 +22,16 @@
 // "delete_event"
 //-----------------------------------------------------------------------------
 
 // "delete_event"
 //-----------------------------------------------------------------------------
 

-static 
+static

 bool gtk_filedialog_delete_callback( GtkWidget *WXUNUSED(widget), GdkEvent *WXUNUSED(event), wxDialog *win )
 bool gtk_filedialog_delete_callback( GtkWidget *WXUNUSED(widget), GdkEvent *WXUNUSED(event), wxDialog *win )

-{ 
+{

 /*
     printf( "OnDelete from " );
     if (win->GetClassInfo() && win->GetClassInfo()->GetClassName())
         printf( win->GetClassInfo()->GetClassName() );
     printf( ".\n" );
 */
 /*
     printf( "OnDelete from " );
     if (win->GetClassInfo() && win->GetClassInfo()->GetClassName())
         printf( win->GetClassInfo()->GetClassName() );
     printf( ".\n" );
 */

-  
+

     win->Close();
 
     return TRUE;
     win->Close();
 
     return TRUE;


@@ -41,7 +41,7 @@ bool gtk_filedialog_delete_callback( GtkWidget *WXUNUSED(widget), GdkEvent *WXUN
 // "clicked" for OK-button
 //-----------------------------------------------------------------------------
 
 // "clicked" for OK-button
 //-----------------------------------------------------------------------------
 

-static 
+static

 void gtk_filedialog_ok_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 {
     wxFileDialog *dialog = (wxFileDialog*)data;
 void gtk_filedialog_ok_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 {
     wxFileDialog *dialog = (wxFileDialog*)data;


@@ -50,14 +50,21 @@ void gtk_filedialog_ok_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 
     style = dialog->GetStyle();
 
 
     style = dialog->GetStyle();
 

-    if((style&wxSAVE)&&(style&wxOVERWRITE_PROMPT))
+    if( (style & wxSAVE ) && ( style&wxOVERWRITE_PROMPT ) )

     {
     {

-       if(wxFileExists(gtk_file_selection_get_filename(GTK_FILE_SELECTION(dialog->m_widget) ))) 
-       {
-         if(wxMessageBox(_("File exists. Overwrite?"),
-                         _("Confirm"), wxYES_NO) != wxYES)
-               return;
-       }
+        char *filename = gtk_file_selection_get_filename(
+                            GTK_FILE_SELECTION(dialog->m_widget)
+                         );
+
+        if(wxFileExists( filename ))
+        {
+            wxString msg;
+            msg.Printf(_("File '%s' already exists, do you really want to "
+                         "overwrite it?"), filename);
+
+            if( wxMessageBox(msg, _("Confirm"), wxYES_NO) != wxYES)
+                return;
+        }

     }
 
     dialog->OnOK( event );
     }
 
     dialog->OnOK( event );


@@ -67,7 +74,7 @@ void gtk_filedialog_ok_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 // "clicked" for Cancel-button
 //-----------------------------------------------------------------------------
 
 // "clicked" for Cancel-button
 //-----------------------------------------------------------------------------
 

-static 
+static

 void gtk_filedialog_cancel_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 {
     wxFileDialog *dialog = (wxFileDialog*)data;
 void gtk_filedialog_cancel_callback( GtkWidget *WXUNUSED(widget), gpointer data )
 {
     wxFileDialog *dialog = (wxFileDialog*)data;


@@ -98,11 +105,11 @@ wxFileDialog::wxFileDialog( wxWindow *parent, const wxString& message,
     m_filterIndex = 1;
 
     m_widget = gtk_file_selection_new( m_message );
     m_filterIndex = 1;
 
     m_widget = gtk_file_selection_new( m_message );

-  
+

     int x = (gdk_screen_width () - 400) / 2;
     int y = (gdk_screen_height () - 400) / 2;
     gtk_widget_set_uposition( m_widget, x, y );
     int x = (gdk_screen_width () - 400) / 2;
     int y = (gdk_screen_height () - 400) / 2;
     gtk_widget_set_uposition( m_widget, x, y );

-  
+

     GtkFileSelection *sel = GTK_FILE_SELECTION(m_widget);
 
     m_path.Append(m_dir);
     GtkFileSelection *sel = GTK_FILE_SELECTION(m_widget);
 
     m_path.Append(m_dir);


@@ -117,9 +124,9 @@ wxFileDialog::wxFileDialog( wxWindow *parent, const wxString& message,
     gtk_signal_connect( GTK_OBJECT(sel->cancel_button), "clicked",
       GTK_SIGNAL_FUNC(gtk_filedialog_cancel_callback), (gpointer*)this );
 
     gtk_signal_connect( GTK_OBJECT(sel->cancel_button), "clicked",
       GTK_SIGNAL_FUNC(gtk_filedialog_cancel_callback), (gpointer*)this );
 

-    gtk_signal_connect( GTK_OBJECT(m_widget), "delete_event", 
+    gtk_signal_connect( GTK_OBJECT(m_widget), "delete_event",

         GTK_SIGNAL_FUNC(gtk_filedialog_delete_callback), (gpointer)this );
         GTK_SIGNAL_FUNC(gtk_filedialog_delete_callback), (gpointer)this );

-    
+

 }
 
 int wxFileDialog::ShowModal(void)
 }
 
 int wxFileDialog::ShowModal(void)


@@ -157,7 +164,7 @@ wxString wxFileSelector( const char *title,
         defaultFilenameString = defaultFileName;
     else
         defaultFilenameString = "";
         defaultFilenameString = defaultFileName;
     else
         defaultFilenameString = "";

- 
+

     wxFileDialog fileDialog( parent, title, defaultDirString, defaultFilenameString, filter2, flags, wxPoint(x, y) );
 
     if ( fileDialog.ShowModal() == wxID_OK )
     wxFileDialog fileDialog( parent, title, defaultDirString, defaultFilenameString, filter2, flags, wxPoint(x, y) );
 
     if ( fileDialog.ShowModal() == wxID_OK )