From 6b5c2d5262df98e197736d81d8a9fe54d1932ab0 Mon Sep 17 00:00:00 2001 From: Stefan Neis Date: Sat, 29 Mar 2008 19:34:37 +0000 Subject: [PATCH] OS/2 port: reorganized wxBitmap/wxBitmapHandler classes to use wxBitmapType instead of "long" where possible; made them const-correct etc. (see r52820) git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@52911 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- include/wx/os2/bitmap.h | 31 +++++++++++++++------------ include/wx/os2/gdiimage.h | 26 +++++++++++------------ include/wx/os2/icon.h | 4 ++-- src/os2/bitmap.cpp | 21 ++++++++++--------- src/os2/gdiimage.cpp | 44 +++++++++++++++++++-------------------- src/os2/icon.cpp | 4 ++-- 6 files changed, 68 insertions(+), 62 deletions(-) diff --git a/include/wx/os2/bitmap.h b/include/wx/os2/bitmap.h index 5f0be316b6..82639748f2 100644 --- a/include/wx/os2/bitmap.h +++ b/include/wx/os2/bitmap.h @@ -26,6 +26,7 @@ class WXDLLIMPEXP_FWD_CORE wxMask; class WXDLLIMPEXP_FWD_CORE wxCursor; class WXDLLIMPEXP_FWD_CORE wxControl; class WXDLLIMPEXP_FWD_CORE wxImage; +class WXDLLIMPEXP_FWD_CORE wxPixelDataBase; // ---------------------------------------------------------------------------- // Bitmap data @@ -96,17 +97,17 @@ public: // Load a resource wxBitmap( int nId - ,long lType = wxBITMAP_TYPE_BMP_RESOURCE + ,wxBitmapType lType = wxBITMAP_TYPE_BMP_RESOURCE ); // For compatiability with other ports, under OS/2 does same as default ctor inline wxBitmap( const wxString& WXUNUSED(rFilename) - ,long WXUNUSED(lType) + ,wxBitmapType WXUNUSED(lType) ) { Init(); } // New constructor for generalised creation from data wxBitmap( const void* pData - ,long lType + ,wxBitmapType lType ,int nWidth ,int nHeight ,int nDepth = 1 @@ -154,25 +155,29 @@ public: ,int nDepth = -1 ); virtual bool Create( const void* pData - ,long lType + ,wxBitmapType lType ,int nWidth ,int nHeight ,int nDepth = 1 ); virtual bool LoadFile( int nId - ,long lType = wxBITMAP_TYPE_BMP_RESOURCE + ,wxBitmapType lType = wxBITMAP_TYPE_BMP_RESOURCE ); virtual bool LoadFile( const wxString& rName - ,long lType = wxBITMAP_TYPE_XPM + ,wxBitmapType lType = wxBITMAP_TYPE_XPM ); virtual bool SaveFile( const wxString& rName - ,int lType + ,wxBitmapType lType ,const wxPalette* pCmap = NULL ); inline wxBitmapRefData* GetBitmapData() const { return (wxBitmapRefData *)m_refData; } + // raw bitmap access support functions + void *GetRawData(wxPixelDataBase& data, int bpp); + void UngetRawData(wxPixelDataBase& data); + inline int GetQuality() const { return (GetBitmapData() ? GetBitmapData()->m_nQuality : 0); } @@ -324,26 +329,26 @@ public: virtual bool Create( wxGDIImage* pImage ,const void* pData - ,long lFlags + ,wxBitmapType lFlags ,int nWidth ,int nHeight ,int nDepth = 1 ); virtual bool Load( wxGDIImage* pImage ,int nId - ,long lFlags + ,wxBitmapType lFlags ,int nDesiredWidth ,int nDesiredHeight ); - virtual bool Save( wxGDIImage* pImage - ,const wxString& rName - ,int lType + virtual bool Save( const wxGDIImage* pImage + ,const wxString& rName + ,wxBitmapType lType ) const; private: inline virtual bool Load( wxGDIImage* WXUNUSED(pImage) ,const wxString& WXUNUSED(rName) ,WXHANDLE WXUNUSED(hPs) - ,long WXUNUSED(lFlags) + ,wxBitmapType WXUNUSED(lFlags) ,int WXUNUSED(nDesiredWidth) ,int WXUNUSED(nDesiredHeight) ) diff --git a/include/wx/os2/gdiimage.h b/include/wx/os2/gdiimage.h index f529c787cf..8b5625abdf 100644 --- a/include/wx/os2/gdiimage.h +++ b/include/wx/os2/gdiimage.h @@ -88,7 +88,7 @@ public: wxGDIImageHandler() { m_lType = wxBITMAP_TYPE_INVALID; } wxGDIImageHandler( const wxString& rName ,const wxString& rExt - ,long lType + ,wxBitmapType lType ) : m_sName(rName) , m_sExtension(rExt) @@ -99,16 +99,16 @@ public: // accessors void SetName(const wxString& rName) { m_sName = rName; } void SetExtension(const wxString& rExt) { m_sExtension = rExt; } - void SetType(long lType) { m_lType = lType; } + void SetType(wxBitmapType lType) { m_lType = lType; } wxString GetName() const { return m_sName; } wxString GetExtension() const { return m_sExtension; } - long GetType() const { return m_lType; } + wxBitmapType GetType() const { return m_lType; } // real handler operations: to implement in derived classes virtual bool Create( wxGDIImage* pImage ,const void* pData - ,long lFlags + ,wxBitmapType lFlags ,int nWidth ,int nHeight ,int nDepth = 1 @@ -116,25 +116,25 @@ public: virtual bool Load( wxGDIImage* pImage ,const wxString& rName ,HPS hPs - ,long lFlags + ,wxBitmapType lFlags ,int nDesiredWidth ,int nDesiredHeight ) = 0; virtual bool Load( wxGDIImage* pImage ,int nId - ,long lFlags + ,wxBitmapType lFlags ,int nDesiredWidth ,int nDesiredHeight ) = 0; - virtual bool Save( wxGDIImage* pImage - ,const wxString& rName - ,int lType - ) = 0; + virtual bool Save( const wxGDIImage* pImage + ,const wxString& rName + ,wxBitmapType lType + ) const = 0; protected: wxString m_sName; wxString m_sExtension; - long m_lType; + wxBitmapType m_lType; }; // end of wxGDIImageHandler // ---------------------------------------------------------------------------- @@ -154,8 +154,8 @@ public: static bool RemoveHandler(const wxString& rName); static wxGDIImageHandler* FindHandler(const wxString& rName); - static wxGDIImageHandler* FindHandler(const wxString& rExtension, long lType); - static wxGDIImageHandler* FindHandler(long lType); + static wxGDIImageHandler* FindHandler(const wxString& rExtension, wxBitmapType lType); + static wxGDIImageHandler* FindHandler(wxBitmapType lType); static void InitStandardHandlers(); static void CleanUpHandlers(); diff --git a/include/wx/os2/icon.h b/include/wx/os2/icon.h index 5ccf5424a0..7ed62517f6 100644 --- a/include/wx/os2/icon.h +++ b/include/wx/os2/icon.h @@ -49,7 +49,7 @@ public: wxIcon(char** ppData) { CreateIconFromXpm(wx_const_cast(const char* const*, ppData)); } #endif wxIcon( const wxString& rName - ,long lFlags = wxBITMAP_TYPE_ICO_RESOURCE + ,wxBitmapType lFlags = wxBITMAP_TYPE_ICO_RESOURCE ,int nDesiredWidth = -1 ,int nDesiredHeight = -1 ); @@ -61,7 +61,7 @@ public: virtual ~wxIcon(); bool LoadFile( const wxString& rName - ,long lFlags = wxBITMAP_TYPE_ICO_RESOURCE + ,wxBitmapType lFlags = wxBITMAP_TYPE_ICO_RESOURCE ,int nDesiredWidth = -1 ,int nDesiredHeight = -1 ); diff --git a/src/os2/bitmap.cpp b/src/os2/bitmap.cpp index 73912fabc7..d4c45a3099 100644 --- a/src/os2/bitmap.cpp +++ b/src/os2/bitmap.cpp @@ -290,7 +290,7 @@ wxBitmap::wxBitmap( wxBitmap::wxBitmap( const void* pData -, long lType +, wxBitmapType lType , int nWidth , int nHeight , int nDepth @@ -308,12 +308,12 @@ wxBitmap::wxBitmap( wxBitmap::wxBitmap( int nId -, long lType +, wxBitmapType lType ) { Init(); LoadFile( nId - ,(int)lType + ,lType ); SetId(nId); } // end of wxBitmap::wxBitmap @@ -398,7 +398,7 @@ bool wxBitmap::Create( return Ok(); } // end of wxBitmap::Create -bool wxBitmap::LoadFile(const wxString& filename, long type) +bool wxBitmap::LoadFile(const wxString& filename, wxBitmapType type) { UnRef(); @@ -428,7 +428,7 @@ bool wxBitmap::LoadFile(const wxString& filename, long type) bool wxBitmap::LoadFile( int nId -, long lType +, wxBitmapType lType ) { UnRef(); @@ -456,7 +456,7 @@ bool wxBitmap::LoadFile( bool wxBitmap::Create( const void* pData -, long lType +, wxBitmapType lType , int nWidth , int nHeight , int nDepth @@ -488,7 +488,7 @@ bool wxBitmap::Create( bool wxBitmap::SaveFile( const wxString& rFilename -, int lType +, wxBitmapType lType , const wxPalette* pPalette ) { @@ -1456,7 +1456,7 @@ bool wxMask::Create( bool wxBitmapHandler::Create( wxGDIImage* pImage, const void* pData, - wxBitmapType WXUNUSED(lFlags), + wxBitmapType lType, int nWidth, int nHeight, int nDepth) @@ -1467,6 +1467,7 @@ bool wxBitmapHandler::Create( wxGDIImage* pImage, return(pBitmap ? Create( pBitmap ,pData + ,lType ,nWidth ,nHeight ,nDepth @@ -1494,9 +1495,9 @@ bool wxBitmapHandler::Load( } bool wxBitmapHandler::Save( - wxGDIImage* pImage + const wxGDIImage* pImage , const wxString& rName -, int lType +, wxBitmapType lType ) const { wxBitmap* pBitmap = wxDynamicCast( pImage diff --git a/src/os2/gdiimage.cpp b/src/os2/gdiimage.cpp index 4c2fee5425..94d7e4fb89 100644 --- a/src/os2/gdiimage.cpp +++ b/src/os2/gdiimage.cpp @@ -50,20 +50,20 @@ public: virtual bool LoadFile( wxBitmap* pBitmap ,const wxString& rName ,HPS hPs - ,long lFlags + ,wxBitmapType lFlags ,int nDesiredWidth ,int nDesiredHeight ); virtual bool SaveFile( wxBitmap* pBitmap ,const wxString& rName - ,int lType + ,wxBitmapType lType ,const wxPalette* pPalette = NULL ); private: inline virtual bool LoadFile( wxBitmap* pBitmap ,int nId - ,long lFlags + ,wxBitmapType lFlags ,int nDesiredWidth ,int nDesiredHeight ) @@ -89,7 +89,7 @@ public: virtual bool LoadFile( wxBitmap* pBitmap ,int nId - ,long lFlags + ,wxBitmapType lFlags ,int nDesiredWidth ,int nDesiredHeight ); @@ -103,7 +103,7 @@ class WXDLLEXPORT wxIconHandler : public wxGDIImageHandler public: wxIconHandler( const wxString& rName ,const wxString& rExt - ,long lType + ,wxBitmapType lType ) : wxGDIImageHandler( rName ,rExt ,lType @@ -114,7 +114,7 @@ public: // creating and saving icons is not supported virtual bool Create( wxGDIImage* WXUNUSED(pImage) ,const void* WXUNUSED(pData) - ,long WXUNUSED(lFlags) + ,wxBitmapType WXUNUSED(lFlags) ,int WXUNUSED(nWidth) ,int WXUNUSED(nHeight) ,int WXUNUSED(nDepth) = 1 @@ -123,17 +123,17 @@ public: return false; } - virtual bool Save( wxGDIImage* WXUNUSED(pImage) - ,const wxString& WXUNUSED(rName) - ,int WXUNUSED(nType) - ) + virtual bool Save( const wxGDIImage* WXUNUSED(pImage) + ,const wxString& WXUNUSED(rName) + ,wxBitmapType WXUNUSED(nType) + ) const { return false; } virtual bool Load( wxGDIImage* pImage ,const wxString& rName ,HPS hPs - ,long lFlags + ,wxBitmapType lFlags ,int nDesiredWidth ,int nDesiredHeight ) @@ -154,14 +154,14 @@ protected: virtual bool LoadIcon( wxIcon* pIcon ,const wxString& rName ,HPS hPs - ,long lFlags + ,wxBitmapType lFlags ,int nDesiredWidth = -1 ,int nDesiredHeight = -1 ) = 0; private: inline virtual bool Load( wxGDIImage* WXUNUSED(pImage), int WXUNUSED(nId), - long WXUNUSED(lFlags), + wxBitmapType WXUNUSED(lFlags), int WXUNUSED(nDesiredWidth), int WXUNUSED(nDesiredHeight) ) { @@ -181,7 +181,7 @@ public: virtual bool LoadIcon( wxIcon * pIcon ,const wxString& rName ,HPS hPs - ,long lFlags + ,wxBitmapType lFlags ,int nDesiredWidth = -1 ,int nDesiredHeight = -1 ); @@ -202,7 +202,7 @@ public: virtual bool LoadIcon( wxIcon* pIcon ,const wxString& rName ,HPS hPs - ,long lFlags + ,wxBitmapType lFlags ,int nDesiredWidth = -1 ,int nDesiredHeight = -1 ); @@ -292,7 +292,7 @@ wxGDIImageHandler* wxGDIImage::FindHandler( wxGDIImageHandler* wxGDIImage::FindHandler( const wxString& rExtension -, long lType +, wxBitmapType lType ) { wxGDIImageHandlerList::compatibility_iterator pNode = ms_handlers.GetFirst(); @@ -311,7 +311,7 @@ wxGDIImageHandler* wxGDIImage::FindHandler( } wxGDIImageHandler* wxGDIImage::FindHandler( - long lType + wxBitmapType lType ) { wxGDIImageHandlerList::compatibility_iterator pNode = ms_handlers.GetFirst(); @@ -356,7 +356,7 @@ void wxGDIImage::InitStandardHandlers() bool wxBMPResourceHandler::LoadFile( wxBitmap* pBitmap, int nId, - long WXUNUSED(lFlags), + wxBitmapType WXUNUSED(lFlags), int WXUNUSED(nDesiredWidth), int WXUNUSED(nDesiredHeight) ) { @@ -391,7 +391,7 @@ bool wxBMPResourceHandler::LoadFile( wxBitmap* pBitmap, bool wxBMPFileHandler::LoadFile( wxBitmap* pBitmap, const wxString& WXUNUSED(rName), HPS WXUNUSED(hPs), - long WXUNUSED(lFlags), + wxBitmapType WXUNUSED(lFlags), int WXUNUSED(nDesiredWidth), int WXUNUSED(nDesiredHeight) ) { @@ -419,7 +419,7 @@ bool wxBMPFileHandler::LoadFile( wxBitmap* pBitmap, bool wxBMPFileHandler::SaveFile( wxBitmap* pBitmap, const wxString& WXUNUSED(rName), - int WXUNUSED(nType), + wxBitmapType WXUNUSED(nType), const wxPalette* pPal ) { #if defined(wxUSE_IMAGE_LOADING_IN_OS2) && wxUSE_IMAGE_LOADING_IN_OS2 @@ -446,7 +446,7 @@ bool wxBMPFileHandler::SaveFile( wxBitmap* pBitmap, bool wxICOFileHandler::LoadIcon( wxIcon* pIcon, const wxString& WXUNUSED(rName), HPS WXUNUSED(hPs), - long WXUNUSED(lFlags), + wxBitmapType WXUNUSED(lFlags), int WXUNUSED(nDesiredWidth), int WXUNUSED(nDesiredHeight) ) { @@ -463,7 +463,7 @@ bool wxICOFileHandler::LoadIcon( wxIcon* pIcon, bool wxICOResourceHandler::LoadIcon( wxIcon* pIcon, const wxString& rName, HPS WXUNUSED(hPs), - long WXUNUSED(lFlags), + wxBitmapType WXUNUSED(lFlags), int WXUNUSED(nDesiredWidth), int WXUNUSED(nDesiredHeight) ) { diff --git a/src/os2/icon.cpp b/src/os2/icon.cpp index a25b0fb21d..bfef80f19c 100644 --- a/src/os2/icon.cpp +++ b/src/os2/icon.cpp @@ -61,7 +61,7 @@ wxIcon::wxIcon( const char WXUNUSED(bits)[], } wxIcon::wxIcon( const wxString& rIconFile, - long lFlags, + wxBitmapType lFlags, int nDesiredWidth, int nDesiredHeight ) :m_bIsXpm(false) @@ -257,7 +257,7 @@ void wxIcon::CopyFromBitmap( const wxBitmap& rBmp ) } // end of wxIcon::CopyFromBitmap bool wxIcon::LoadFile( const wxString& rFilename, - long lType, + wxBitmapType lType, int nDesiredWidth, int nDesiredHeight ) { -- 2.45.2