[MPlayer-dev-eng] [PATCH] protocol names should not be case sensitive

Ulion ulion2002 at gmail.com
Wed Dec 19 02:28:23 CET 2007


2007/12/17, Ulion <ulion2002 at gmail.com>:
> 2007/12/12, iber at physics.rutgers.edu <iber at physics.rutgers.edu>:
> > Hello,
> >
> > Currently, mplayer thinks that protocol names should all be lower case.
> > There's no strict requirement specified in any standard (as far as I
> > know), and at least one website I know breaks mplayer by using
> > capitalized URLs.
> >
> > Bug behavior: give mplayer a url where the protocol name is capitalized,
> > and it will break. For example:
> >
> >
> > --------------------
> > ilya at aldan:/opt/src/mplayer/mplayer> mplayer
> > HTTP://www.wnyc.org/stream/fm.pls
> > MPlayer dev-SVN-rUNKNOWN-4.1.2 (C) 2000-2007 MPlayer Team
> > CPU: AMD Athlon(tm) XP 2500+ (Family: 6, Model: 10, Stepping: 0)
> > CPUflags:  MMX: 1 MMX2: 1 3DNow: 1 3DNow2: 1 SSE: 1 SSE2: 0
> > Compiled for x86 CPU with extensions: MMX MMX2 3DNow 3DNowEx SSE
> >
> > Playing HTTP://www.wnyc.org/stream/fm.pls.
> > No stream found to handle url HTTP://www.wnyc.org/stream/fm.pls
> >
> >
> > Exiting... (End of file)
> > -------------------
> >
> > The fix is attached.
>
> Protocol name indeed is case-insensitive, this should be fixed, anyone
> care about this?
> If there's no objection, I will apply this patch (without the comment
> in it) in 2 days.

Commited.


-- 
Ulion



More information about the MPlayer-dev-eng mailing list