Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#11208 closed Bug Report - General (fixed)

Streaming does not work in *.pls when linebreaks are '\r\n'

Reported by: beebof@… Owned by: paulh
Priority: minor Milestone: 0.26.1
Component: Plugin - MythMusic Version: Unspecified
Severity: medium Keywords:
Cc: Ticket locked: no

Description

When using streaming radio, *.pls files do not work with line break \r\n. (e.g. provisioning.streamtheworld.com/pls/KNDDFM.pls)

Simple Patch attached.

Attachments (1)

pls.diff (460 bytes) - added by beebof@… 6 years ago.
ignore '\r' in playlists

Download all attachments as: .zip

Change History (5)

Changed 6 years ago by beebof@…

Attachment: pls.diff added

ignore '\r' in playlists

comment:1 Changed 5 years ago by paulh <mythtv@…>

I have updated the pls parsing to use QSettings which eliminates this problem altogether.

https://github.com/paul-h/mythtv/commit/00b80d29bd

comment:2 Changed 5 years ago by Paul Harrison <mythtv@…>

Owner: set to Paul Harrison <mythtv@…>
Resolution: fixed
Status: newclosed

In 8d2d9e92f15653928d9abdf831ecb30eb314b52e/mythtv:

MythMusic: use QSettings for the pls parser

This refactors the playlist parsing to use QSettings for the pls parser instead
of our custom one which has problems with windows style line endings.

Fixes #11208.

comment:3 Changed 5 years ago by Paul Harrison <mythtv@…>

In 571e9c4d96189279e3b936a583df645b5153bb27/mythtv:

MythMusic: use QSettings for the pls parser

This refactors the playlist parsing to use QSettings for the pls parser instead
of our custom one which has problems with windows style line endings.

Fixes #11208.
(cherry picked from commit 8d2d9e92f15653928d9abdf831ecb30eb314b52e)

comment:4 Changed 5 years ago by paulh

Milestone: unknown0.26.1
Owner: changed from Paul Harrison <mythtv@…> to paulh
Note: See TracTickets for help on using tickets.