[FFmpeg-devel] [PATCH] H.264: fix decoding of plain still images (broken since revision 14289)
Sun Jan 4 15:23:29 CET 2009
On Sat, Jan 03, 2009 at 07:01:32PM +0100, Reinhard Nissl wrote:
> the offending change in that revision seems to me the following:
> - if(prev && pics <= s->avctx->has_b_frames ||
> - out = prev;
> + if(pics <= s->avctx->has_b_frames || out_of_order)
> + out = NULL;
> In the case of a plain still image, there are no previous images.
> Revision 14288 honored this by checking prev and therefore
> output the current image (the still image).
> In Revision 14289, this test is missing and therefore, no picture
> is output. I've to add that pics as well as has_b_frames was 1
> while out_of_order was 0 when checking the condition in both
> As you can see from the attached patch, the location of this test
> has moved meanwhile and the condition as well as the behavior has
> been inverted, but still lacks a test for "no previous image".
> I've added such a test by using outputted_poc and comparing it to
> INT_MIN, which is the initial value of this variable.
> For testing, I've uploaded two sample files into subdirectory
> rnissl: one with a still frame and one with a pair of fields. See
> readme.txt for more information.
this patch totally breaks h264 decoding, thus rejected.
id post the list of what it breaks but its as far as i can see
every single h264 file from the conformance suite so theres no point.
Besides i do not know what you mean by "still image" which page
of the h264 spec describes it?
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel