From 722ed5bed5c8816c57b8c90471ce423548c34151 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Fri, 18 May 2007 16:44:31 +0000 Subject: [PATCH] removed unused statusbar parameter from wxFrameBase::ShowMenuHelp() git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@46113 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- include/wx/frame.h | 7 ++++--- src/common/framecmn.cpp | 4 ++-- src/gtk/mdi.cpp | 2 +- src/gtk1/mdi.cpp | 2 +- 4 files changed, 8 insertions(+), 7 deletions(-) diff --git a/include/wx/frame.h b/include/wx/frame.h index 6740046652..c9bf79c946 100644 --- a/include/wx/frame.h +++ b/include/wx/frame.h @@ -211,9 +211,10 @@ protected: // something changes virtual void PositionStatusBar() { } - // show the help string for this menu item in the given status bar: the - // status bar pointer can be NULL; return true if help was shown - bool ShowMenuHelp(wxStatusBar *statbar, int helpid); + // show the help string for the given menu item using DoGiveHelp() if the + // given item does have a help string, return false if there is no help for + // such item + bool ShowMenuHelp(int helpid); wxStatusBar *m_frameStatusBar; #endif // wxUSE_STATUSBAR diff --git a/src/common/framecmn.cpp b/src/common/framecmn.cpp index 300c0798d2..0c964bb6f4 100644 --- a/src/common/framecmn.cpp +++ b/src/common/framecmn.cpp @@ -253,7 +253,7 @@ void wxFrameBase::UpdateWindowUI(long flags) void wxFrameBase::OnMenuHighlight(wxMenuEvent& event) { #if wxUSE_STATUSBAR - (void)ShowMenuHelp(GetStatusBar(), event.GetMenuId()); + (void)ShowMenuHelp(event.GetMenuId()); #endif // wxUSE_STATUSBAR } @@ -349,7 +349,7 @@ void wxFrameBase::PopStatusText(int number) m_frameStatusBar->PopStatusText(number); } -bool wxFrameBase::ShowMenuHelp(wxStatusBar *WXUNUSED(statbar), int menuId) +bool wxFrameBase::ShowMenuHelp(int menuId) { #if wxUSE_MENUS // if no help string found, we will clear the status bar text diff --git a/src/gtk/mdi.cpp b/src/gtk/mdi.cpp index 1335dd02bb..df17918271 100644 --- a/src/gtk/mdi.cpp +++ b/src/gtk/mdi.cpp @@ -408,7 +408,7 @@ void wxMDIChildFrame::OnMenuHighlight( wxMenuEvent& event ) { #if wxUSE_STATUSBAR wxMDIParentFrame *mdi_frame = (wxMDIParentFrame*)m_parent->GetParent(); - if ( !ShowMenuHelp(mdi_frame->GetStatusBar(), event.GetMenuId()) ) + if ( !ShowMenuHelp(event.GetMenuId()) ) { // we don't have any help text for this item, but may be the MDI frame // does? diff --git a/src/gtk1/mdi.cpp b/src/gtk1/mdi.cpp index 47b3493b95..295aef91fb 100644 --- a/src/gtk1/mdi.cpp +++ b/src/gtk1/mdi.cpp @@ -409,7 +409,7 @@ void wxMDIChildFrame::OnMenuHighlight( wxMenuEvent& event ) { #if wxUSE_STATUSBAR wxMDIParentFrame *mdi_frame = (wxMDIParentFrame*)m_parent->GetParent(); - if ( !ShowMenuHelp(mdi_frame->GetStatusBar(), event.GetMenuId()) ) + if ( !ShowMenuHelp(event.GetMenuId()) ) { // we don't have any help text for this item, but may be the MDI frame // does? -- 2.45.2