[vlc-devel] [PATCH] Demux/Playlist: If an URI contains whitespaces, it will now be encoded to %20. Fixes #2924.

[Rémi Denis-Courmont]

Le lundi 2 janvier 2012 22:40:18 Thomas Siegbert, vous avez écrit :
> Am 02.01.2012 10:11, schrieb Rémi Denis-Courmont:
> >    Hello,
> > 
> > On Sunday 01 January 2012, Thomas Siegbert wrote:
> >> Am 01.01.2012 21:11, schrieb Rémi Denis-Courmont:
> >>>    Hello,
> >>> 
> >>> I do not understand what this patch is supposed to do or it is wrong.
> >> 
> >> Replace white spaces in an URL by "%20". Or did I do anything wrong?
> > 
> > Why replace spaces? Then why replaces only spaces?
> 
> I thought over it. Do you think, we should encode the entire URL?

There is no such thing as "encoding an entire URL". A URL is always encoded, 
or it is not an URL.

You could however URL-encode bytes that are neither URL-safe nor URL-reserved. 
That would turn a non-URL URL-lookalike string into a possibly valid URL.

-- 
Rémi Denis-Courmont
http://www.remlab.net/
http://fi.linkedin.com/in/remidenis