]> git.saurik.com Git - wxWidgets.git/blobdiff - include/wx/mac/carbon/window.h
correct access for virtuals
[wxWidgets.git] / include / wx / mac / carbon / window.h
index 97d3c3cfd789ffc3dff2cafedee401931385e233..363767bdfcce6b6c7124ad3f0a36a320e3d961fe 100644 (file)
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////
-// Name:        window.h
+// Name:        include/wx/mac/carbon/window.h
 // Purpose:     wxWindowMac class
 // Author:      Stefan Csomor
 // Modified by:
 // Purpose:     wxWindowMac class
 // Author:      Stefan Csomor
 // Modified by:
 #ifndef _WX_WINDOW_H_
 #define _WX_WINDOW_H_
 
 #ifndef _WX_WINDOW_H_
 #define _WX_WINDOW_H_
 
-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-#pragma interface "window.h"
-#endif
-
 #include "wx/brush.h"
 #include "wx/dc.h"
 
 #include "wx/brush.h"
 #include "wx/dc.h"
 
-// ---------------------------------------------------------------------------
-// forward declarations
-// ---------------------------------------------------------------------------
-
 class WXDLLEXPORT wxButton;
 class WXDLLEXPORT wxScrollBar;
 class WXDLLEXPORT wxTopLevelWindowMac;
 
 class WXDLLEXPORT wxButton;
 class WXDLLEXPORT wxScrollBar;
 class WXDLLEXPORT wxTopLevelWindowMac;
 
-// internal implementation classes
-
 class wxMacControl ;
 
 class wxMacControl ;
 
-// ---------------------------------------------------------------------------
-// constants
-// ---------------------------------------------------------------------------
-
 class WXDLLEXPORT wxWindowMac: public wxWindowBase
 {
     DECLARE_DYNAMIC_CLASS(wxWindowMac)
 class WXDLLEXPORT wxWindowMac: public wxWindowBase
 {
     DECLARE_DYNAMIC_CLASS(wxWindowMac)
-        
+
     friend class wxDC;
     friend class wxPaintDC;
     friend class wxDC;
     friend class wxPaintDC;
-    
+
 public:
 public:
-    
-    wxWindowMac() ;
+    wxWindowMac();
 
 
-    wxWindowMac(wxWindowMac *parent,
+    wxWindowMac( wxWindowMac *parent,
                 wxWindowID id,
                 const wxPoint& pos = wxDefaultPosition,
                 const wxSize& size = wxDefaultSize,
                 long style = 0,
                 wxWindowID id,
                 const wxPoint& pos = wxDefaultPosition,
                 const wxSize& size = wxDefaultSize,
                 long style = 0,
-                const wxString& name = wxPanelNameStr;
+                const wxString& name = wxPanelNameStr );
 
     virtual ~wxWindowMac();
 
 
     virtual ~wxWindowMac();
 
-    bool Create(wxWindowMac *parent,
+    bool Create( wxWindowMac *parent,
                 wxWindowID id,
                 const wxPoint& pos = wxDefaultPosition,
                 const wxSize& size = wxDefaultSize,
                 long style = 0,
                 wxWindowID id,
                 const wxPoint& pos = wxDefaultPosition,
                 const wxSize& size = wxDefaultSize,
                 long style = 0,
-                const wxString& name = wxPanelNameStr);
-
+                const wxString& name = wxPanelNameStr );
 
     // implement base class pure virtuals
 
     // implement base class pure virtuals
-    virtual void SetTitle( const wxString& title);
-    virtual wxString GetTitle() const;
+    virtual void SetLabel( const wxString& label );
+    virtual wxString GetLabel() const;
 
     virtual void Raise();
     virtual void Lower();
 
 
     virtual void Raise();
     virtual void Lower();
 
-    virtual bool Show( bool show = TRUE );
-    virtual bool Enable( bool enable = TRUE );
+    virtual bool Show( bool show = true );
+    virtual bool Enable( bool enable = true );
 
     virtual void SetFocus();
 
 
     virtual void SetFocus();
 
-    virtual void WarpPointer(int x, int y);
+    virtual void WarpPointer( int x, int y );
 
 
-    virtual void Refresh( bool eraseBackground = TRUE,
-                          const wxRect *rect = (const wxRect *) NULL );
-    virtual void Freeze() ;
-    virtual void Thaw() ;
+    virtual void Refresh( bool eraseBackground = true,
+                          const wxRect *rect = NULL );
+    virtual void Freeze();
+    virtual void Thaw();
 
     virtual void Update() ;
 
     virtual void Update() ;
-    virtual void ClearBackground() ;
+    virtual void ClearBackground();
 
     virtual bool SetCursor( const wxCursor &cursor );
 
     virtual bool SetCursor( const wxCursor &cursor );
-    virtual bool SetFont( const wxFont &font ) ;
+    virtual bool SetFont( const wxFont &font );
     virtual bool SetBackgroundColour( const wxColour &colour );
     virtual bool SetForegroundColour( const wxColour &colour );
 
     virtual int GetCharHeight() const;
     virtual int GetCharWidth() const;
     virtual bool SetBackgroundColour( const wxColour &colour );
     virtual bool SetForegroundColour( const wxColour &colour );
 
     virtual int GetCharHeight() const;
     virtual int GetCharWidth() const;
-    virtual void GetTextExtent(const wxString& string,
+    virtual void GetTextExtent( const wxString& string,
                                int *x, int *y,
                                int *x, int *y,
-                               int *descent = (int *) NULL,
-                               int *externalLeading = (int *) NULL,
-                               const wxFont *theFont = (const wxFont *) NULL)
+                               int *descent = NULL,
+                               int *externalLeading = NULL,
+                               const wxFont *theFont = NULL )
                                const;
                                const;
-
+protected:
     virtual bool DoPopupMenu( wxMenu *menu, int x, int y );
     virtual bool DoPopupMenu( wxMenu *menu, int x, int y );
-
+public:
     virtual void SetScrollbar( int orient, int pos, int thumbVisible,
     virtual void SetScrollbar( int orient, int pos, int thumbVisible,
-                               int range, bool refresh = TRUE );
-    virtual void SetScrollPos( int orient, int pos, bool refresh = TRUE );
+                               int range, bool refresh = true );
+    virtual void SetScrollPos( int orient, int pos, bool refresh = true );
     virtual int GetScrollPos( int orient ) const;
     virtual int GetScrollThumb( int orient ) const;
     virtual int GetScrollRange( int orient ) const;
     virtual void ScrollWindow( int dx, int dy,
                                const wxRect* rect = (wxRect *) NULL );
     virtual int GetScrollPos( int orient ) const;
     virtual int GetScrollThumb( int orient ) const;
     virtual int GetScrollRange( int orient ) const;
     virtual void ScrollWindow( int dx, int dy,
                                const wxRect* rect = (wxRect *) NULL );
+    virtual bool Reparent( wxWindowBase *newParent );
 
 #if wxUSE_DRAG_AND_DROP
     virtual void SetDropTarget( wxDropTarget *dropTarget );
 
 #if wxUSE_DRAG_AND_DROP
     virtual void SetDropTarget( wxDropTarget *dropTarget );
-#endif // wxUSE_DRAG_AND_DROP
+#endif
 
     // Accept files for dragging
 
     // Accept files for dragging
-    virtual void DragAcceptFiles(bool accept);
+    virtual void DragAcceptFiles( bool accept );
 
     // implementation from now on
     // --------------------------
 
 
     // implementation from now on
     // --------------------------
 
-    void MacClientToRootWindow( int *x , int *y ) const ;
-    void MacRootWindowToClient( int *x , int *y ) const ;
+    void MacClientToRootWindow( int *x , int *y ) const;
+    void MacRootWindowToClient( int *x , int *y ) const;
+
+    void MacWindowToRootWindow( int *x , int *y ) const;
+    void MacWindowToRootWindow( short *x , short *y ) const;
 
 
-    void MacWindowToRootWindow( int *x , int *y ) const ;
-    void MacWindowToRootWindow( short *x , short *y ) const ;
+    void MacRootWindowToWindow( int *x , int *y ) const;
+    void MacRootWindowToWindow( short *x , short *y ) const;
 
 
-    void MacRootWindowToWindow( int *x , int *y ) const ;
-    void MacRootWindowToWindow( short *x , short *y ) const ;
-    
-    virtual wxString MacGetToolTipString( wxPoint &where ) ;
+    virtual wxString MacGetToolTipString( wxPoint &where );
 
     // simple accessors
     // ----------------
 
 
     // simple accessors
     // ----------------
 
-    virtual WXWidget GetHandle() const ;
+    virtual WXWidget GetHandle() const;
 
 #if WXWIN_COMPATIBILITY_2_4
     bool GetTransparentBackground() const { return m_backgroundTransparent; }
 
 #if WXWIN_COMPATIBILITY_2_4
     bool GetTransparentBackground() const { return m_backgroundTransparent; }
-    void SetTransparent(bool t = TRUE) { m_backgroundTransparent = t; }
+    void SetTransparent(bool t = true) { m_backgroundTransparent = t; }
 #endif
 
     // event handlers
     // --------------
 #endif
 
     // event handlers
     // --------------
-    void OnSetFocus(wxFocusEvent& event) ;
-    void OnPaint(wxPaintEvent& event);
-    void OnNcPaint(wxNcPaintEvent& event);
-    void OnEraseBackground(wxEraseEvent& event);
-    void OnMouseEvent( wxMouseEvent &event ) ;
+    void OnSetFocus( wxFocusEvent& event );
+    void OnPaint( wxPaintEvent& event );
+    void OnNcPaint( wxNcPaintEvent& event );
+    void OnEraseBackground(wxEraseEvent& event );
+    void OnMouseEvent( wxMouseEvent &event );
+
+    void MacOnScroll( wxScrollEvent&event );
 
 
-    void MacOnScroll(wxScrollEvent&event ) ;
-    
-    bool AcceptsFocus() const ;
+    bool AcceptsFocus() const;
+
+public:
+    static long         MacRemoveBordersFromStyle( long style ) ;
 
 public:
     void OnInternalIdle();
 
 
 public:
     void OnInternalIdle();
 
-    // For implementation purposes - sometimes decorations make the client area
-    // smaller
+    // For implementation purposes:
+    // sometimes decorations make the client area smaller
     virtual wxPoint GetClientAreaOrigin() const;
 
     wxWindowMac *FindItem(long id) const;
     virtual wxPoint GetClientAreaOrigin() const;
 
     wxWindowMac *FindItem(long id) const;
-    wxWindowMac *FindItemByHWND(WXHWND hWnd, bool controlOnly = FALSE) const;
+    wxWindowMac *FindItemByHWND(WXHWND hWnd, bool controlOnly = false) const;
 
 
-public:
     virtual void        MacHandleControlClick( WXWidget control , wxInt16 controlpart , bool mouseStillDown ) ;
     virtual bool        MacDoRedraw( WXHRGN updatergn , long time ) ;
     virtual bool        MacCanFocus() const ;
     virtual void        MacHandleControlClick( WXWidget control , wxInt16 controlpart , bool mouseStillDown ) ;
     virtual bool        MacDoRedraw( WXHRGN updatergn , long time ) ;
     virtual bool        MacCanFocus() const ;
-    
-    // this should not be overriden in classes above wxWindowMac because it is called from its destructor via DeleteChildren
+
+    // this should not be overriden in classes above wxWindowMac
+    // because it is called from its destructor via DeleteChildren
     virtual void        RemoveChild( wxWindowBase *child );
     virtual void        MacPaintBorders( int left , int top ) ;
     virtual void        RemoveChild( wxWindowBase *child );
     virtual void        MacPaintBorders( int left , int top ) ;
-    // invalidates the borders and focus area around the control
+
+    // invalidates the borders and focus area around the control;
     // must not be virtual as it will be called during destruction
     void                MacInvalidateBorders() ;
     // must not be virtual as it will be called during destruction
     void                MacInvalidateBorders() ;
-    WXWindow            MacGetTopLevelWindowRef() const  ;
+
+    WXWindow            MacGetTopLevelWindowRef() const ;
     wxTopLevelWindowMac* MacGetTopLevelWindow() const ;
     wxTopLevelWindowMac* MacGetTopLevelWindow() const ;
-    
+
     virtual long        MacGetLeftBorderSize() const ;
     virtual long        MacGetRightBorderSize() const ;
     virtual long        MacGetTopBorderSize() const ;
     virtual long        MacGetBottomBorderSize() const ;
     virtual long        MacGetLeftBorderSize() const ;
     virtual long        MacGetRightBorderSize() const ;
     virtual long        MacGetTopBorderSize() const ;
     virtual long        MacGetBottomBorderSize() const ;
-    
-    static long         MacRemoveBordersFromStyle( long style ) ;
-    
+
     virtual void        MacSuperChangedPosition() ;
     virtual void        MacSuperChangedPosition() ;
-    // the absolute coortinates of this window's root have changed
+
+    // absolute coordinates of this window's root have changed
     virtual void        MacTopLevelWindowChangedPosition() ;
     virtual void        MacTopLevelWindowChangedPosition() ;
-    
+
     virtual void        MacChildAdded() ;
     virtual void        MacVisibilityChanged() ;
     virtual void        MacEnabledStateChanged() ;
     virtual void        MacHiliteChanged() ;
     virtual wxInt32     MacControlHit( WXEVENTHANDLERREF handler , WXEVENTREF event ) ;
     virtual void        MacChildAdded() ;
     virtual void        MacVisibilityChanged() ;
     virtual void        MacEnabledStateChanged() ;
     virtual void        MacHiliteChanged() ;
     virtual wxInt32     MacControlHit( WXEVENTHANDLERREF handler , WXEVENTREF event ) ;
-    
+
     bool                MacIsReallyShown() ;
     bool                MacIsReallyEnabled() ;
     bool                MacIsReallyHilited() ;
     bool                MacIsReallyShown() ;
     bool                MacIsReallyEnabled() ;
     bool                MacIsReallyHilited() ;
-    
-    bool                       MacIsUserPane() { return m_macIsUserPane; }
+
+    bool                MacIsUserPane() { return m_macIsUserPane; }
 
     virtual bool        MacSetupCursor( const wxPoint& pt ) ;
 
     virtual bool        MacSetupCursor( const wxPoint& pt ) ;
-        
     virtual void        MacSetBackgroundBrush( const wxBrush &brush ) ;
     const wxBrush&      MacGetBackgroundBrush() const { return m_macBackgroundBrush ; }
     virtual void        MacSetBackgroundBrush( const wxBrush &brush ) ;
     const wxBrush&      MacGetBackgroundBrush() const { return m_macBackgroundBrush ; }
-    
+
+    // return the rectangle that would be visible of this control,
+    // regardless whether controls are hidden
+    // only taking into account clipping by parent windows
+    const wxRect&       MacGetClippedClientRect() const ;
+    const wxRect&       MacGetClippedRect() const ;
+    const wxRect&       MacGetClippedRectWithOuterStructure() const ;
+
     // returns the visible region of this control in window ie non-client coordinates
     // returns the visible region of this control in window ie non-client coordinates
-    
-    wxRegion            MacGetVisibleRegion( bool includeOuterStructures = false ) ;
-    // returns true if children have to clipped to the content area (eg scrolled window)
-    virtual bool               MacClipChildren() const { return false ; }
-    // returns true if the grandchildren have to be clipped to the children's content area (eg
-    // splitter window)
-    virtual bool               MacClipGrandChildren() const { return false ; }
+    const wxRegion&     MacGetVisibleRegion( bool includeOuterStructures = false ) ;
+
+    // returns true if children have to clipped to the content area
+    // (e.g., scrolled windows)
+    bool                MacClipChildren() const { return m_clipChildren ; }
+    void                MacSetClipChildren( bool clip ) { m_clipChildren = clip ; }
+
+    // returns true if the grandchildren need to be clipped to the children's content area
+    // (e.g., splitter windows)
+    virtual bool        MacClipGrandChildren() const { return false ; }
     bool                MacIsWindowScrollbar( const wxScrollBar* sb )
     bool                MacIsWindowScrollbar( const wxScrollBar* sb )
-        { return (m_hScrollBar == sb || m_vScrollBar == sb) ; }
+    { return (m_hScrollBar == sb || m_vScrollBar == sb) ; }
 
 
+    virtual void        MacInstallEventHandler(WXWidget native) ;
+    void                MacPostControlCreate(const wxPoint& pos, const wxSize& size) ;
     wxList&             GetSubcontrols() { return m_subControls; }
     wxList&             GetSubcontrols() { return m_subControls; }
-    virtual void               MacInstallEventHandler(WXWidget native) ;
     WXEVENTHANDLERREF   MacGetControlEventHandler() { return m_macControlEventHandler ; }
     WXEVENTHANDLERREF   MacGetControlEventHandler() { return m_macControlEventHandler ; }
-    void                MacPostControlCreate(const wxPoint& pos, const wxSize& size) ;
+
 #ifndef __WXMAC_OSX__
     virtual void            MacControlUserPaneDrawProc(wxInt16 part) ;
     virtual wxInt16         MacControlUserPaneHitTestProc(wxInt16 x, wxInt16 y) ;
 #ifndef __WXMAC_OSX__
     virtual void            MacControlUserPaneDrawProc(wxInt16 part) ;
     virtual wxInt16         MacControlUserPaneHitTestProc(wxInt16 x, wxInt16 y) ;
@@ -230,51 +229,79 @@ public:
     virtual wxInt16         MacControlUserPaneFocusProc(wxInt16 action) ;
     virtual void            MacControlUserPaneBackgroundProc(void* info) ;
 #endif
     virtual wxInt16         MacControlUserPaneFocusProc(wxInt16 action) ;
     virtual void            MacControlUserPaneBackgroundProc(void* info) ;
 #endif
-    // translate wxWidgets coords into ones suitable to be passed to
-    // the CreateControl calls
+
+    // translate wxWidgets coords into ones suitable
+    // to be passed to CreateControl calls
     //
     //
-    // returns TRUE if non default coords are returned, FALSE otherwise
+    // returns true if non-default coords are returned, false otherwise
     bool                MacGetBoundsForControl(const wxPoint& pos,
                                            const wxSize& size,
                                            int& x, int& y,
                                            int& w, int& h , bool adjustForOrigin ) const ;
     bool                MacGetBoundsForControl(const wxPoint& pos,
                                            const wxSize& size,
                                            int& x, int& y,
                                            int& w, int& h , bool adjustForOrigin ) const ;
+
     // calculates the real window position and size from the native control
     void                MacGetPositionAndSizeFromControl(int& x, int& y,
                                            int& w, int& h) const ;
     // calculates the real window position and size from the native control
     void                MacGetPositionAndSizeFromControl(int& x, int& y,
                                            int& w, int& h) const ;
+
     // gets the inset from every part
     virtual void        MacGetContentAreaInset( int &left , int &top , int &right , int &bottom ) ;
 
     // gets the inset from every part
     virtual void        MacGetContentAreaInset( int &left , int &top , int &right , int &bottom ) ;
 
-    // flash the current invalid area, useful for debugging in OSX double buffered situation
+    // visibly flash the current invalid area:
+    // useful for debugging in OSX composited (double-buffered) situation
     void                MacFlashInvalidAreas() ;
 
     // the 'true' OS level control for this wxWindow
     wxMacControl*       GetPeer() const { return m_peer ; }
     void                MacFlashInvalidAreas() ;
 
     // the 'true' OS level control for this wxWindow
     wxMacControl*       GetPeer() const { return m_peer ; }
+
 #if wxMAC_USE_CORE_GRAPHICS
     void *              MacGetCGContextRef() { return m_cgContextRef ; }
     void                MacSetCGContextRef(void * cg) { m_cgContextRef = cg ; }
 #endif
 #if wxMAC_USE_CORE_GRAPHICS
     void *              MacGetCGContextRef() { return m_cgContextRef ; }
     void                MacSetCGContextRef(void * cg) { m_cgContextRef = cg ; }
 #endif
+
 protected:
 protected:
-    // For controls like radiobuttons which are really composite
+    // For controls like radio buttons which are genuinely composite
     wxList              m_subControls;
     wxList              m_subControls;
+
     // number of calls to Freeze() minus number of calls to Thaw()
     unsigned int        m_frozenness;
     // number of calls to Freeze() minus number of calls to Thaw()
     unsigned int        m_frozenness;
+
     // the peer object, allowing for cleaner API support
     // the peer object, allowing for cleaner API support
-    wxMacControl*       m_peer ;
+    wxMacControl *       m_peer ;
+
 #if wxMAC_USE_CORE_GRAPHICS
     void *              m_cgContextRef ;
 #endif
 #if wxMAC_USE_CORE_GRAPHICS
     void *              m_cgContextRef ;
 #endif
+
+    // cache the clipped rectangles within the window hierarchy
+    void                MacUpdateClippedRects() const ;
+
+    mutable bool        m_cachedClippedRectValid ;
+    mutable wxRect      m_cachedClippedRectWithOuterStructure ;
+    mutable wxRect      m_cachedClippedRect ;
+    mutable wxRect      m_cachedClippedClientRect ;
+    mutable wxRegion    m_cachedClippedRegionWithOuterStructure ;
+    mutable wxRegion    m_cachedClippedRegion ;
+    mutable wxRegion    m_cachedClippedClientRegion ;
+
     // true if is is not a native control but a wxWindow control
     // true if is is not a native control but a wxWindow control
-       bool                            m_macIsUserPane ;
+    bool                m_macIsUserPane ;
     wxBrush             m_macBackgroundBrush ;
     wxBrush             m_macBackgroundBrush ;
-    // topleft inset of the mac control from the wx top left corner
+
+    // insets of the mac control from the wx top left corner
     wxPoint             m_macTopLeftInset ;
     wxPoint             m_macTopLeftInset ;
-    // bottom right inset of the mac control from the wx bottom right corner
     wxPoint             m_macBottomRightInset ;
     wxPoint             m_macBottomRightInset ;
-    
+
     wxScrollBar*        m_hScrollBar ;
     wxScrollBar*        m_vScrollBar ;
     wxString            m_label ;
     wxScrollBar*        m_hScrollBar ;
     wxScrollBar*        m_vScrollBar ;
     wxString            m_label ;
-    
+
+    // set to true if we do a sharp clip at the content area of this window
+    // must be dynamic as eg a panel normally is not clipping precisely, but if
+    // it becomes the target window of a scrolled window it has to...
+    bool                m_clipChildren ;
+
+    virtual bool        MacIsChildOfClientArea( const wxWindow* child ) const ;
+
     void                MacCreateScrollBars( long style ) ;
     void                MacRepositionScrollBars() ;
     void                MacUpdateControlFont() ;
     void                MacCreateScrollBars( long style ) ;
     void                MacRepositionScrollBars() ;
     void                MacUpdateControlFont() ;
@@ -282,15 +309,14 @@ protected:
     void                MacPropagateVisibilityChanged() ;
     void                MacPropagateEnabledStateChanged() ;
     void                MacPropagateHiliteChanged() ;
     void                MacPropagateVisibilityChanged() ;
     void                MacPropagateEnabledStateChanged() ;
     void                MacPropagateHiliteChanged() ;
-    
+
 #if WXWIN_COMPATIBILITY_2_4
     bool                 m_backgroundTransparent ;
 #endif
 
     // implement the base class pure virtuals
 #if WXWIN_COMPATIBILITY_2_4
     bool                 m_backgroundTransparent ;
 #endif
 
     // implement the base class pure virtuals
-       virtual wxSize DoGetBestSize() const;
-       virtual wxSize DoGetSizeFromClientSize( const wxSize & size ) const;
+    virtual wxSize DoGetBestSize() const;
+    virtual wxSize DoGetSizeFromClientSize( const wxSize & size ) const;
     virtual void DoClientToScreen( int *x, int *y ) const;
     virtual void DoScreenToClient( int *x, int *y ) const;
     virtual void DoGetPosition( int *x, int *y ) const;
     virtual void DoClientToScreen( int *x, int *y ) const;
     virtual void DoScreenToClient( int *x, int *y ) const;
     virtual void DoGetPosition( int *x, int *y ) const;
@@ -308,15 +334,12 @@ protected:
     // from both DoSetSize() and DoSetClientSize() and would usually just call
     // ::MoveWindow() except for composite controls which will want to arrange
     // themselves inside the given rectangle
     // from both DoSetSize() and DoSetClientSize() and would usually just call
     // ::MoveWindow() except for composite controls which will want to arrange
     // themselves inside the given rectangle
-    virtual void DoMoveWindow(int x, int y, int width, int height);
-    virtual void DoSetWindowVariant( wxWindowVariant variant ) ;
+    virtual void DoMoveWindow( int x, int y, int width, int height );
+    virtual void DoSetWindowVariant( wxWindowVariant variant );
 
 #if wxUSE_TOOLTIPS
     virtual void DoSetToolTip( wxToolTip *tip );
 
 #if wxUSE_TOOLTIPS
     virtual void DoSetToolTip( wxToolTip *tip );
-#endif // wxUSE_TOOLTIPS
-
-private :
+#endif
 
 private:
     // common part of all ctors
 
 private:
     // common part of all ctors
@@ -324,10 +347,8 @@ private:
 
     WXEVENTHANDLERREF    m_macControlEventHandler ;
 
 
     WXEVENTHANDLERREF    m_macControlEventHandler ;
 
-
     DECLARE_NO_COPY_CLASS(wxWindowMac)
     DECLARE_EVENT_TABLE()
 };
 
     DECLARE_NO_COPY_CLASS(wxWindowMac)
     DECLARE_EVENT_TABLE()
 };
 
-#endif
-    // _WX_WINDOW_H_
+#endif // _WX_WINDOW_H_