Re: [FFmpeg-devel] [PATCH 2/2] avcodec/bsf/showinfo: print packet data checksum

[Anton Khirnov <anton@khirnov.net>]

Quoting James Almer (2024-07-12 02:33:07)
> Signed-off-by: James Almer <jamrial@gmail.com>
> ---
>  libavcodec/bsf/showinfo.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/libavcodec/bsf/showinfo.c b/libavcodec/bsf/showinfo.c
> index 0899263545..57da3fc73c 100644
> --- a/libavcodec/bsf/showinfo.c
> +++ b/libavcodec/bsf/showinfo.c
> @@ -23,6 +23,7 @@
>  #include "bsf.h"
>  #include "bsf_internal.h"
>  
> +#include "libavutil/adler32.h"
>  #include "libavutil/internal.h"
>  #include "libavutil/log.h"
>  #include "libavutil/timestamp.h"
> @@ -34,22 +35,25 @@ typedef struct ShowinfoContext {
>  static int showinfo_filter(AVBSFContext *ctx, AVPacket *pkt)
>  {
>      ShowinfoContext *priv = ctx->priv_data;
> +    uint32_t crc;
>      int ret;
>  
>      ret = ff_bsf_get_packet_ref(ctx, pkt);
>      if (ret < 0)
>          return ret;
>  
> +    crc = av_adler32_update(0, pkt->data, pkt->size);
>      av_log(ctx, AV_LOG_INFO,
>             "n:%7"PRIu64" "
>             "size:%7d "
>             "pts:%s pt:%s "
>             "dts:%s dt:%s "
>             "ds:%"PRId64" d:%s "
> +           "0x%08"PRIx32,

could you prefix it with checksum: or something, so it's clear what it
means?

-- 
Anton Khirnov
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".