On Sunday 01 May 2016 11:15:19 David Faure wrote:
> 
> +    void signalTempoChanged(tempoT);
> +    void signalMidiInLabel(const MappedEvent *event);
> +    void signalMidiOutLabel(const MappedEvent *event);
> +    void signalPlayButtonChecked(bool checked);
> +    void signalRecordButtonChecked(bool checked);
> +    void signalMetronomeButtonChecked(bool checked);

On second thought, this could of course clearly be improved.
This version is pure decoupling of classes, but actual decoupling
of concepts would lead to something like
 void playStatusChanged( some enum with playing, recording, whatever )

Or at least I could name these something like
  signalPlaying, signalRecording, signalMetronomeActive?

-- 
David Faure, fa...@kde.org, http://www.davidfaure.fr
Working on KDE Frameworks 5


------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
_______________________________________________
Rosegarden-devel mailing list
Rosegarden-devel@lists.sourceforge.net - use the link below to unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel

Reply via email to