Opened 14 years ago
Closed 14 years ago
#2766 closed enhancement (fixed)
FreeboxChannelFetcher::ParsePlaylist does not parse dos text files with CRLF
Reported by: | anonymous | Owned by: | danielk |
---|---|---|---|
Priority: | trivial | Milestone: | 0.21 |
Component: | mythtv | Version: | head |
Severity: | low | Keywords: | |
Cc: | Ticket locked: | no |
Description
If a playlist.m3u file contains CRLF instead of simply LF for new lines then it is not parsed and gives the message: Invalid channel list header ...
It would be best if ParsePlaylist? could handle files with CRLF as well as LF for new lines.
Attachments (1)
Change History (6)
comment:1 Changed 14 years ago by
Owner: | changed from Isaac Richards to danielk |
---|
comment:2 Changed 14 years ago by
Type: | enhancement → defect |
---|
comment:3 Changed 14 years ago by
Milestone: | unknown → 0.21 |
---|---|
Type: | defect → enhancement |
Version: | 0.20 → head |
Verified problem exists in SVN head. I've attached a potential fix, I'm not sure it is robust enough (what is the newline convention on Macs these days?)
comment:4 Changed 14 years ago by
The patch looks simple enough and ought to work just fine as I read the code. I'm not sure I'll have time to test it though.
Note: See
TracTickets for help on using
tickets.
Have you tested the parser in SVN?
It has changed since 0.20, but I don't know if this issue was addressed.