eonpatapon / mpDris2

MPRIS V2.1 support for mpd
GNU General Public License v3.0
210 stars 46 forks source link

Refactor cover art detection into a single tag loop #152

Closed lilyinstarlight closed 2 years ago

lilyinstarlight commented 2 years ago

Following the comments in #151, this refactors the cover art detection into a single tag loop. This seems to work in testing after making a small tweak from the version linked in that other PR

Nothing but FLAC has a pictures property from what I can see in Mutagen, so moving it above the loop should not have any regression in functionality, and MP3, OGG Vorbis, and MP4 all have entirely different sets of tags so merging them into the same loop should not have any regression in functionality either