search

LMS-Community / slimserver

Server for Squeezebox and compatible players. This server is also called Lyrion Music Server.
https://lyrion.org
Other
1.2k stars 298 forks source link

Drop unused imports from Slim::Web::ImageProxy #1173

Closed mavit closed 1 month ago

mavit commented 1 month ago

Whilst looking at Slim::Web::ImageProxy, I noticed that some of the used packaged weren’t actually used.

There is a command perlimports that can work this out for us, so I have added a configuration file for it. We could run this on everything in Slim/, but it isn’t perfect, and there is a risk it would remove something that we actually need, so I haven’t done that yet.

michaelherger commented 1 month ago

Thanks for this! I'll definitely have to play with this tool. After refactoring one of my plugins I just spent too much time trying to do this kind of cleanup.