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".