[FFmpeg-cvslog] r10453 - trunk/libavcodec/snow.c

[Rich Felker]

On Sun, Sep 09, 2007 at 07:12:35PM +0200, Michael Niedermayer wrote:
> Hi
> 
> On Sun, Sep 09, 2007 at 05:29:08PM +0200, Guillaume POIRIER wrote:
> > HI,
> > 
> > On 9/9/07, michael <subversion at mplayerhq.hu> wrote:
> > 
> > > argh i really wish id never accepted these slice patches ...
> > 
> > But aren't slices a good idea in general in non block-based codecs
> > such as the ones like Snow or Dirac which use wavelets, since they
> > allow the codec to be more CPU cache-friendly?
> > 
> > Or am I missing something?
> 
> yes you optimize code AFTER having a final spec not before

While I agree in principle, it's also important to know (1) whether
the optimization will be possible (IIRC old snow bitstream precluded
slices..?), and (2) what benefit the optimization might have, so you
know if it's worthwhile to make design tradeoffs needed to keep the
possibility of that optimization open. I think those were the
motivation behind the slice patches.

Rich