Hi Nariman, Thank you for your contribution. Please note that the contribution process is transitioning to Forgejo (https://code.ffmpeg.org) and would pivot any future patches there. With regards to this specific patch can you give some context as to why you think it is needed? Is there a known fault or trigger? Also return line needs indentation. I do not see other subtitle decoders doing this check, and am leaning toward trusting the API that decode() should not be called as such.
See callee here in the doxygen (and if you follow into the function in decode.c you can see that avctx is already used) https://ffmpeg.org/doxygen/trunk/group__lavc__decoding.html#ga5c30b73f0ec105f93d4e86464f541f21 _______________________________________________ ffmpeg-devel mailing list -- [email protected] To unsubscribe send an email to [email protected]
