Review Request: Initial look into equalizer scritping. Very experimental.

[Myriam Schweingruber]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102798/#review7162
-----------------------------------------------------------


Since it is very experimental this belongs to /playground, please do not submit it for /src

- Myriam Schweingruber


On Oct. 7, 2011, 2:43 a.m., Ryan McCoskrie wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102798/
> -----------------------------------------------------------
> 
> (Updated Oct. 7, 2011, 2:43 a.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Description
> -------
> 
> This patch is the beginning of an effort to make a generic equalizer interface for plugins and scripts to access.
> 
> 0: Gives the EngineController controller class three new functions for examining and choosing the current preset.
> 0: Started to add equalizer support into the class AmarokEngineScript.
> 0: Began work to make the EqualizerDialog class a front end on the equalizer facilities in EngineController.
> 
> WARNING: Adding custom presets with this patch in place is almost certainly not possible.
> 
> 
> Diffs
> -----
> 
>   src/EngineController.h 3da2f23 
>   src/EngineController.cpp 5bea5c6 
>   src/dialogs/EqualizerDialog.h 9dad055 
>   src/dialogs/EqualizerDialog.cpp ddcd300 
>   src/scriptengine/AmarokEngineScript.h f1cdb8c 
>   src/scriptengine/AmarokEngineScript.cpp 395c504 
> 
> Diff: http://git.reviewboard.kde.org/r/102798/diff/diff
> 
> 
> Testing
> -------
> 
> 0: Briefly tested the Amarok.Engine.eqChangeCurrentPreset() function in the script console. Works as expected.
> 0: Amarok.Engine.eqBandsFreq() seems to return no value.
> 
> 
> Thanks,
> 
> Ryan McCoskrie
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20111007/278d2cec/attachment.html>