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

Reply via email to