[FFmpeg-devel] [PATCH] avcodec/cuviddec: update amount of decoder surfaces from within sequence decode callback

Timo Rothenpieler timo at rothenpieler.org
Tue Jun 6 16:35:26 EEST 2023


On 05.06.2023 14:29, Roman Arzumanyan wrote:
> Thanks for the review, Anton. Please find the updated patch attached.
> BTW - what do you mean by "you should also forward the actual error code"?
> Within the cuvid_handle_video_sequence() function in case of error,
> function still returns 0 and sets internal context error, so I kept
> behavior intact. Do I miss something?

applied with minor amendments.

Turned the if-statements around, we don't do them that way around in 
ffmpeg code.
And switched the surfaces option to have a default of -1, to not rely on 
a magic value of 25. That could surprise users who happen to request 25 
surfaces.


More information about the ffmpeg-devel mailing list