[MPlayer-dev-eng] [PATCH] Fallback to playlist after all demuxers

Arpi arpi at thot.banki.hu
Sat Jan 11 22:24:36 CET 2003


Hi,

> this patch adds an fallback to playlist after all demuxers failed ...
> 
> However I think the message: "Sorry, this file format is not 
> recognized/supported" will after that change never appear :-(( because 
> textplain-playlist is too stupid ...
> 
> I vote for an variable to set textplain parsing to optional ... (useful in 
> fallback, not useful when spcifically using -playlist)
> 
> However I don't now what you think about it ?

imho that parse_playtree() thingie should have a parameter like 'int
forced', set to 1 if called with -playlist, and set to 0 if called after
demuxers failed. it shouldn't try unreliable parsers (like the stupid
plaintext-list) for force==0.

also the other parsers should be reviewed and more sanity checks added.

> Index: mplayer.c

ok the patch looks clean.
(as clean as a patch for this mess can be :))


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu


More information about the MPlayer-dev-eng mailing list