[MPlayer-dev-eng] [PATCH] 302 redirected playlist

[Roberto Togni]

On 2004.04.23 05:04, adland wrote:
[...]
> this playlist request is redirected to another URL.
> wget supports this.
> missing was support to handle this in mplayer for the
> playlist
> 
> I created a patch to fix this.
> 
> testing performed on patch
> //missing argument
> mplayer -playlist
> //missing local playlist file
> mplayer -playlist junk
> //local file playlist
> mplayer -playlist CT0408-SHOW.asx
> //302 redirection URL playlist
> mplayer -playlist
> http://cartalk.cars.com/Radio/Show/Audio/200408/wms/CT0408-SHOW.asx
> //404 not found playlist
> mplayer -playlist
> http://www.cartalk.com/Radio/Show/Audio/200408/wms/CT11408-SHOW.asx
> 
> note: file is not a valid asx playlist but rather just
> a plaintext list of 10 URL's

Please confirm the role of varoius changes:
- parser-mpcmd.c patch is to avoid mplayer print help text with empty/ 
bad playlists
- network.c is the real fix for the problem (and it also handles others  
error codes from the server)
- open.c is to stop mplayer from continuing to parse the url after a  
connection failure (why was it commented out? can you guess a reason  
for it?)

I'll commit 1 and 2 tomorrow if noone complain.
I'm not sure about 3, maybe that code was commented out for a reason.

Ciao,
 Roberto