search

LMS-Community / slimserver

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

Release types $pt refactoring #1084

Closed darrell-k closed 1 month ago

darrell-k commented 1 month ago

Before looking at backporting the recent changes to 8.5, I have attempted to tidy up: I didn't like the dclone I introduced very much, so replaced with direct manipulation of $pt->{searchTags}.

For me, at least, this is now cleaner and easier to understand.

Thoughts?

michaelherger commented 1 month ago

This certainly looks a lot cleaner! Thanks! Maybe we should leave it alone for a few days to see whether there's new feedback before trying to back port?