Looking over the mailing list archives, I was able to refresh my memory as to why I reverted the bank queuing. Programs like QSynth rely on the notion that querying current bank/program numbers assigned to channels will reflect the most recently assigned values. For some reason I opted to revert the queuing behavior, since it went against the mentioned expectation, rather than try and have the state machine reflect the most recent assignment. The right way to fix all this is to re-work the FluidSynth core thread related code to get rid of queuing where possible. As has been discussed a little previously between David and myself, as much of the state machine as possible should be protected by the synth mutex and queuing used only to update the synth core, which would then be purely a voice synthesis engine.
If there is a quick fix though, to resolve the out of order issues AND also work in regards to returning the latest bank/program number when queried, I'm all for that. Simply reverting back to the queued case though, will break QSynth. Cheers, Elimar On Fri, Jun 4, 2010 at 3:06 PM, Elimar Green <elimargr...@gmail.com> wrote: > Ticket reporting on SourceForge is working. I got an update. Just haven't > gotten around to researching why I originally changed it from queuing > program changes. I'm sure there was some reason to that, but event ordering > issues are obviously not acceptable. I'll try and get around to reviewing > this soon. Perhaps there was some mailing list traffic around revision 277 > that would clarify why the queuing got reverted. > > Regards, > > Elimar > > > On Fri, Jun 4, 2010 at 2:01 PM, Graham Goode <ggoode...@gmail.com> wrote: > >> Hi Guys, >> >> Sven Meier of the jOrgan project has made some fixes for fluidsynth >> 1.1.1 (Tickets 65 and 79). We as jOrgan users have his patched >> fluidsynth version but it looks like these have not been accepted into >> the SourceForge repository yet. Is there someone looking into these or >> is sourceforge not reporting ticket changes yet? >> >> GrahamG >> >> _______________________________________________ >> fluid-dev mailing list >> fluid-dev@nongnu.org >> http://lists.nongnu.org/mailman/listinfo/fluid-dev >> > >
_______________________________________________ fluid-dev mailing list fluid-dev@nongnu.org http://lists.nongnu.org/mailman/listinfo/fluid-dev
