compilation warning fix

[wxWidgets.git] / src / generic / imaglist.cpp

diff --git a/src/generic/imaglist.cpp b/src/generic/imaglist.cpp
index 368a72735a259320367b4cb4527137267461c56b..aa3a2922080ae0b27b0c18f1ca7f992a0d5c21b2 100644 (file)
--- a/src/generic/imaglist.cpp
+++ b/src/generic/imaglist.cpp
@@ -1,18 +1,26 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        imaglist.cpp
+// Name:        generic/imaglist.cpp

 // Purpose:
 // Author:      Robert Roebling
 // Purpose:
 // Author:      Robert Roebling

-// Created:     01/02/97
-// Id:
-// Copyright:   (c) 1998 Robert Roebling, Julian Smart and Markus Holzem
-// Licence:    wxWindows licence
+// Id:          $id$
+// Copyright:   (c) 1998 Robert Roebling
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 #pragma implementation "imaglist.h"
 #endif
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 #pragma implementation "imaglist.h"
 #endif

- 
-#include "wx/imaglist.h"
+
+// For compilers that support precompilation, includes "wx.h".
+#include "wx/wxprec.h"
+
+#ifdef __BORLANDC__
+#pragma hdrstop
+#endif
+
+#include "wx/generic/imaglist.h"
+#include "wx/icon.h"
+#include "wx/image.h"

 
 //-----------------------------------------------------------------------------
 //  wxImageList
 
 //-----------------------------------------------------------------------------
 //  wxImageList


@@ -20,93 +28,149 @@
 
 IMPLEMENT_DYNAMIC_CLASS(wxImageList, wxObject)
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxImageList, wxObject)
 

-wxImageList::wxImageList(void)
+wxImageList::wxImageList( int width, int height, bool mask, int initialCount )
+{
+    (void)Create(width, height, mask, initialCount);
+}
+
+wxImageList::~wxImageList()

 {
 {

-  Create();
-};
+}

 
 

-wxImageList::~wxImageList(void)
+int wxImageList::GetImageCount() const

 {
 {

-};
+    return m_images.Number();
+}

 
 

-int wxImageList::GetImageCount(void) const
+bool wxImageList::Create( int width, int height, bool WXUNUSED(mask), int WXUNUSED(initialCount) )

 {
 {

-  return m_images.Number();
-};
+    m_width = width;
+    m_height = height;

 
 

-bool wxImageList::Create(void)
+    return Create();
+}
+
+bool wxImageList::Create()

 {
 {

-  m_images.DeleteContents( TRUE );
-  return TRUE;
-};
+    m_images.DeleteContents( TRUE );
+    return TRUE;
+}

 
 int wxImageList::Add( const wxBitmap &bitmap )
 {
 
 int wxImageList::Add( const wxBitmap &bitmap )
 {

-  m_images.Append( new wxBitmap(bitmap) );
-  return m_images.Number();
-};
+    if (bitmap.IsKindOf(CLASSINFO(wxIcon)))
+        m_images.Append( new wxIcon( (const wxIcon&) bitmap ) );
+    else
+        m_images.Append( new wxBitmap(bitmap) );
+    return m_images.Number()-1;
+}

 
 

-bool wxImageList::Replace( const int index, const wxBitmap &bitmap )
+int wxImageList::Add( const wxBitmap& bitmap, const wxBitmap& mask )

 {
 {

-  wxNode *node = m_images.Nth( index );
-  if (!node) return FALSE;
-  
-  if (index == m_images.Number()-1)
-  {
-    m_images.DeleteNode( node );
-    m_images.Append( new wxBitmap(bitmap) );
-  }
-  else
-  {
-    wxNode *next = node->Next();
-    m_images.DeleteNode( node );
-    m_images.Insert( next, new wxBitmap(bitmap) );
-  };
-  
-  return TRUE;
-};
+    wxBitmap bmp(bitmap);
+    if (mask.Ok())
+        bmp.SetMask(new wxMask(mask));
+    return Add(bmp);
+}

 
 

-bool wxImageList::Remove( const int index )
+int wxImageList::Add( const wxBitmap& bitmap, const wxColour& maskColour )

 {
 {

-  wxNode *node = m_images.Nth( index );
-  if (node) m_images.DeleteNode( node );
-  return (node != NULL);
-};
+    wxImage img(bitmap);
+    img.SetMaskColour(maskColour.Red(), maskColour.Green(), maskColour.Blue());
+    return Add(img.ConvertToBitmap());
+}

 
 

-bool wxImageList::RemoveAll(void)
+const wxBitmap *wxImageList::GetBitmap( int index ) const

 {
 {

-  m_images.Clear();
-  return TRUE;
-};
+    wxNode *node = m_images.Nth( index );
+
+    wxCHECK_MSG( node, (wxBitmap *) NULL, wxT("wrong index in image list") );
+
+    return (wxBitmap*)node->Data();
+}

 
 

-bool wxImageList::GetSize( const int index, int &width, int &height ) const
+bool wxImageList::Replace( int index, const wxBitmap &bitmap )

 {
 {

-  wxNode *node = m_images.Nth( index );
-  if (node)
-  {
+    wxNode *node = m_images.Nth( index );
+
+    wxCHECK_MSG( node, FALSE, wxT("wrong index in image list") );
+
+    wxBitmap* newBitmap = NULL;
+    if (bitmap.IsKindOf(CLASSINFO(wxIcon)))
+#if defined(__VISAGECPP__)
+//just can't do this in VisualAge now, with all this new Bitmap-Icon stuff
+//so construct it from a bitmap object until I can figure this nonsense out. (DW)
+        newBitmap = new wxBitmap(bitmap) ;
+#else
+        newBitmap = new wxBitmap( (const wxIcon&) bitmap );
+#endif
+    else
+        newBitmap = new wxBitmap(bitmap) ;
+
+    if (index == m_images.Number()-1)
+    {
+        m_images.DeleteNode( node );
+        m_images.Append( newBitmap );
+    }
+    else
+    {
+        wxNode *next = node->Next();
+        m_images.DeleteNode( node );
+        m_images.Insert( next, newBitmap );
+    }
+
+    return TRUE;
+}
+
+bool wxImageList::Remove( int index )
+{
+    wxNode *node = m_images.Nth( index );
+
+    wxCHECK_MSG( node, FALSE, wxT("wrong index in image list") );
+
+    m_images.DeleteNode( node );
+
+    return TRUE;
+}
+
+bool wxImageList::RemoveAll()
+{
+    m_images.Clear();
+
+    return TRUE;
+}
+
+bool wxImageList::GetSize( int index, int &width, int &height ) const
+{
+    width = 0;
+    height = 0;
+
+    wxNode *node = m_images.Nth( index );
+
+    wxCHECK_MSG( node, FALSE, wxT("wrong index in image list") );
+

     wxBitmap *bm = (wxBitmap*)node->Data();
     width = bm->GetWidth();
     height = bm->GetHeight();
     wxBitmap *bm = (wxBitmap*)node->Data();
     width = bm->GetWidth();
     height = bm->GetHeight();

+

     return TRUE;
     return TRUE;

-  }
-  else
-  {
-    width = 0;
-    height = 0;
-    return FALSE;
-  };
-};
+}

 
 

-bool wxImageList::Draw( const int index, wxDC &dc, 
-                        const int x, const int y,
-                        const int WXUNUSED(flags), const bool WXUNUSED(solidBackground) )
+bool wxImageList::Draw( int index, wxDC &dc, int x, int y,
+                        int flags, bool WXUNUSED(solidBackground) )

 {
 {

-  wxNode *node = m_images.Nth( index );
-  if (!node) return FALSE;
-  wxBitmap *bm = (wxBitmap*)node->Data();
-  wxIcon *icon = (wxIcon*)bm;
-  dc.DrawIcon( *icon, x, y );
-  return TRUE;
-};
+    wxNode *node = m_images.Nth( index );
+
+    wxCHECK_MSG( node, FALSE, wxT("wrong index in image list") );
+
+    wxBitmap *bm = (wxBitmap*)node->Data();
+
+    if (bm->IsKindOf(CLASSINFO(wxIcon)))
+        dc.DrawIcon( * ((wxIcon*) bm), x, y);
+    else
+        dc.DrawBitmap( *bm, x, y, (flags & wxIMAGELIST_DRAW_TRANSPARENT) > 0 );
+
+    return TRUE;
+}