Improve handling of ASX playlists

Basically our asx parser is broken.

• It doesn't handle relative links.
• It should also URL encode the hrefs (spaces in the url need to be converted to %0.9.x maintenance etc).
• protocol rollover, authors, title's etc etc aren't handled.
• It is still located in the old demux/m3u.c module instead of modules/demux/playlist/

See this for asx examples.

assigned to @sigmunau

changed the description

added 1 deleted label

changed milestone to %Features paradize

changed milestone to %0.8.5-test3

changed title from ASX playlist parser is broken to Improve handling of ASX playlists

This is a feature request, actually.

changed milestone to %Features paradize

assigned to @hartman and unassigned @sigmunau

Mostly implemented in [15957] Still need to add url_encode and simplify the code a bit more.

Protocol rollover needs support in the core. perhaps by adding the secondary urls to the item by Info or by a variable. That leaves the problem that the core does not know the difference between eof and the difference in errors of the plugins. So we need to focus work there first.

changed milestone to %0.8.6

Actually, we do, we have p_input->b_eof and p_input->b_error, but some plugins don't set it correctly. The other huge problem is that the PS demuxer always kicks in, preventing us from detecting unhandled streams.

Updating.

Original author: tumu

added Type::feature label

changed milestone to %Features paradize

assigned to @ileoo and unassigned @hartman

New asx parser could be easier to write with lua, just needs to test if luas xml-parser handles some oddities that seem to happen in asx-playlist files.

added Component::Demuxers: Playlist label

My partner and I are taking on this enhancement for a school project.

Replying to [comment:13 lxrv22]:

My partner and I are taking on this enhancement for a school project.

Great. porting it to the vlc_xml API would be the best way to clean and improve it.