On Sat, Oct 24, 2020 at 13:32:25 +1300, Michael Keeley wrote:
> -    if (memcmp(p->buf, "DHAV", 4))
> +    if (memcmp(p->buf, "DHAV", 4) != 0 && memcmp(p->buf, "ZLAV", 4) != 0)

The "!= 0" isn't necessary.

> -        if (!memcmp(signature, "DHAV", 4)) {
> +        if (memcmp(signature, "DHAV", 4) == 0 || memcmp(signature, "ZLAV", 
> 4) == 0) {

Also here, ffmpeg prefers the "!memcmp()" style.

Moritz
_______________________________________________
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