[FFmpeg-devel] [PATCH 3/4] fftools/textformat: infer type in mermaid_print_value

[softworkz .]

Hi Marvin,

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
> Marvin Scholz
> Sent: Mittwoch, 11. Juni 2025 21:57
> To: ffmpeg-devel at ffmpeg.org
> Subject: [FFmpeg-devel] [PATCH 3/4] fftools/textformat: infer type in
> mermaid_print_value
> 
> Instead of the caller having to indicate if it is passing an
> integer, just use the fact that str is NULL when an integer is
> to be printed.
> ---
>  fftools/textformat/tf_mermaid.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/fftools/textformat/tf_mermaid.c
> b/fftools/textformat/tf_mermaid.c
> index dbe489a7a7..da371c8fff 100644
> --- a/fftools/textformat/tf_mermaid.c
> +++ b/fftools/textformat/tf_mermaid.c
> @@ -547,7 +547,7 @@ static void
> mermaid_print_section_footer(AVTextFormatContext *tfc)
>  }
> 
>  static void mermaid_print_value(AVTextFormatContext *tfc, const char
> *key,
> -                                const char *str, int64_t num, const
> int is_int)
> +                                const char *str, int64_t num)
>  {
>      MermaidContext *mmc = tfc->priv;
>      const AVTextFormatSection *section = tf_get_section(tfc, tfc-
> >level);
> @@ -587,7 +587,7 @@ static void
> mermaid_print_value(AVTextFormatContext *tfc, const char *key,
>          switch (mmc->diagram_config->diagram_type) {
>          case AV_DIAGRAMTYPE_GRAPH:
> 
> -            if (is_int) {
> +            if (!str) {

This is not right - it would print 0 when the str is NULL.
See tf_xml, it uses the same pattern (is_int).

Thanks,
sw