Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
Merge pull request #222 from FluidSynth/cmd-handler
shell command handler decoupled
- Loading branch information
Showing
11 changed files
with
528 additions
and
466 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
cb036d2There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@derselbst a quick question: as I'm currently doing the LADSPA cleanup and refactor, I also have to fix the 5 ladspa command handlers to use this new interface. I'm thinking of moving the command handling stuff to fluid_cmd.c, calling into functions in fluid_ladspa.c for the real work. Does that sound good to you?
cb036d2There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh good point. Yes please do so. Also it seems that adjusting their signature was completely forgotten by this commit due to that error hiding cast in
fluid_commands[]. Highly appreciated.cb036d2There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I'll leave the source formatting for later after all, doing more of a refactor now. I guess it would be good to get this fixed sooner rather than later.
cb036d2There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No hurry, this commit is only on master, this has time until next major release (not this year, maybe next).
cb036d2There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, that's good to know, thanks! I'll take more time then and (try to) do it properly.