search

DeaDBeeF-Player / deadbeef

DeaDBeeF Player
https://deadbeef.sourceforge.io/
Other
1.65k stars 178 forks source link

fix #3127 (autosort) #3128

Closed rsekman closed 2 weeks ago

rsekman commented 2 months ago

This PR is a draft to resolve #3127. It adds tests for sorting playlists directly and with autosort, plus some tweaks to the test suite (build with debuginfo, link with lld for better error messages).

Oleksiy-Yakovenko commented 3 weeks ago

It looks good, but I can't merge this PR before the tests are green. If you need to have the failing test in the code -- you'd have to disable it before merging.

Oleksiy-Yakovenko commented 3 weeks ago

I have played around with calling plt_ausort, and I can confirm it's quite broken. Every time it's called, it corrupts the previous settings.

Oleksiy-Yakovenko commented 3 weeks ago

The tests should be green after you rebase on master.