lavc/ccaption_dec: reap_screen is not necessary when clearing screen or buffer
authorAman Gupta <aman@tmm1.net>
Tue, 5 Jan 2016 23:06:02 +0000 (15:06 -0800)
committerClément Bœsch <u@pkh.me>
Fri, 8 Jan 2016 19:39:55 +0000 (20:39 +0100)
libavcodec/ccaption_dec.c

index 02bce01..517e2c0 100644 (file)
@@ -361,17 +361,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;
 {
     struct Screen *screen = ctx->screen + n_screen;
-
-    reap_screen(ctx, pts);
     screen->row_used = 0;
 }
 
 static void handle_eoc(CCaptionSubContext *ctx, int64_t 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;
 }
     ctx->active_screen = !ctx->active_screen;
     ctx->cursor_column = 0;
 }
@@ -452,7 +450,7 @@ static void process_cc608(CCaptionSubContext *ctx, int64_t pts, uint8_t hi, uint
             break;
         case 0x2c:
             /* erase display memory */
             break;
         case 0x2c:
             /* erase display memory */
-            handle_erase(ctx, pts, ctx->active_screen);
+            handle_erase(ctx, ctx->active_screen);
             break;
         case 0x2d:
             /* carriage return */
             break;
         case 0x2d:
             /* carriage return */
@@ -463,7 +461,7 @@ static void process_cc608(CCaptionSubContext *ctx, int64_t pts, uint8_t hi, uint
             break;
         case 0x2e:
             /* erase non displayed memory */
             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 */
             break;
         case 0x2f:
             /* end of caption */