[FFmpeg-devel] [PATCH 4/6] libavcodec/ccaption_dec: reap_screen is not necessary when clearing screen or buffer

[Aman Gupta]

From: Aman Gupta <aman at tmm1.net>

---
 libavcodec/ccaption_dec.c | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c
index 6f1e70d..8373a0c 100644
--- a/libavcodec/ccaption_dec.c
+++ b/libavcodec/ccaption_dec.c
@@ -395,17 +395,15 @@ static void handle_pac(CCaptionSubContext *ctx, uint8_t hi, uint8_t lo)
     }
 }
 
-static void handle_erase(CCaptionSubContext *ctx, int64_t pts, int n_screen)
+static void handle_erase(CCaptionSubContext *ctx, int n_screen)
 {
     struct Screen *screen = ctx->screen + n_screen;
-
-    reap_screen(ctx, pts);
     screen->row_used = 0;
 }
 
 static void handle_eoc(CCaptionSubContext *ctx, int64_t pts)
 {
-    handle_erase(ctx, pts, ctx->active_screen);
+    reap_screen(ctx, pts);
     ctx->active_screen = !ctx->active_screen;
     ctx->cursor_column = 0;
 }
@@ -486,7 +484,7 @@ static void process_cc608(CCaptionSubContext *ctx, int64_t pts, uint8_t hi, uint
             break;
         case 0x2c:
             /* erase display memory */
-            handle_erase(ctx, pts, ctx->active_screen);
+            handle_erase(ctx, ctx->active_screen);
             break;
         case 0x2d:
             /* carriage return */
@@ -497,7 +495,7 @@ static void process_cc608(CCaptionSubContext *ctx, int64_t pts, uint8_t hi, uint
             break;
         case 0x2e:
             /* erase non displayed memory */
-            handle_erase(ctx, pts, !ctx->active_screen);
+            handle_erase(ctx, !ctx->active_screen);
             break;
         case 0x2f:
             /* end of caption */
-- 
2.5.3