Re: [FFmpeg-devel] [PATCH 4/4] fftools/textformat: remove leftover comments in mermaid_print_value

["softworkz ." <softworkz-at-hotmail.com@ffmpeg.org>]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of
> Marvin Scholz
> Sent: Mittwoch, 11. Juni 2025 21:57
> To: ffmpeg-devel@ffmpeg.org
> Subject: [FFmpeg-devel] [PATCH 4/4] fftools/textformat: remove
> leftover comments in mermaid_print_value
> 
> Remove some leftover commented code and an extraneous semicolon.
> ---
>  fftools/textformat/tf_mermaid.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/fftools/textformat/tf_mermaid.c
> b/fftools/textformat/tf_mermaid.c
> index da371c8fff..9d98b6a36f 100644
> --- a/fftools/textformat/tf_mermaid.c
> +++ b/fftools/textformat/tf_mermaid.c
> @@ -575,7 +575,7 @@ static void
> mermaid_print_value(AVTextFormatContext *tfc, const char *key,
>      }
> 
>      if (section->linktype_key && !strcmp(section->linktype_key,
> key)) {
> -        mmc->section_data[tfc->level].link_type =
> (AVTextFormatLinkType)num;;
> +        mmc->section_data[tfc->level].link_type =
> (AVTextFormatLinkType)num;
>          exit = 1;
>      }
> 
> @@ -590,10 +590,7 @@ static void
> mermaid_print_value(AVTextFormatContext *tfc, const char *key,
>              if (!str) {
>                  writer_printf(tfc, "<span class=\"%s\">%s:
> %"PRId64"</span>", key, key, num);
>              } else {
> -                ////AVBPrint b;
> -                ////av_bprint_init(&b, 0, AV_BPRINT_SIZE_UNLIMITED);
>                  const char *tmp = av_strireplace(str, "\"", "'");
> -                ////av_bprint_escape(&b, str, NULL,
> AV_ESCAPE_MODE_AUTO, AV_ESCAPE_FLAG_STRICT);
>                  writer_printf(tfc, "<span class=\"%s\">%s</span>",
> key, tmp);
>                  av_freep(&tmp);
>              }
> --
> 2.39.5 (Apple Git-154)
> 
> _______________________________________________

LGTM, thanks!

_______________________________________________
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".