TheBigW/ParametricEQ — pull request #2 by fossfreedom (2014-02-04 21:55:29+00:00)
Title: removed unneeded args on show_ui, added the latest rb3compat from covera...

...rt-browser and added .gitignore so that compiled files are not committed to git

ok - I didnt get a segmentation fault - just an error about show_ui not having enough arguments. This was resolved by just using *args - you didnt appear to be using any arguments in that function anyway, but you need a variable number of args since the callback function works different between rb2.96 and rb2.99.

Anyway - the plugin and the equalizer dialog appears to display correctly in both versions of rhythmbox ok.

Note - I havent investigated why the sound didnt update with changing the value of any equalizer bands - I presume this is something you are working on?

you perhaps should also think about running 2to3-3.3 on every .py file EXCEPT the rb3compat module - then your plugin will be almost ready for Rhythmbox 3 as well - N.B. you need to change the Loader=python to Loader=python3 in the .plugin file to make the plugin run in RB3.