On Sat, 10 Mar 2018 17:41:19 +0800
blubee blubeeme <gurenchan@xxxxxxxxx> wrote:
On Sat, Mar 10, 2018 at 5:03 PM, Will Godfrey <willgodfrey@xxxxxxxxxxxxxxx>
wrote:
On Sat, 10 Mar 2018 16:57:04 +0800I just did a quick test and it does compile after making a few edits.
blubee blubeeme <gurenchan@xxxxxxxxx> wrote:
On Sat, Mar 10, 2018 at 4:46 PM, blubee blubeeme <gurenchan@xxxxxxxxx>willgodfrey@xxxxxxxxxxxxxxx
wrote:
On Sat, Mar 10, 2018 at 4:40 PM, Will Godfrey <
overloadedwrote:
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:PDial.h:34:
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/Widget
DynamicTooltip.h:52:8:/usr/ports/audio/yoshimi/work/yoshimi-bbcb8a2/src/UI/
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
investigated.virtual
parametersfunction 'Fl_Menu_Window::show' declared here: different number of
(0 vs 1)
void show();
^
IMHO this kind of warning should be taken seriously and
breakThis kind of errors can be quite insidious, because as a result the
wrong overload of a method gets called, which typically does not
https://sourceforge.net/ ;the program, but could cause subtly wrong behavior.I've renamed the local declaration to dynshow(float timeout)
-- Hermann
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:
andprojects/yoshimiI just realized why the copyfile.h error is happening.
Or: https://github.com/Yoshimi/yoshimi
Our list archive is at: https://www.freelists.org/archive/yoshimi
To post, email to yoshimi@xxxxxxxxxxxxx
You guys implemented the new feature and uses sys/sendfile on Linux
copyadded the stub for FreeBSD based on copyfile.
I am not sure if we have that header but basically you're looking to
thefiles.
Do you plan on using all the features defined in sys/sendfile; like
functiondifferent copying methods or just COPYFILE_ALL.
If it's just copyfile_all, I can just write a simple copy file
pairingand
call it a day. If you'll be using more advanced features I'll have tolook
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,
whenup
those load and save text functions might be good enough. After all,
Actually a little more digging; FreeBSD has sendfile: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
I was looking at the MiscFuncs.cpp
Removing the copyfile.h from the __FreeBSD__ ifdef and I get compilation
errors about the fcopy command around line 357.
I don't think that you'll need to write a whole new implementation.
There's actually quite a few copyfiles that I've seen so far.
There's this one on SO:https://stackoverflow.com/a/2180788
There's also a FreeBSD github repo with a copyfile implementation:
https://github.com/freebsd/freebsd/blob/master/bin/cp/utils.c#L78
if you're not doing a lot of copying one of these might be good enough,
maybe?
Best,
Owen
https://www.freebsd.org/cgi/man.cgi?query=sendfile&sektion=2
Maybe we can avert all that re-implementing all together!
That's why I suggested commenting out those lines to see if it would work
on
local files.
--
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
Everything compiles.
There's a few differences between sendfile and FreeBSD implementation as
this one requires 7 arguments
Other than a few changes the program compiles to completion.
I've attached a small diff file to show the changes.