[yoshimi] Re: V 1.5.7

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

On Sat, 10 Mar 2018 16:12:24 +0800
blubee blubeeme <gurenchan@xxxxxxxxx> wrote:

On Sat, Mar 10, 2018 at 4:08 PM, Will Godfrey <willgodfrey@xxxxxxxxxxxxxxx>
wrote:

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
 
I just realized why the copyfile.h error is happening.

You guys implemented the new feature and uses sys/sendfile on Linux and
added the stub for FreeBSD based on copyfile.

I am not sure if we have that header but basically you're looking to copy
files.

Do you plan on using all the features defined in sys/sendfile; like the
different copying methods or just COPYFILE_ALL.

If it's just copyfile_all, I can just write a simple copy file function and
call it a day. If you'll be using more advanced features I'll have to look
deeper to see what's available on *BSD.

Best,
Owen


Ha! Semms our posts overlapped.

We are selecting files to copy, not just doing a directory dump.
As the bank files are comparatively small and are all text ones, pairing up
those load and save text functions might be good enough. After all, when being
used, they are bulk loaded into memory.

Will

-- 
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: