https://bugs.kde.org/show_bug.cgi?id=333745

Tuomas Nurmi <tuo...@norsumanageri.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/mult
                   |                            |imedia/amarok/-/commit/3a33
                   |                            |67e6aeee5a26c4191821055f656
                   |                            |ce7ce3f4a
         Resolution|---                         |FIXED
             Status|CONFIRMED                   |RESOLVED

--- Comment #20 from Tuomas Nurmi <tuo...@norsumanageri.org> ---
Git commit 3a3367e6aeee5a26c4191821055f656ce7ce3f4a by Tuomas Nurmi.
Committed on 06/07/2024 at 16:24.
Pushed by nurmi into branch 'master'.

Improve encoding when exporting playlist files

Try to avoid excessive escaping local paths when exporting playlist files, and
take
one step more to ensure that remote URLs are encoded sensibly.

I tested exporting to each playlist format and both Amarok and VLC seemed to be
able
to open each of the files and URLs on them (including local files with
non-UTF-8
characters and remote streams with spaces in URL etc). Rhythmbox apparently
doesn't
support streams in playlists, but loaded the local files OK, too. In theory,
e.g.
m3u shouldn't be UTF-8 (m3u8 is a separate file format for that), but at least
all
three tested players successfully loaded tracks with paths saved as UTF-8, so
it's
probably OK.

I'm definitely not confident this is perfect now, but I'm quite sure it is
notably
better than it was before.

M  +10   -3    src/core-impl/playlists/types/file/PlaylistFile.cpp

https://invent.kde.org/multimedia/amarok/-/commit/3a3367e6aeee5a26c4191821055f656ce7ce3f4a

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to