On Thu, 21 Nov 2024 02:12:28 +0100 Michael Niedermayer <[email protected]> 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 > [email protected] > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > [email protected] with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
