[vlc-devel] RE : [PATCH] -- still image demux

[Laurent Aimar]

On Sat, Jan 17, 2009, brezhoneg1 wrote:
> 
> 
> > -----Message d'origine-----
> > De : vlc-devel-bounces at videolan.org [mailto:vlc-devel-
> > bounces at videolan.org] De la part de Antoine Cellerier
> > Envoyé : samedi 17 janvier 2009 16:11
> > À : vlc-devel at videolan.org
> > Objet : Re: [vlc-devel] RE : [PATCH] -- still image demux
> > 
> > The fake patch has been applied (well an old version that I changed a
> > bit). Thanks.
> > 
> > I'm kind of against auto image loading by default in the directory://
> > stuff since that would load album art for example.
> > 
> 
> That's okay with me. No problem.
> 
> 
> Just a minor point that still needs some tweaking :
> 
> When explicitly importing images with the qt4 playlist, 
> - regular files are okay
> - directories are not.
> 
> The difference lies in that regular files are added to the playlist
> without any access prefix whereas directories use dir:// that prefixes
> filenames with file://
> 
> file://image.ext either doesn't work at all (e.g png files) or work in a
> very unwelcome manner with a very short display of less than 1 second
> (e.g jpg files).
> 
> A solution could be to either remove the file:// prefix or be more
> specific (fake:// for images, file:// for others) in the directory.c
> file.
> 
> I proposed a patch for directory.c without success. If someone's got a
> better idea to work out this problem, he/she is welcome.
 That might be solved if the fake module is converted to a real demuxer.
It would then load the file and send it to a standard decoder (or it could do
the decoding itself if easier).
 To not lose the decoder fake functionnality, it could still works the old way
when specifically asked by fake://

 So not forced: standard demuxer reading the file itself.
 When forced: like before/now.

-- 
fenrir