[vlc-devel] commit: NEWS update (Jean-Philippe Andre )

[Pierre d'Herbemont]

On Wed, Jan 28, 2009 at 2:07 PM, Jean-Philippe André <jpeg at dinauz.org> wrote:
> Hi,
>
> On Wed, 28 Jan 2009 12:46:26 +0100, "Pierre d'Herbemont"
> <pdherbemont at free.fr> wrote:
>>> + * On-the-fly zip file decompression and browsing (MRL of the form
>> zip://file.zip|file.avi to specify the file)
>>
>> IMO, the PATH_SEP separator should have been used instead of | which
>> is also a valid char in path.
>>
>> That requires more disk IO to locate the real zip file, but this is
>> stricter.
>>
>> What do you think?
>
> Well, actually, there is a trick in the stream_filter that makes the use
> of | possible: the base path to the zip file (which is not necessarily on
> the
> local filesystem) is encoded (using web chars like %20 for ' ').

Hum, ok. I don't think that really addresses my question. (Remi has a
good follow up on that one ;) )

I was asking about what you think about using '/' (or '\\' on windows)
instead of '|'.

If you don't think that it's a great idea, then do you check if the
full path exists before assuming that | indicates a .zip contents?

for instance, is the zip demux fooled by:
"zip:///home/me/music||video/test.zip"?

Pierre.