I seem to have fixed the DeleteSubGroups() bug - PLEASE CHECK!

[wxWidgets.git] / src / common / tbarsmpl.cpp

diff --git a/src/common/tbarsmpl.cpp b/src/common/tbarsmpl.cpp
index 0e1a739d299db287bc9a0f86f785abadbb181a4d..e973cfdedbbbe6c48a436f63986048e5ababe7b0 100644 (file)
--- a/src/common/tbarsmpl.cpp
+++ b/src/common/tbarsmpl.cpp
@@ -66,6 +66,7 @@ bool wxToolBarSimple::Create(wxWindow *parent, wxWindowID id, const wxPoint& pos
   m_yMargin = 0;
   m_toolPacking = 1;
   m_toolSeparation = 5;
   m_yMargin = 0;
   m_toolPacking = 1;
   m_toolSeparation = 5;

+  SetCursor(*wxSTANDARD_CURSOR);

   
   return TRUE;
 }
   
   return TRUE;
 }


@@ -74,7 +75,7 @@ wxToolBarSimple::~wxToolBarSimple ()
 {
 }
 
 {
 }
 

-void wxToolBarSimple::OnPaint (wxPaintEvent& event)
+void wxToolBarSimple::OnPaint (wxPaintEvent& WXUNUSED(event))

 {
   wxPaintDC dc(this);
   PrepareDC(dc);
 {
   wxPaintDC dc(this);
   PrepareDC(dc);


@@ -102,7 +103,7 @@ void wxToolBarSimple::OnSize ( wxSizeEvent& event )
   wxToolBarBase::OnSize(event);
 }
 
   wxToolBarBase::OnSize(event);
 }
 

-void wxToolBarSimple::OnKillFocus (wxFocusEvent& event)
+void wxToolBarSimple::OnKillFocus (wxFocusEvent& WXUNUSED(event))

 {
   OnMouseEnter(m_pressedTool = m_currentTool = -1);
 }
 {
   OnMouseEnter(m_pressedTool = m_currentTool = -1);
 }


@@ -211,8 +212,10 @@ void wxToolBarSimple::DrawTool(wxDC& dc, wxMemoryDC& memDC, wxToolBarTool *tool)
 
   if (bitmap && bitmap->Ok())
   {
 
   if (bitmap && bitmap->Ok())
   {

+#ifndef __WXGTK__

     if (bitmap->GetPalette())
       memDC.SetPalette(*bitmap->GetPalette());
     if (bitmap->GetPalette())
       memDC.SetPalette(*bitmap->GetPalette());

+#endif

 
     int ax = (int)tool->m_x,
         ay = (int)tool->m_y,
 
     int ax = (int)tool->m_x,
         ay = (int)tool->m_y,


@@ -245,17 +248,19 @@ void wxToolBarSimple::DrawTool(wxDC& dc, wxMemoryDC& memDC, wxToolBarTool *tool)
             &memDC, 0, 0);
     }
     memDC.SelectObject(wxNullBitmap);
             &memDC, 0, 0);
     }
     memDC.SelectObject(wxNullBitmap);

+#ifndef __WXGTK__

     memDC.SetPalette(wxNullPalette);
     memDC.SetPalette(wxNullPalette);

+#endif

   }
   // No second bitmap, so draw a thick line around bitmap, or invert if mono
   else if (tool->m_toggleState)
   {
     bool drawBorder = FALSE;
   }
   // No second bitmap, so draw a thick line around bitmap, or invert if mono
   else if (tool->m_toggleState)
   {
     bool drawBorder = FALSE;

-    #ifdef __X__ // X doesn't invert properly on colour
+#ifdef __X__ // X doesn't invert properly on colour

     drawBorder = wxColourDisplay();
     drawBorder = wxColourDisplay();

-    #else       // Inversion works fine under Windows
+#else       // Inversion works fine under Windows

     drawBorder = FALSE;
     drawBorder = FALSE;

-    #endif
+#endif

 
     if (!drawBorder)
     {
 
     if (!drawBorder)
     {


@@ -314,7 +319,8 @@ void wxToolBarSimple::DrawTool(wxDC& dc, wxMemoryDC& memDC, wxToolBarTool *tool)
 
 void wxToolBarSimple::ToggleTool(int index, bool toggle)
 {
 
 void wxToolBarSimple::ToggleTool(int index, bool toggle)
 {

-  wxNode *node = m_tools.Find((long)index);
+  wxNode *node = (wxNode*) NULL;
+  node = m_tools.Find((long)index);

   if (node)
   {
     wxToolBarTool *tool = (wxToolBarTool *)node->Data();
   if (node)
   {
     wxToolBarTool *tool = (wxToolBarTool *)node->Data();


@@ -339,7 +345,8 @@ void wxToolBarSimple::ToggleTool(int index, bool toggle)
 //
 void wxToolBarSimple::SpringUpButton(int index)
 {
 //
 void wxToolBarSimple::SpringUpButton(int index)
 {

-  wxNode *node=m_tools.Find((long)index);
+  wxNode *node = (wxNode*) NULL;
+  node=m_tools.Find((long)index);

   if (node) {
     wxToolBarTool *tool = (wxToolBarTool *)node->Data();
     if (tool && !tool->m_isToggle && tool->m_toggleState){
   if (node) {
     wxToolBarTool *tool = (wxToolBarTool *)node->Data();
     if (tool && !tool->m_isToggle && tool->m_toggleState){