On Sat, Dec 16, 2023 at 09:13:21AM +0100, Thilo Borgmann via ffmpeg-devel wrote: [...]
> +// get number of bytes from cur to '\0'
> +static int buf_get_zero(FsyncContext *ctx)
maybe doxygen syntax would make sense for comments descrining functions
its not public api but still maybe
> +{
> + return av_strnlen(ctx->cur, ctx->end - ctx->cur);
> +}
> +
[...]
> + if (s->last_frame) {
> + ret = av_sscanf(s->cur, "%"PRId64" %"PRId64" %d/%d", &s->ptsi,
> &s->pts, &s->tb_num, &s->tb_den);
> + if (ret != 4) {
> + av_log(ctx, AV_LOG_ERROR, "Unexpected format found (%i / 4).\n",
> ret);
> + ff_outlink_set_status(outlink, AVERROR_INVALIDDATA,
> AV_NOPTS_VALUE);
> + return AVERROR_INVALIDDATA;
> + }
> +
> + av_log(ctx, AV_LOG_DEBUG, "frame %lli ", s->last_frame->pts);
warning: format ‘%lli’ expects argument of type ‘long long int’, but argument 4
has type ‘int64_t {aka long int}’ [-Wformat=]
"%"PRIi64 / "%"PRId64 / "%"PRIu64 / "%"PRIx64
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Never trust a computer, one day, it may think you are the virus. -- Compn
signature.asc
Description: PGP signature
_______________________________________________ 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".
