- 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;
-};
-
-bool wxImageList::Remove( int index )
+ wxList::compatibility_iterator node = m_images.Item( index );
+
+ wxCHECK_MSG( node, false, wxT("wrong index in image list") );
+
+ wxBitmap* newBitmap = (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)
+ new wxBitmap(bitmap)
+ #else
+ new wxBitmap( (const wxIcon&) bitmap )
+ #endif
+ : new wxBitmap(bitmap) ;
+
+ if (index == (int) m_images.GetCount() - 1)
+ {
+ delete node->GetData();
+ m_images.Erase( node );
+ m_images.Append( newBitmap );
+ }
+ else
+ {
+ wxList::compatibility_iterator next = node->GetNext();
+ delete node->GetData();
+ m_images.Erase( node );
+ m_images.Insert( next, newBitmap );
+ }
+
+ return true;
+}
+
+bool wxGenericImageList::Remove( int index )