[FFmpeg-devel] [PATCH] Warn about PAFF & Spatial

[Michael Niedermayer]

On Mon, Jul 21, 2008 at 04:11:10PM +0000, Carl Eugen Hoyos wrote:
> Hi!
> 
> Loren Merritt <lorenm <at> u.washington.edu> writes:
> 
> > > The warning for interlaced + spatial direct was (accidentally I think)
> > > removed in r11806 when MBAFF + spatial direct was implemented.
> > > So I am for restoring it as per the patch.
> > 
> > Not accidental. I removed the warning because I was unaware of any 
> > difference between PAFF direct and MBAFF direct, and I still don't see 
> > anywhere in the standard that they differ.
> 
> (Not surprisingly) You were probably right:
> While I know of one file in the conformance suite, HCAFF1_HHI, that claims not
> to decode correctly because of PAFF and spatial direct, but has all crc's
> correct (and a second one, Sharp_MP_PAFF_1r2, that decodes much better than the
> message would suggest), cabac_mot_picaff0_full shows the typical artifacts that
> are reported so often, but does not contain spatial direct mode.
> 
> Should I remove the warning?

I have no objections.

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080721/2a102be9/attachment.pgp>