fix GCC warning

[wxWidgets.git] / src / os2 / clipbrd.cpp

diff --git a/src/os2/clipbrd.cpp b/src/os2/clipbrd.cpp
index 3474d380ab6a59e49330cb04c5a9eaae43d85ee3..523ee7229439d3fbcd7146ea0f8297ec5793e8c0 100644 (file)
--- a/src/os2/clipbrd.cpp
+++ b/src/os2/clipbrd.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        clipbrd.cpp
+// Name:        src/os2/clipbrd.cpp

 // Purpose:     Clipboard functionality
 // Author:      David Webster
 // Modified by:
 // Purpose:     Clipboard functionality
 // Author:      David Webster
 // Modified by:


@@ -12,12 +12,10 @@
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 

-#ifndef WX_PRECOMP
-    #include "wx/setup.h"
-#endif
-

 #if wxUSE_CLIPBOARD
 
 #if wxUSE_CLIPBOARD
 

+#include "wx/clipbrd.h"
+

 #ifndef WX_PRECOMP
     #include "wx/object.h"
     #include "wx/event.h"
 #ifndef WX_PRECOMP
     #include "wx/object.h"
     #include "wx/event.h"


@@ -26,26 +24,20 @@
     #include "wx/bitmap.h"
     #include "wx/utils.h"
     #include "wx/intl.h"
     #include "wx/bitmap.h"
     #include "wx/utils.h"
     #include "wx/intl.h"

+    #include "wx/log.h"
+    #include "wx/dataobj.h"

 #endif
 
 #if wxUSE_METAFILE
     #include "wx/metafile.h"
 #endif
 
 #endif
 
 #if wxUSE_METAFILE
     #include "wx/metafile.h"
 #endif
 

-#include "wx/log.h"
-#include "wx/clipbrd.h"
-

 #include <string.h>
 
 #include "wx/os2/private.h"
 
 // wxDataObject is tied to OLE/drag and drop implementation,
 // therefore so is wxClipboard :-(
 #include <string.h>
 
 #include "wx/os2/private.h"
 
 // wxDataObject is tied to OLE/drag and drop implementation,
 // therefore so is wxClipboard :-(

-#if wxUSE_DRAG_AND_DROP
-    #include "wx/dataobj.h"
-
-//    static bool wxSetClipboardData(wxDataObject *data);
-#endif

 
 // ===========================================================================
 // implementation
 
 // ===========================================================================
 // implementation


@@ -55,7 +47,7 @@
 // old-style clipboard functions using Windows API
 // ---------------------------------------------------------------------------
 
 // old-style clipboard functions using Windows API
 // ---------------------------------------------------------------------------
 

-static bool gs_wxClipboardIsOpen = FALSE;
+static bool gs_wxClipboardIsOpen = false;

 
 bool wxOpenClipboard()
 {
 
 bool wxOpenClipboard()
 {


@@ -68,7 +60,9 @@ bool wxOpenClipboard()
         gs_wxClipboardIsOpen = ::OpenClipboard((HWND)win->GetHWND()) != 0;
 
         if ( !gs_wxClipboardIsOpen )
         gs_wxClipboardIsOpen = ::OpenClipboard((HWND)win->GetHWND()) != 0;
 
         if ( !gs_wxClipboardIsOpen )

+        {

             wxLogSysError(_("Failed to open the clipboard."));
             wxLogSysError(_("Failed to open the clipboard."));

+        }

 
         return gs_wxClipboardIsOpen;
     }
 
         return gs_wxClipboardIsOpen;
     }


@@ -76,24 +70,24 @@ bool wxOpenClipboard()
     {
         wxLogDebug(wxT("Can not open clipboard without a main window."));
 
     {
         wxLogDebug(wxT("Can not open clipboard without a main window."));
 

-        return FALSE;
+        return false;

     }
 */
     }
 */

-    return FALSE;
+    return false;

 }
 
 bool wxCloseClipboard()
 {
 }
 
 bool wxCloseClipboard()
 {

-    wxCHECK_MSG( gs_wxClipboardIsOpen, FALSE, wxT("clipboard is not opened") );
+    wxCHECK_MSG( gs_wxClipboardIsOpen, false, wxT("clipboard is not opened") );

 // TODO:
 /*
 // TODO:
 /*

-    gs_wxClipboardIsOpen = FALSE;
+    gs_wxClipboardIsOpen = false;

 
     if ( ::CloseClipboard() == 0 )
     {
         wxLogSysError(_("Failed to close the clipboard."));
 
 
     if ( ::CloseClipboard() == 0 )
     {
         wxLogSysError(_("Failed to close the clipboard."));
 

-        return FALSE;
+        return false;

     }
 */
     return true;
     }
 */
     return true;


@@ -107,7 +101,7 @@ bool wxEmptyClipboard()
     {
         wxLogSysError(_("Failed to empty the clipboard."));
 
     {
         wxLogSysError(_("Failed to empty the clipboard."));
 

-        return FALSE;
+        return false;

     }
 */
     return true;
     }
 */
     return true;


@@ -137,7 +131,7 @@ static bool wxSetClipboardData(wxDataObject *data)
         wxLogSysError(_("Failed to allocate %dKb of memory for clipboard "
                         "transfer."), size / 1024);
 
         wxLogSysError(_("Failed to allocate %dKb of memory for clipboard "
                         "transfer."), size / 1024);
 

-        return FALSE;
+        return false;

     }
 
     LPVOID lpGlobalMemory = ::GlobalLock(hGlobal);
     }
 
     LPVOID lpGlobalMemory = ::GlobalLock(hGlobal);


@@ -152,7 +146,7 @@ static bool wxSetClipboardData(wxDataObject *data)
         wxLogSysError(_("Failed to set clipboard data in format %s"),
                       wxDataObject::GetFormatName(format));
 
         wxLogSysError(_("Failed to set clipboard data in format %s"),
                       wxDataObject::GetFormatName(format));
 

-        return FALSE;
+        return false;

     }
 */
     return true;
     }
 */
     return true;


@@ -160,9 +154,9 @@ static bool wxSetClipboardData(wxDataObject *data)
 #endif // wxUSE_DRAG_AND_DROP
 #endif
 
 #endif // wxUSE_DRAG_AND_DROP
 #endif
 

-bool wxSetClipboardData(wxDataFormat dataFormat,
-                        const void *data,
-                        int width, int height)
+bool wxSetClipboardData(wxDataFormat WXUNUSED(dataFormat),
+                        const void *WXUNUSED(data),
+                        int WXUNUSED(width), int WXUNUSED(height))

 {
 // TODO:
 /*
 {
 // TODO:
 /*


@@ -185,7 +179,7 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
                     SelectObject(hdcSrc, old);
                     DeleteDC(hdcMem);
                     DeleteDC(hdcSrc);
                     SelectObject(hdcSrc, old);
                     DeleteDC(hdcMem);
                     DeleteDC(hdcSrc);

-                    return FALSE;
+                    return false;

                 }
 
                 HBITMAP old1 = (HBITMAP) SelectObject(hdcMem, hBitmap);
                 }
 
                 HBITMAP old1 = (HBITMAP) SelectObject(hdcMem, hBitmap);


@@ -242,7 +236,7 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
         default:
             {
                 wxLogError(_("Unsupported clipboard format."));
         default:
             {
                 wxLogError(_("Unsupported clipboard format."));

-                return FALSE;
+                return false;

             }
 
         case wxDF_OEMTEXT:
             }
 
         case wxDF_OEMTEXT:


@@ -275,13 +269,13 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
     {
         wxLogSysError(_("Failed to set clipboard data."));
 
     {
         wxLogSysError(_("Failed to set clipboard data."));
 

-        return FALSE;
+        return false;

     }
 */
     return true;
 }
 
     }
 */
     return true;
 }
 

-void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
+void *wxGetClipboardData(wxDataFormat WXUNUSED(dataFormat), long *WXUNUSED(len))

 {
 //  void *retval = NULL;
 // TODO:
 {
 //  void *retval = NULL;
 // TODO:


@@ -342,7 +336,7 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
         case wxDF_DIB:
             {
                 wxLogError(_("Unsupported clipboard format."));
         case wxDF_DIB:
             {
                 wxLogError(_("Unsupported clipboard format."));

-                return FALSE;
+                return NULL;

             }
 
         case wxDF_OEMTEXT:
             }
 
         case wxDF_OEMTEXT:


@@ -414,15 +408,15 @@ wxDataFormat wxEnumClipboardFormats(wxDataFormat dataFormat)
   return dataFormat;
 }
 
   return dataFormat;
 }
 

-int wxRegisterClipboardFormat(wxChar *formatName)
+int wxRegisterClipboardFormat(wxChar *WXUNUSED(formatName))

 {
   // TODO: return ::RegisterClipboardFormat(formatName);
   return 0;
 }
 
 {
   // TODO: return ::RegisterClipboardFormat(formatName);
   return 0;
 }
 

-bool wxGetClipboardFormatName(wxDataFormat dataFormat,
-                              wxChar *formatName,
-                              int maxCount)
+bool wxGetClipboardFormatName(wxDataFormat WXUNUSED(dataFormat),
+                              wxChar *WXUNUSED(formatName),
+                              int WXUNUSED(maxCount))

 {
   // TODO: return ::GetClipboardFormatName((int)dataFormat, formatName, maxCount) > 0;
   return 0;
 {
   // TODO: return ::GetClipboardFormatName((int)dataFormat, formatName, maxCount) > 0;
   return 0;


@@ -450,7 +444,7 @@ void wxClipboard::Clear()
 bool wxClipboard::Flush()
 {
     // TODO:
 bool wxClipboard::Flush()
 {
     // TODO:

-    return FALSE;
+    return false;

 }
 
 bool wxClipboard::Open()
 }
 
 bool wxClipboard::Open()


@@ -463,7 +457,7 @@ bool wxClipboard::IsOpened() const
     return wxIsClipboardOpened();
 }
 
     return wxIsClipboardOpened();
 }
 

-bool wxClipboard::SetData( wxDataObject *data )
+bool wxClipboard::SetData( wxDataObject *WXUNUSED(data) )

 {
     (void)wxEmptyClipboard();
     // TODO:
 {
     (void)wxEmptyClipboard();
     // TODO:


@@ -478,10 +472,10 @@ bool wxClipboard::SetData( wxDataObject *data )
 
 bool wxClipboard::AddData( wxDataObject *data )
 {
 
 bool wxClipboard::AddData( wxDataObject *data )
 {

-    wxCHECK_MSG( data, FALSE, wxT("data is invalid") );
+    wxCHECK_MSG( data, false, wxT("data is invalid") );

 
 #if wxUSE_DRAG_AND_DROP
 
 #if wxUSE_DRAG_AND_DROP

-    wxCHECK_MSG( wxIsClipboardOpened(), FALSE, wxT("clipboard not open") );
+    wxCHECK_MSG( wxIsClipboardOpened(), false, wxT("clipboard not open") );

 
 //    wxDataFormat format = data->GetPreferredFormat();
 // TODO:
 
 //    wxDataFormat format = data->GetPreferredFormat();
 // TODO:


@@ -521,9 +515,9 @@ bool wxClipboard::AddData( wxDataObject *data )
     }
 #else // !wxUSE_DRAG_AND_DROP
 */
     }
 #else // !wxUSE_DRAG_AND_DROP
 */

-    return FALSE;
+    return false;

 #else
 #else

-    return FALSE;
+    return false;

 #endif // wxUSE_DRAG_AND_DROP/!wxUSE_DRAG_AND_DROP
 }
 
 #endif // wxUSE_DRAG_AND_DROP/!wxUSE_DRAG_AND_DROP
 }
 


@@ -537,9 +531,9 @@ bool wxClipboard::IsSupported( const wxDataFormat& format )
     return wxIsClipboardFormatAvailable(format);
 }
 
     return wxIsClipboardFormatAvailable(format);
 }
 

-bool wxClipboard::GetData( wxDataObject& data )
+bool wxClipboard::GetData( wxDataObject& WXUNUSED(data) )

 {
 {

-    wxCHECK_MSG( wxIsClipboardOpened(), FALSE, wxT("clipboard not open") );
+    wxCHECK_MSG( wxIsClipboardOpened(), false, wxT("clipboard not open") );

 
 #if wxUSE_DRAG_AND_DROP
 //    wxDataFormat format = data.GetPreferredFormat();
 
 #if wxUSE_DRAG_AND_DROP
 //    wxDataFormat format = data.GetPreferredFormat();


@@ -559,7 +553,7 @@ bool wxClipboard::GetData( wxDataObject& data )
                 return true;
             }
             else
                 return true;
             }
             else

-                return FALSE;
+                return false;

         }
 
         case wxDF_BITMAP:
         }
 
         case wxDF_BITMAP:


@@ -574,7 +568,7 @@ bool wxClipboard::GetData( wxDataObject& data )
                 return true;
             }
             else
                 return true;
             }
             else

-                return FALSE;
+                return false;

         }
 #if wxUSE_METAFILE
         case wxDF_METAFILE:
         }
 #if wxUSE_METAFILE
         case wxDF_METAFILE:


@@ -588,7 +582,7 @@ bool wxClipboard::GetData( wxDataObject& data )
                 return true;
             }
             else
                 return true;
             }
             else

-                return FALSE;
+                return false;

         }
 #endif
         default:
         }
 #endif
         default:


@@ -597,7 +591,7 @@ bool wxClipboard::GetData( wxDataObject& data )
                 void *buf = wxGetClipboardData(format, &len);
                 if ( buf )
                 {
                 void *buf = wxGetClipboardData(format, &len);
                 if ( buf )
                 {

-                    // FIXME this is for testing only!!
+                    // FIXME this is for testing only!

                     ((wxPrivateDataObject &)data).SetData(buf, len);
                     free(buf);
 
                     ((wxPrivateDataObject &)data).SetData(buf, len);
                     free(buf);
 


@@ -605,13 +599,13 @@ bool wxClipboard::GetData( wxDataObject& data )
                 }
             }
 
                 }
             }
 

-            return FALSE;
+            return false;

     }
 #else
 */
     }
 #else
 */

-    return FALSE;
+    return false;

 #else
 #else

-    return FALSE;
+    return false;

 #endif
 }
 
 #endif
 }