On 3/23/22 07:38, Anton Khirnov wrote:
+        return 0;
+    if (avpriv_jpegxl_verify_codestream_header(NULL, p->buf, p->buf_size) == 0)
This function is way too complicated for a probe function. Can you think
of something simpler?
This is necessary to prevent the probetest from returning false positives. My original patch lacked something like this and it was rejected on those grounds.

Leo Izen (thebombzen)


_______________________________________________
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