[vlc-devel] commit: Increased fifo decoder size to 400mb. (Laurent Aimar )

[Laurent Aimar]

Hi

On Sat, Oct 24, 2009, Marian Ďurkovič wrote:
> On Fri, 23 Oct 2009 23:14:52 +0200 (CEST), git version control wrote
> > vlc | branch: 1.0-bugfix | Laurent Aimar <fenrir at videolan.org> | Wed 
> > Jul 15 21:23:56 2009 +0200| [eafadf15e9fe064c404c985c1fd8e0a183d30eb9] 
> > | committer: Laurent Aimar
> > 
> > Increased fifo decoder size to 400mb.
> > 
> >  I don't think there is any good fixed value, I have tuned it for
> > 50mbit/s video with a cache of 60s.
> 
> Is this one not too dangerous for stable branch? I mean if 1.0.3 might require
> ~500 MB of real free memory, that's a *huge* jump from 1.0.2 and might prevent
> VLC from working in some setups. Yes, RAM is cheap today, but laptops with
> <= 1 GB RAM are still in common use.
 It has already been discussed on this ML.
 This commit does not really change VLC requirement, it just lets VLC play more
valid files (provided you have more memory than 50 MB). As there is no really
good fixed value, I simply but a very large one (mainly for devs working on
the input/decoder).
 I should have removed the test, it would have been easier...

Regards,

-- 
fenrir