[FFmpeg-devel] [PATCH 2/5] avcodec/vc1: fix mquant calculation for interlace field pictures

Jerome Borsboom jerome.borsboom at carpalis.nl
Sun May 20 12:00:12 EEST 2018


>> On Fri, May 18, 2018 at 05:06:23PM +0200, Jerome Borsboom wrote:
>>> For interlace field pictures s->mb_height indicates the height of the
>>> full
>>> picture in MBs, i.e. the two fields combined. A single field is half this
>>> size. When calculating mquant for interlace field pictures, the bottom
>>> edge
>>> is the last MB row of the field.
>>>
>>> Signed-off-by: Jerome Borsboom <jerome.borsboom at carpalis.nl>
>>> ---
>>>  libavcodec/vc1_block.c | 3 ++-
>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> for patch 1 you list a file that it improves.
> 
> The file - afaict - is related to the patchset as a whole.
> The current version of the patchset does not fix the
> file though, so I believe we should wait for a new
> version of the patchset.
> 
> The sample is >20MB, most of the reference files are
> very small, so perhaps we can avoid using this one.
> 
> Carl Eugen

The issue that remains is not related to this patch set. In my view,
there is no need to wait for a new patch set. I will just submit a new
patch for that issue.

Regards,
Jerome


More information about the ffmpeg-devel mailing list