[FFmpeg-devel] [PATCH] PB-frames support for i263

[Kostya]

On Sun, Feb 22, 2009 at 06:54:59PM +0100, Michael Niedermayer wrote:
> On Sun, Feb 22, 2009 at 07:55:26PM +0200, Kostya wrote:
> > On Sun, Feb 22, 2009 at 02:06:21PM +0100, Michael Niedermayer wrote:
> > > On Sun, Feb 22, 2009 at 10:41:31AM +0200, Kostya wrote:
> > > > On Sat, Feb 21, 2009 at 03:28:55PM +0100, Michael Niedermayer wrote:
> > > > > On Sat, Feb 21, 2009 at 02:29:09PM +0200, Kostya wrote:
> > > > [...]
> > > > > 
> > > > > and your patch looks ok and for the release too if its tested with a few h263
> > > > > i263 videos and they are _binary_ identical
> > > > 
> > > > They are not since now loop filter also may be enabled. When I ust skip that bit,
> > > > results are the same.
> > > 
> > > do these files look better or worse with the loop filter enabled?
> > 
> > http://samples.mplayerhq.hu/V-codecs/I263/i263.avi looks a bit smoothier, for the
> > rest of samples looks almost the same (i.e. can't tell that quality has changed
> > for them).
> 
> if they look better than change ok (its even a bugfix)

Applied i263 header parsing patch, the rest will probably applied tomorrow.
 
> [...]
> 
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB