[yoshimi] Re: V 1.5.7

  • From: blubee blubeeme <gurenchan@xxxxxxxxx>
  • To: yoshimi@xxxxxxxxxxxxx
  • Date: Sat, 10 Mar 2018 16:12:24 +0800

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

Other related posts:

  1. Home
  2. yoshimi
  3. Archive
  4. 03-2018