lör 2020-10-03 klockan 00:23 +0200 skrev Nicolas Gaullier: > + if (container_word_bits && !(container_word_bits == 16 && word_bits == > 16) && !(container_word_bits == 24 && word_bits == 20) && > !(container_word_bits == 24 && word_bits == 24)) > + return AVERROR_INVALIDDATA; >
That if line is overly long. Maybe reformat it a bit like if (container_word_bits && !(container_word_bits == 16 && word_bits == 16) && !(container_word_bits == 24 && word_bits == 20) && !(container_word_bits == 24 && word_bits == 24)) { return AVERROR_INVALIDDATA; } The rest of the patch looks OK /Tomas _______________________________________________ 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".