> I've made a new plugin called Title Switcher which is now available in a

Two comments:

- imho you're initializing stuff which isn't needed in this plugin:

use Slim::Buttons::Home;
use File::Spec::Functions qw(:ALL);
use DBI qw(:sql_types);
use FindBin qw($Bin);
my @pluginDirs = ();

removing them would probably help save a few bytes

- couldn't you use

$client->pluginData($format)

instead of

$clientFormats->{$client->id}->{$format}

This again would help us save a few bytes, especially in cases where  
players are shut down or only temporarily connected.

Michael
_______________________________________________
plugins mailing list
plugins@lists.slimdevices.com
http://lists.slimdevices.com/lists/listinfo/plugins

Reply via email to