From 548fa9c1eb08b38b561b7861e1cf93a2ce78772a Mon Sep 17 00:00:00 2001 From: =?utf8?q?V=C3=A1clav=20Slav=C3=ADk?= Date: Tue, 1 Mar 2011 12:16:49 +0000 Subject: [PATCH] Pass wxRect/wxPoint arguments to wxDataViewCustomRenderer by reference. Instead of passing them by value, use const reference. This change is safe to do, because these methods' signatures were different in 2.8 anyhow. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@67099 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- docs/changes.txt | 4 ++++ docs/doxygen/overviews/changes_since28.h | 6 ++++++ include/wx/dvrenderers.h | 10 +++++----- include/wx/generic/dvrenderer.h | 6 +++--- include/wx/generic/dvrenderers.h | 12 ++++++------ interface/wx/dataview.h | 9 +++++---- samples/dataview/dataview.cpp | 5 +++-- src/generic/datavgen.cpp | 6 +++--- 8 files changed, 35 insertions(+), 23 deletions(-) diff --git a/docs/changes.txt b/docs/changes.txt index 46761a3..8687861 100644 --- a/docs/changes.txt +++ b/docs/changes.txt @@ -184,6 +184,10 @@ Changes in behaviour not resulting in compilation errors, please read this! wxAutomationInstance_SilentIfNone flag to prevent the error message if no currently running instances of this object are available. +- Signatures of wxDataViewCustomRenderer::Activate(), LeftClick() and + StartDrag() virtual methods changed. You will need to change them in your + derived renderer class too if you override them. + Changes in behaviour which may result in compilation errors ----------------------------------------------------------- diff --git a/docs/doxygen/overviews/changes_since28.h b/docs/doxygen/overviews/changes_since28.h index 5fc6678..8ec1c9f 100644 --- a/docs/doxygen/overviews/changes_since28.h +++ b/docs/doxygen/overviews/changes_since28.h @@ -176,5 +176,11 @@ Finally, a few structure fields, notable @c wxCmdLineEntryDesc::shortName, wxScrolled::GetSizeAvailableForScrollTarget() method to compute the size available for the scroll target as function of the main window size, please see the documentation of this method for more details. + +- Signatures of wxDataViewCustomRenderer::Activate(), + wxDataViewCustomRenderer::LeftClick() and + wxDataViewCustomRenderer::StartDrag() virtual methods changed. You will need + to change them in your derived renderer class too if you override them. + */ diff --git a/include/wx/dvrenderers.h b/include/wx/dvrenderers.h index 97539bf..a25fec9 100644 --- a/include/wx/dvrenderers.h +++ b/include/wx/dvrenderers.h @@ -230,21 +230,21 @@ public: // to drag it: by default they all simply return false indicating that the // events are not handled - virtual bool Activate(wxRect WXUNUSED(cell), + virtual bool Activate(const wxRect& WXUNUSED(cell), wxDataViewModel *WXUNUSED(model), const wxDataViewItem & WXUNUSED(item), unsigned int WXUNUSED(col)) { return false; } - virtual bool LeftClick(wxPoint WXUNUSED(cursor), - wxRect WXUNUSED(cell), + virtual bool LeftClick(const wxPoint& WXUNUSED(cursor), + const wxRect& WXUNUSED(cell), wxDataViewModel *WXUNUSED(model), const wxDataViewItem & WXUNUSED(item), unsigned int WXUNUSED(col) ) { return false; } - virtual bool StartDrag(wxPoint WXUNUSED(cursor), - wxRect WXUNUSED(cell), + virtual bool StartDrag(const wxPoint& WXUNUSED(cursor), + const wxRect& WXUNUSED(cell), wxDataViewModel *WXUNUSED(model), const wxDataViewItem & WXUNUSED(item), unsigned int WXUNUSED(col) ) diff --git a/include/wx/generic/dvrenderer.h b/include/wx/generic/dvrenderer.h index 3914e83..aa7f615 100644 --- a/include/wx/generic/dvrenderer.h +++ b/include/wx/generic/dvrenderer.h @@ -47,14 +47,14 @@ public: // generic implementation uses these ones for all of them, including the // standard ones. - virtual bool WXOnActivate(wxRect WXUNUSED(cell), + virtual bool WXOnActivate(const wxRect& WXUNUSED(cell), wxDataViewModel *WXUNUSED(model), const wxDataViewItem & WXUNUSED(item), unsigned int WXUNUSED(col)) { return false; } - virtual bool WXOnLeftClick(wxPoint WXUNUSED(cursor), - wxRect WXUNUSED(cell), + virtual bool WXOnLeftClick(const wxPoint& WXUNUSED(cursor), + const wxRect& WXUNUSED(cell), wxDataViewModel *WXUNUSED(model), const wxDataViewItem & WXUNUSED(item), unsigned int WXUNUSED(col) ) diff --git a/include/wx/generic/dvrenderers.h b/include/wx/generic/dvrenderers.h index dc1bc24..f9ae414 100644 --- a/include/wx/generic/dvrenderers.h +++ b/include/wx/generic/dvrenderers.h @@ -26,7 +26,7 @@ public: // see the explanation of the following WXOnXXX() methods in wx/generic/dvrenderer.h - virtual bool WXOnActivate(wxRect cell, + virtual bool WXOnActivate(const wxRect& cell, wxDataViewModel *model, const wxDataViewItem& item, unsigned int col) @@ -34,8 +34,8 @@ public: return Activate(cell, model, item, col); } - virtual bool WXOnLeftClick(wxPoint cursor, - wxRect cell, + virtual bool WXOnLeftClick(const wxPoint& cursor, + const wxRect& cell, wxDataViewModel *model, const wxDataViewItem &item, unsigned int col) @@ -121,8 +121,8 @@ public: wxSize GetSize() const; // Implementation only, don't use nor override - virtual bool WXOnLeftClick(wxPoint cursor, - wxRect cell, + virtual bool WXOnLeftClick(const wxPoint& cursor, + const wxRect& cell, wxDataViewModel *model, const wxDataViewItem& item, unsigned int col); @@ -206,7 +206,7 @@ public: virtual wxSize GetSize() const; // Implementation only, don't use nor override - virtual bool WXOnActivate(wxRect cell, + virtual bool WXOnActivate(const wxRect& cell, wxDataViewModel *model, const wxDataViewItem& item, unsigned int col); diff --git a/interface/wx/dataview.h b/interface/wx/dataview.h index f8961f8..57d066e 100644 --- a/interface/wx/dataview.h +++ b/interface/wx/dataview.h @@ -1571,7 +1571,7 @@ public: Override this to react to double clicks or ENTER. This method will only be called in wxDATAVIEW_CELL_ACTIVATABLE mode. */ - virtual bool Activate( wxRect cell, + virtual bool Activate( const wxRect& cell, wxDataViewModel* model, const wxDataViewItem & item, unsigned int col ); @@ -1640,8 +1640,8 @@ public: Override this to react to a left click. This method will only be called in @c wxDATAVIEW_CELL_ACTIVATABLE mode. */ - virtual bool LeftClick( wxPoint cursor, - wxRect cell, + virtual bool LeftClick( const wxPoint& cursor, + const wxRect& cell, wxDataViewModel * model, const wxDataViewItem & item, unsigned int col ); @@ -1665,7 +1665,8 @@ public: /** Override this to start a drag operation. Not yet supported. */ - virtual bool StartDrag(wxPoint cursor, wxRect cell, + virtual bool StartDrag(const wxPoint& cursor, + const wxRect& cell, wxDataViewModel* model, const wxDataViewItem & item, unsigned int col); diff --git a/samples/dataview/dataview.cpp b/samples/dataview/dataview.cpp index ed3d828..b9d4246 100644 --- a/samples/dataview/dataview.cpp +++ b/samples/dataview/dataview.cpp @@ -189,7 +189,7 @@ public: return true; } - virtual bool Activate( wxRect WXUNUSED(cell), + virtual bool Activate( const wxRect& WXUNUSED(cell), wxDataViewModel *WXUNUSED(model), const wxDataViewItem &WXUNUSED(item), unsigned int WXUNUSED(col) ) @@ -198,7 +198,8 @@ public: return false; } - virtual bool LeftClick( wxPoint cursor, wxRect WXUNUSED(cell), + virtual bool LeftClick(const wxPoint& cursor, + const wxRect& WXUNUSED(cell), wxDataViewModel *WXUNUSED(model), const wxDataViewItem &WXUNUSED(item), unsigned int WXUNUSED(col) ) diff --git a/src/generic/datavgen.cpp b/src/generic/datavgen.cpp index eb1b741..d6ff164 100644 --- a/src/generic/datavgen.cpp +++ b/src/generic/datavgen.cpp @@ -863,8 +863,8 @@ bool wxDataViewToggleRenderer::Render( wxRect cell, wxDC *dc, int WXUNUSED(state return true; } -bool wxDataViewToggleRenderer::WXOnLeftClick(wxPoint WXUNUSED(cursor), - wxRect WXUNUSED(cell), +bool wxDataViewToggleRenderer::WXOnLeftClick(const wxPoint& WXUNUSED(cursor), + const wxRect& WXUNUSED(cell), wxDataViewModel *model, const wxDataViewItem& item, unsigned int col) @@ -1027,7 +1027,7 @@ wxSize wxDataViewDateRenderer::GetSize() const return GetTextExtent(m_date.FormatDate()); } -bool wxDataViewDateRenderer::WXOnActivate(wxRect WXUNUSED(cell), +bool wxDataViewDateRenderer::WXOnActivate(const wxRect& WXUNUSED(cell), wxDataViewModel *model, const wxDataViewItem& item, unsigned int col) -- 2.7.4