Handle cases where an index larger than GetItemCount() is passed into InsertItem.

[wxWidgets.git] / src / mac / carbon / tooltip.cpp

diff --git a/src/mac/carbon/tooltip.cpp b/src/mac/carbon/tooltip.cpp
index 7751eee526ce3349d396dc2bb413fc8ed012c148..122c76b623d0c5574b8aaf8f80ad7a8dee97d872 100644 (file)
--- a/src/mac/carbon/tooltip.cpp
+++ b/src/mac/carbon/tooltip.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        tooltip.cpp
+// Name:        src/mac/carbon/tooltip.cpp

 // Purpose:     wxToolTip implementation
 // Author:      Stefan Csomor
 // Id:          $Id$
 // Purpose:     wxToolTip implementation
 // Author:      Stefan Csomor
 // Id:          $Id$


@@ -11,11 +11,15 @@
 
 #if wxUSE_TOOLTIPS
 
 
 #if wxUSE_TOOLTIPS
 

-#include "wx/app.h"
-#include "wx/dc.h"
-#include "wx/window.h"

 #include "wx/tooltip.h"
 #include "wx/tooltip.h"

-#include "wx/timer.h"
+
+#ifndef WX_PRECOMP
+    #include "wx/app.h"
+    #include "wx/window.h"
+    #include "wx/dc.h"
+    #include "wx/timer.h"
+#endif // WX_PRECOMP
+

 #include "wx/geometry.h"
 #include "wx/mac/uma.h"
 
 #include "wx/geometry.h"
 #include "wx/mac/uma.h"
 


@@ -27,41 +31,48 @@ class wxMacToolTipTimer ;
 
 class wxMacToolTip
 {
 
 class wxMacToolTip
 {

-    public :
-        wxMacToolTip( ) ;
-        ~wxMacToolTip() ;
-        
-        void            Setup( WindowRef window  , const wxString& text , wxPoint localPosition ) ;
-        long            GetMark() { return m_mark ; }
-        void             Draw() ;
-        void            Clear() ;
-        bool            IsShown() { return m_shown  ; }
-    private :
-        
-        wxString    m_label ;
-        wxPoint m_position ;
-        Rect            m_rect ;
-        WindowRef    m_window ;    
-        PicHandle    m_backpict ;
-        bool        m_shown ;
-        long        m_mark ;
-        wxMacToolTipTimer* m_timer ;
+public :
+    wxMacToolTip() ;
+    ~wxMacToolTip() ;
+
+    void Setup( WindowRef window , const wxString& text , const wxPoint& localPosition ) ;
+    void Draw() ;
+    void Clear() ;
+
+    long GetMark()
+    { return m_mark ; }
+
+    bool IsShown()
+    { return m_shown ; }
+
+private :
+    wxString    m_label ;
+    wxPoint m_position ;
+    Rect            m_rect ;
+    WindowRef    m_window ;
+    PicHandle    m_backpict ;
+    bool        m_shown ;
+    long        m_mark ;
+    wxMacToolTipTimer* m_timer ;
+

 #if TARGET_CARBON
 #if TARGET_CARBON

-        wxMacCFStringHolder m_helpTextRef ;
+    wxMacCFStringHolder m_helpTextRef ;

 #endif
 } ;
 
 class wxMacToolTipTimer : public wxTimer
 {
 public:
 #endif
 } ;
 
 class wxMacToolTipTimer : public wxTimer
 {
 public:

-    wxMacToolTipTimer() {} ;

     wxMacToolTipTimer(wxMacToolTip* tip, int iMilliseconds) ;
     wxMacToolTipTimer(wxMacToolTip* tip, int iMilliseconds) ;

+    wxMacToolTipTimer() {} ;

     virtual ~wxMacToolTipTimer() {} ;
     virtual ~wxMacToolTipTimer() {} ;

+

     void Notify()
     {
         if ( m_mark == m_tip->GetMark() )
     void Notify()
     {
         if ( m_mark == m_tip->GetMark() )

-            m_tip->Draw() ;    
+            m_tip->Draw() ;

     }
     }

+

 protected:
     wxMacToolTip*     m_tip;
     long        m_mark ;
 protected:
     wxMacToolTip*     m_tip;
     long        m_mark ;


@@ -79,6 +90,7 @@ static WindowRef s_ToolTipWindowRef = NULL ;
 
 IMPLEMENT_ABSTRACT_CLASS(wxToolTip, wxObject)
 
 
 IMPLEMENT_ABSTRACT_CLASS(wxToolTip, wxObject)
 

+

 wxToolTip::wxToolTip( const wxString &tip )
 {
     m_text = tip;
 wxToolTip::wxToolTip( const wxString &tip )
 {
     m_text = tip;


@@ -92,16 +104,16 @@ wxToolTip::~wxToolTip()
 void wxToolTip::SetTip( const wxString &tip )
 {
     m_text = tip;
 void wxToolTip::SetTip( const wxString &tip )
 {
     m_text = tip;

-    
+

     if ( m_window )
     {
     if ( m_window )
     {

-    /*
+#if 0

     // update it immediately
     wxToolInfo ti(GetHwndOf(m_window));
     ti.lpszText = (wxChar *)m_text.c_str();
     // update it immediately
     wxToolInfo ti(GetHwndOf(m_window));
     ti.lpszText = (wxChar *)m_text.c_str();

-    
-      (void)SendTooltipMessage(GetToolTipCtrl(), TTM_UPDATETIPTEXT, 0, &ti);
-        */
+
+    (void)SendTooltipMessage(GetToolTipCtrl(), TTM_UPDATETIPTEXT, 0, &ti);
+#endif

     }
 }
 
     }
 }
 


@@ -115,6 +127,7 @@ void wxToolTip::Enable( bool flag )
     if ( s_ShowToolTips != flag )
     {
         s_ShowToolTips = flag ;
     if ( s_ShowToolTips != flag )
     {
         s_ShowToolTips = flag ;

+

         if ( s_ShowToolTips )
         {
         }
         if ( s_ShowToolTips )
         {
         }


@@ -149,7 +162,7 @@ void wxToolTip::RelayEvent( wxWindow *win , wxMouseEvent &event )
                 s_ToolTip.Clear() ;
                 s_ToolTipArea = wxRect2DInt( event.m_x - 2 , event.m_y - 2 , 4 , 4 ) ;
                 s_LastWindowEntered = win ;
                 s_ToolTip.Clear() ;
                 s_ToolTipArea = wxRect2DInt( event.m_x - 2 , event.m_y - 2 , 4 , 4 ) ;
                 s_LastWindowEntered = win ;

-                
+

                 WindowRef window = MAC_WXHWND( win->MacGetTopLevelWindowRef() ) ;
                 int x = event.m_x ;
                 int y = event.m_y ;
                 WindowRef window = MAC_WXHWND( win->MacGetTopLevelWindowRef() ) ;
                 int x = event.m_x ;
                 int y = event.m_y ;


@@ -166,6 +179,7 @@ void wxToolTip::RemoveToolTips()
 {
     s_ToolTip.Clear() ;
 }
 {
     s_ToolTip.Clear() ;
 }

+

 // --- mac specific
 
 wxMacToolTipTimer::wxMacToolTipTimer( wxMacToolTip *tip , int msec )
 // --- mac specific
 
 wxMacToolTipTimer::wxMacToolTipTimer( wxMacToolTip *tip , int msec )


@@ -179,32 +193,37 @@ wxMacToolTip::wxMacToolTip()
 {
     m_window = NULL ;
     m_backpict = NULL ;
 {
     m_window = NULL ;
     m_backpict = NULL ;

+    m_timer = NULL ;

     m_mark = 0 ;
     m_shown = false ;
     m_mark = 0 ;
     m_shown = false ;

-    m_timer = NULL ;

 }
 
 }
 

-void wxMacToolTip::Setup( WindowRef win  , const wxString& text , wxPoint localPosition ) 
+void wxMacToolTip::Setup( WindowRef win  , const wxString& text , const wxPoint& localPosition )

 {
     m_mark++ ;
 {
     m_mark++ ;

+

     Clear() ;
     m_position = localPosition ;
     m_label = text ;
     m_window =win;
     s_ToolTipWindowRef = m_window ;
     m_backpict = NULL ;
     Clear() ;
     m_position = localPosition ;
     m_label = text ;
     m_window =win;
     s_ToolTipWindowRef = m_window ;
     m_backpict = NULL ;

+

     if ( m_timer )
         delete m_timer ;
     if ( m_timer )
         delete m_timer ;

+

     m_timer = new wxMacToolTipTimer( this , s_ToolTipDelay ) ;
 }
 
     m_timer = new wxMacToolTipTimer( this , s_ToolTipDelay ) ;
 }
 

-wxMacToolTip::~wxMacToolTip() 
+wxMacToolTip::~wxMacToolTip()

 {
 {

-    if ( m_timer ) {
+    if ( m_timer )
+    {

         delete m_timer ;
         m_timer = NULL;
     }
         delete m_timer ;
         m_timer = NULL;
     }

-    if ( m_backpict ) 
+
+    if ( m_backpict )

         Clear() ;
 }
 
         Clear() ;
 }
 


@@ -213,12 +232,13 @@ const short kTipOffset = 5 ;
 
 void wxMacToolTip::Draw()
 {
 
 void wxMacToolTip::Draw()
 {

-    if ( m_label.Length() == 0 )
+    if ( m_label.empty() )

         return ;
         return ;

-    
+

     if ( m_window == s_ToolTipWindowRef )
     {
         m_shown = true ;
     if ( m_window == s_ToolTipWindowRef )
     {
         m_shown = true ;

+

 #if TARGET_CARBON
         HMHelpContentRec tag ;
         tag.version = kMacHelpVersion;
 #if TARGET_CARBON
         HMHelpContentRec tag ;
         tag.version = kMacHelpVersion;


@@ -226,7 +246,7 @@ void wxMacToolTip::Draw()
 
         QDLocalToGlobalRect( GetWindowPort( m_window ) , &tag.absHotRect ) ;
 
 
         QDLocalToGlobalRect( GetWindowPort( m_window ) , &tag.absHotRect ) ;
 

-        m_helpTextRef.Assign( m_label  , wxFONTENCODING_DEFAULT ) ;
+        m_helpTextRef.Assign( m_label , wxFONTENCODING_DEFAULT ) ;

         tag.content[kHMMinimumContentIndex].contentType = kHMCFStringContent ;
         tag.content[kHMMinimumContentIndex].u.tagCFString = m_helpTextRef ;
         tag.content[kHMMaximumContentIndex].contentType = kHMCFStringContent ;
         tag.content[kHMMinimumContentIndex].contentType = kHMCFStringContent ;
         tag.content[kHMMinimumContentIndex].u.tagCFString = m_helpTextRef ;
         tag.content[kHMMaximumContentIndex].contentType = kHMCFStringContent ;


@@ -234,14 +254,14 @@ void wxMacToolTip::Draw()
         tag.tagSide = kHMDefaultSide;
         HMDisplayTag( &tag );
 #else
         tag.tagSide = kHMDefaultSide;
         HMDisplayTag( &tag );
 #else

-        wxMacPortStateHelper help( (GrafPtr) GetWindowPort( m_window ) );
+        wxMacPortStateHelper help( (GrafPtr)GetWindowPort( m_window ) );

         FontFamilyID fontId ;
         Str255 fontName ;
         SInt16 fontSize ;
         Style fontStyle ;
         FontFamilyID fontId ;
         Str255 fontName ;
         SInt16 fontSize ;
         Style fontStyle ;

-        GetThemeFont(kThemeSmallSystemFont , GetApplicationScript() , fontName , &fontSize , &fontStyle ) ;
+        GetThemeFont( kThemeSmallSystemFont , GetApplicationScript() , fontName , &fontSize , &fontStyle ) ;

         GetFNum( fontName, &fontId );
         GetFNum( fontName, &fontId );

-        
+

         TextFont( fontId ) ;
         TextSize( fontSize ) ;
         TextFace( fontStyle ) ;
         TextFont( fontId ) ;
         TextSize( fontSize ) ;
         TextFace( fontStyle ) ;


@@ -249,27 +269,29 @@ void wxMacToolTip::Draw()
         ::GetFontInfo(&fontInfo);
         short lineh = fontInfo.ascent + fontInfo.descent + fontInfo.leading;
         short height = 0 ;
         ::GetFontInfo(&fontInfo);
         short lineh = fontInfo.ascent + fontInfo.descent + fontInfo.leading;
         short height = 0 ;

-        
+

         int i = 0 ;
         int i = 0 ;

-        int length = m_label.Length() ;
+        int length = m_label.length() ;

         int width = 0 ;
         int thiswidth = 0 ;
         int laststop = 0 ;
         int width = 0 ;
         int thiswidth = 0 ;
         int laststop = 0 ;

-        wxCharBuffer text = m_label.mb_str( wxConvLocal)  ;
+        wxCharBuffer text = m_label.mb_str( wxConvLocal ) ;

 
 

-        while( i < length )
+        while ( i < length )

         {
         {

-            if( text[i] == 13 || text[i] == 10)
+            if ( text[i] == 13 || text[i] == 10)

             {
                 thiswidth = ::TextWidth( text , laststop , i - laststop ) ;
                 if ( thiswidth > width )
                     width = thiswidth ;
             {
                 thiswidth = ::TextWidth( text , laststop , i - laststop ) ;
                 if ( thiswidth > width )
                     width = thiswidth ;

-                
+

                 height += lineh ;
                 height += lineh ;

-                laststop = i+1 ;
+                laststop = i + 1 ;

             }
             }

+

             i++ ;
         }
             i++ ;
         }

+

         if ( i - laststop > 0 )
         {
             thiswidth = ::TextWidth( text , laststop , i - laststop ) ;
         if ( i - laststop > 0 )
         {
             thiswidth = ::TextWidth( text , laststop , i - laststop ) ;


@@ -277,13 +299,14 @@ void wxMacToolTip::Draw()
                 width = thiswidth ;
             height += lineh ;
         }
                 width = thiswidth ;
             height += lineh ;
         }

-        
+

         m_rect.left = m_position.x + kTipOffset;
         m_rect.top = m_position.y + kTipOffset;
         m_rect.right = m_rect.left + width + 2 * kTipBorder;
 
         m_rect.bottom = m_rect.top + height + 2 * kTipBorder;
         Rect r ;
         m_rect.left = m_position.x + kTipOffset;
         m_rect.top = m_position.y + kTipOffset;
         m_rect.right = m_rect.left + width + 2 * kTipBorder;
 
         m_rect.bottom = m_rect.top + height + 2 * kTipBorder;
         Rect r ;

+

         GetPortBounds( GetWindowPort( m_window ) , &r ) ;
         if ( m_rect.top < 0 )
         {
         GetPortBounds( GetWindowPort( m_window ) , &r ) ;
         if ( m_rect.top < 0 )
         {


@@ -305,97 +328,101 @@ void wxMacToolTip::Draw()
             m_rect.top -= (m_rect.bottom - r.bottom) ;
             m_rect.bottom = r.bottom ;
         }
             m_rect.top -= (m_rect.bottom - r.bottom) ;
             m_rect.bottom = r.bottom ;
         }

+
+        GWorldPtr port ;
+        CGrafPtr  origPort ;
+        GDHandle  origDevice ;
+

         ClipRect( &m_rect ) ;
         BackColor( whiteColor ) ;
         ForeColor(blackColor ) ;
         ClipRect( &m_rect ) ;
         BackColor( whiteColor ) ;
         ForeColor(blackColor ) ;

-        GWorldPtr port ;            

         NewGWorld( &port , wxDisplayDepth() , &m_rect , NULL , NULL , 0 ) ;
         NewGWorld( &port , wxDisplayDepth() , &m_rect , NULL , NULL , 0 ) ;

-        CGrafPtr    origPort ;
-        GDHandle    origDevice ;
-        
+

         GetGWorld( &origPort , &origDevice ) ;
         SetGWorld( port , NULL ) ;
         GetGWorld( &origPort , &origDevice ) ;
         SetGWorld( port , NULL ) ;

-        
+

         m_backpict = OpenPicture(&m_rect);
         m_backpict = OpenPicture(&m_rect);

-        
-        CopyBits(GetPortBitMapForCopyBits(GetWindowPort(m_window)), 
-            GetPortBitMapForCopyBits(port), 
-            &m_rect, 
-            &m_rect, 
-            srcCopy, 
+
+        CopyBits(GetPortBitMapForCopyBits(GetWindowPort(m_window)),
+            GetPortBitMapForCopyBits(port),
+            &m_rect,
+            &m_rect,
+            srcCopy,

             NULL);
         ClosePicture();
         SetGWorld( origPort , origDevice ) ;
         DisposeGWorld( port ) ;
         PenNormal() ;
             NULL);
         ClosePicture();
         SetGWorld( origPort , origDevice ) ;
         DisposeGWorld( port ) ;
         PenNormal() ;

-        
+

         RGBColor tooltipbackground = { 0xFFFF , 0xFFFF , 0xC000 } ;
         BackColor( whiteColor ) ;
         RGBForeColor( &tooltipbackground ) ;
         RGBColor tooltipbackground = { 0xFFFF , 0xFFFF , 0xC000 } ;
         BackColor( whiteColor ) ;
         RGBForeColor( &tooltipbackground ) ;

-        
+

         PaintRect( &m_rect ) ;
         ForeColor(blackColor ) ;
         FrameRect( &m_rect ) ;
         SetThemeTextColor(kThemeTextColorNotification,wxDisplayDepth(),true) ;
         ::MoveTo( m_rect.left + kTipBorder , m_rect.top + fontInfo.ascent + kTipBorder);
         PaintRect( &m_rect ) ;
         ForeColor(blackColor ) ;
         FrameRect( &m_rect ) ;
         SetThemeTextColor(kThemeTextColorNotification,wxDisplayDepth(),true) ;
         ::MoveTo( m_rect.left + kTipBorder , m_rect.top + fontInfo.ascent + kTipBorder);

-        
+

         i = 0 ;
         laststop = 0 ;
         height = 0 ;
         i = 0 ;
         laststop = 0 ;
         height = 0 ;

-        
-        while( i < length )
+
+        while ( i < length )

         {
         {

-            if( text[i] == 13 || text[i] == 10)
+            if ( text[i] == 13 || text[i] == 10)

             {
                 ::DrawText( text , laststop , i - laststop ) ;
                 height += lineh ;
                 ::MoveTo( m_rect.left + kTipBorder , m_rect.top + fontInfo.ascent + kTipBorder + height );
                 laststop = i+1 ;
             }
             {
                 ::DrawText( text , laststop , i - laststop ) ;
                 height += lineh ;
                 ::MoveTo( m_rect.left + kTipBorder , m_rect.top + fontInfo.ascent + kTipBorder + height );
                 laststop = i+1 ;
             }

+

             i++ ;
         }
             i++ ;
         }

+

         ::DrawText( text , laststop , i - laststop ) ;
         ::DrawText( text , laststop , i - laststop ) ;

-        ::TextMode( srcOr ) ;        
+        ::TextMode( srcOr ) ;

 #endif
     }
 }
 
 #endif
     }
 }
 

-void wxToolTip::NotifyWindowDelete( WXHWND win ) 
+void wxToolTip::NotifyWindowDelete( WXHWND win )

 {
     if ( win == s_ToolTipWindowRef )
 {
     if ( win == s_ToolTipWindowRef )

-    {

         s_ToolTipWindowRef = NULL ;
         s_ToolTipWindowRef = NULL ;

-    }

 }
 
 void wxMacToolTip::Clear()
 {
     m_mark++ ;
 }
 
 void wxMacToolTip::Clear()
 {
     m_mark++ ;

+

     if ( m_timer )
     {
         delete m_timer ;
         m_timer = NULL ;
     }
     if ( m_timer )
     {
         delete m_timer ;
         m_timer = NULL ;
     }

+

     if ( !m_shown )
         return ;
     if ( !m_shown )
         return ;

+

 #if TARGET_CARBON
     HMHideTag() ;
     m_helpTextRef.Release() ;
 #if TARGET_CARBON
     HMHideTag() ;
     m_helpTextRef.Release() ;

-#else         
+#else

     if ( m_window == s_ToolTipWindowRef && m_backpict )
     {
     if ( m_window == s_ToolTipWindowRef && m_backpict )
     {

-        wxMacPortStateHelper help( (GrafPtr) GetWindowPort(m_window) ) ;
+        wxMacPortStateHelper help( (GrafPtr)GetWindowPort( m_window ) ) ;

 
         m_shown = false ;
 
         BackColor( whiteColor ) ;
 
         m_shown = false ;
 
         BackColor( whiteColor ) ;

-        ForeColor(blackColor ) ;
-        DrawPicture(m_backpict, &m_rect);
-        KillPicture(m_backpict);
+        ForeColor( blackColor ) ;
+        DrawPicture( m_backpict, &m_rect );
+        KillPicture( m_backpict );

         m_backpict = NULL ;
     }
 #endif
 }
 
         m_backpict = NULL ;
     }
 #endif
 }
 

-#endif
-
+#endif // wxUSE_TOOLTIPS