diff mbox

[FFmpeg-devel,2/3] avformat/assenc: optimize writing dialogs

Message ID 20190805213454.9548-2-cus@passwd.hu
State Accepted
Commit 61b4daf78ce09534b258aae6b86934ce98d2b27f
Headers show

Commit Message

Marton Balint Aug. 5, 2019, 9:34 p.m. UTC
An example where the preivously added API can be useful.

Also fixes ticket #6390.

Signed-off-by: Marton Balint <cus@passwd.hu>
---
 libavformat/assenc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Paul B Mahol Aug. 6, 2019, 7:40 a.m. UTC | #1
On Mon, Aug 5, 2019 at 11:35 PM Marton Balint <cus@passwd.hu> wrote:

> An example where the preivously added API can be useful.
>
> Also fixes ticket #6390.
>
> Signed-off-by: Marton Balint <cus@passwd.hu>
> ---
>  libavformat/assenc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/assenc.c b/libavformat/assenc.c
> index d50f18feb1..12aadca171 100644
> --- a/libavformat/assenc.c
> +++ b/libavformat/assenc.c
> @@ -95,7 +95,7 @@ static void purge_dialogues(AVFormatContext *s, int
> force)
>                     ass->expected_readorder, dialogue->readorder);
>              ass->expected_readorder = dialogue->readorder;
>          }
> -        avio_printf(s->pb, "Dialogue: %s\r\n", dialogue->line);
> +        avio_print(s->pb, "Dialogue: ", dialogue->line, "\r\n");
>          if (dialogue == ass->last_added_dialogue)
>              ass->last_added_dialogue = next;
>          av_freep(&dialogue->line);
> --
> 2.16.4
>
> _______________________________________________
> 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".


Whole patch set looks fine to me, but perhaps you want additional reviews.
diff mbox

Patch

diff --git a/libavformat/assenc.c b/libavformat/assenc.c
index d50f18feb1..12aadca171 100644
--- a/libavformat/assenc.c
+++ b/libavformat/assenc.c
@@ -95,7 +95,7 @@  static void purge_dialogues(AVFormatContext *s, int force)
                    ass->expected_readorder, dialogue->readorder);
             ass->expected_readorder = dialogue->readorder;
         }
-        avio_printf(s->pb, "Dialogue: %s\r\n", dialogue->line);
+        avio_print(s->pb, "Dialogue: ", dialogue->line, "\r\n");
         if (dialogue == ass->last_added_dialogue)
             ass->last_added_dialogue = next;
         av_freep(&dialogue->line);