On Thu, 21 Nov 2024 02:12:28 +0100 Michael Niedermayer <mich...@niedermayer.cc>
wrote:
> > +static int validate_params(SwsContext *ctx)
> > +{
> > +#define VALIDATE(field, min, max) \
> > + if (ctx->field < min || ctx->field > max) { \
> > + av_log(ctx, AV_LOG_ERROR, "'%s' (%d) out of range [%d, %d]\n", \
> > + #field, (int) ctx->field, min, max); \
> > + return AVERROR(EINVAL); \
> > + }
> > +
>
> > + VALIDATE(threads, 0, 8192);
>
> a named "#define" would be better than a litteral number
Fixed.
>
> should be ok otherwise
>
> thx
>
> [...]
> --
> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> If the United States is serious about tackling the national security threats
> related to an insecure 5G network, it needs to rethink the extent to which it
> values corporate profits and government espionage over security.-Bruce
> Schneier
> _______________________________________________
> 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".
_______________________________________________
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".
