Firstly, coding standards - but I understand that this is "legacy" code, so that's fine.
Secondly, if you're working on the songs plugin, rather create a new branch off trunk with an appropriate name (like "songsplugin"). It's the reason we went with Bazaar :-)
Looks fine. Just 2 things though.
Firstly, coding standards - but I understand that this is "legacy" code, so that's fine.
Secondly, if you're working on the songs plugin, rather create a new branch off trunk with an appropriate name (like "songsplugin"). It's the reason we went with Bazaar :-)