[MPlayer-dev-eng] Re: [PATCH] fix for issue cant play filenames with spaces

[adland]

> Please Adland test it and report. 
 
I did retest this patch update change alone and report that it
 was not working - see this thread.

however reported that some parts of it do need inserted related to memory usage.

I tested this with my change as well while I was at it.

> Please, do NOT commit the hack in the url.c that Adland commit earlier. 

okay but this was the one patch that worked.
can you explain why you need to always escape % 
even if part of an escape sequence already?
ie why %20 becomes %2520? this fails on my and with Compn tests.

before % was never escaped, now it always is,
however this patch escapes only if not part of previous escape sequences 
which is between the two extremes.

thanks