Opened 18 years ago
Closed 17 years ago
Last modified 5 years ago
#8476 closed patch
MIDI gain setting for FluidSynth
|Reported by:||eriktorbjorn||Owned by:||sev-|
This patch adds a "MIDI gain" setting, and uses it to control the output level from the FluidSynth MIDI driver. It should apply cleanly to both the trunk and the branch. The main problem with it, as far as I know, is that it won't fit into the MIDI tab with the small version of the GUI.
See bug #1328632.
Ticket imported from: #1336171. Ticket imported from: patches/581.
Change History (14)
by , 18 years ago
comment:1 by , 17 years ago
Not really assigning to Fingolfin in the sense that this is his department, but I would like his input on it since it does involve the GUI...
The main thing that's keeping me from doing anything about this patch is the GUI issues, and that really depends on the future plans for the GUI as well. I.e. is it worth the trouble trying to make a GUI for this setting that also fits in the small version of it?
Maybe I should just commit the non-GUI parts, and document the setting? That way it could even go into 0.8.1 without affecting anything except the FluidSynth driver...
comment:2 by , 17 years ago
comment:3 by , 17 years ago
comment:4 by , 17 years ago
I'd separate the non-GUI and the GUI part, just like you suggested yourself. Commit the non-GUI part, ideally also updating the docs at the same time (README and docs/*.tex stuff).
The new GUI (or should I say, new new GUI :-)) ... well this is pretty much stalled at this time, I am afraid.
I haven't tried your patch yet, but the "Global options" dialog has plenty of space on the MIDI tab. It's the "Edit game" dialog which is too small, but nothing else (or am I missing something). However, that dialog could easily be made a lot bigger. In gui/launcher.cpp, line 154: _h = screenH - 2 * 40; // TODO/FIXME
-> well just change the 40 to e.g. 10 and you gain 60 pixels vertically, which should be enough for your needs, no?
comment:5 by , 17 years ago
Oops, I meant line 160; and you'll also have to adjust _y etc. But the idea should be clear, I hope :-)
comment:6 by , 17 years ago
The non-GUI part has been comitted, with some changes, to both trunk and 0.8 branch. I'll take a look at the GUI part later. Until then, the attached patch is, of course, out of date.
comment:7 by , 17 years ago
Any news on this?
comment:8 by , 17 years ago
Not really. As I said, I added the config setting, but then the great GUI re-working began and it seemed better to wait until things settled down. Though perhaps they have now...
by , 17 years ago
Patch against current SVN
comment:9 by , 17 years ago
Here's an attempt at adding a GUI setting for it.
comment:10 by , 17 years ago
comment:11 by , 17 years ago
|Status:||new → closed|
comment:12 by , 5 years ago
Patch against an October 25 CVS snapshot