avfilter/drawtext: fix box sizing
authorGyan Doshi <ffmpeg@gyani.pro>
Tue, 10 Jul 2018 09:44:06 +0000 (15:14 +0530)
committerGyan Doshi <ffmpeg@gyani.pro>
Fri, 13 Jul 2018 04:41:15 +0000 (10:11 +0530)
At present, box size is clipped to frame size before being drawn,
which can lead to the box not fully covering animated text which is
longer than one or both frame dimensions.

Since ff_blend_rectangle correctly takes care of clipping, it is skipped
here which results in correct box sizing

libavfilter/vf_drawtext.c

index 3affa73..cca2cbc 100644 (file)
@@ -1407,8 +1407,8 @@ static int draw_text(AVFilterContext *ctx, AVFrame *frame,
     update_color_with_alpha(s, &bordercolor, s->bordercolor);
     update_color_with_alpha(s, &boxcolor   , s->boxcolor   );
 
-    box_w = FFMIN(width - 1 , max_text_line_w);
-    box_h = FFMIN(height - 1, y + s->max_glyph_h);
+    box_w = max_text_line_w;
+    box_h = y + s->max_glyph_h;
 
     if (s->fix_bounds) {