[yoshimi] Re: V 1.5.7

  • From: Will Godfrey <willgodfrey@xxxxxxxxxxxxxxx>
  • To: yoshimi@xxxxxxxxxxxxx
  • Date: Sat, 10 Mar 2018 08:08:08 +0000

On Sat, 10 Mar 2018 07:55:07 +0100
Ichthyostega <prg@xxxxxxxxxxxxxxx> wrote:

In file included from  
/usr/ports/audio/yoshimi/work/yoshimi-bbcb8a2/src/UI/WidgetPDial.h:34:
/usr/ports/audio/yoshimi/work/yoshimi-bbcb8a2/src/UI/DynamicTooltip.h:52:8:
warning: 'DynTooltip::show' hides overloaded virtual function  
[-Woverloaded-virtual]
 void show(float timeout=0);
      ^
/usr/local/include/FL/Fl_Menu_Window.H:35:8: note: hidden overloaded virtual
function 'Fl_Menu_Window::show' declared here: different number of parameters 

(0 vs 1)
 void show();
      ^  


IMHO this kind of warning should be taken seriously and investigated.
This kind of errors can be quite insidious, because as a result the
wrong overload of a method gets called, which typically does not break
the program, but could cause subtly wrong behavior.

-- Hermann
I've renamed the local declaration to dynshow(float timeout)
This seems to resolve that particular problem.

-- 
Will J Godfrey
http://www.musically.me.uk
Say you have a poem and I have a tune.
Exchange them and we can both have a poem, a tune, and a song.
Yoshimi source code is available from either: 
https://sourceforge.net/projects/yoshimi
Or: https://github.com/Yoshimi/yoshimi
Our list archive is at: https://www.freelists.org/archive/yoshimi
To post, email to yoshimi@xxxxxxxxxxxxx

Other related posts: