[FFmpeg-devel] [PATCH] avcodec/cbs_av1: fix setting FrameWidth in frame_size_with_refs()

James Almer jamrial at gmail.com
Wed Aug 26 00:48:01 EEST 2020


On 8/25/2020 6:38 PM, Derek Buitenhuis wrote:
> On 24/08/2020 00:47, James Almer wrote:
>> Section 5.9.7 of the spec states
>>
>>     UpscaledWidth = RefUpscaledWidth[ ref_frame_idx[ i ] ]
>>     FrameWidth    = UpscaledWidth
>>     FrameHeight   = RefFrameHeight[ ref_frame_idx[ i ] ]
>>     RenderWidth   = RefRenderWidth[ ref_frame_idx[ i ] ]
>>     RenderHeight  = RefRenderHeight[ ref_frame_idx[ i ] ]
>>
>> Meaning FrameWidth must not be set to RefFrameWidth[ ref_frame_idx[ i ] ]
>> as we're currently doing.
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavcodec/cbs_av1_syntax_template.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> LGTM. (I went and double checked my own parse too ;))

Did you make the same mistake? :p

Pushed, thanks.

> 
> - Derek
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> 



More information about the ffmpeg-devel mailing list