On Thu, Oct 08, 2020 at 09:53:12PM +0200, Andreas Rheinhardt wrote:
> It can't because the corresponding trees don't have any loose ends.
> 
> Removing the checks also removed an instance of av_log(NULL (with a
> nonsense message) from the codebase.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com>
> ---
>  libavcodec/mdec.c      | 2 --
>  libavcodec/mpeg12.h    | 4 ----
>  libavcodec/mpeg12dec.c | 4 ----
>  3 files changed, 10 deletions(-)

It is possible to encode nonsensical dc differnces, these could be checked
for and trigger errors. Not in the current codebase no, but one could add
a check for it
Without such checks, your patch is probably ok

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to