drawbox: Respect thickness parameter
authorJean Delvare <khali@linux-fr.org>
Wed, 12 Jun 2013 08:48:57 +0000 (10:48 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Fri, 14 Jun 2013 01:52:32 +0000 (03:52 +0200)
commitcef42ded8dd099bfc8de6cfcf75c579d13e582ad
tree0c1ec8764d580112e3384889f6d80ed81d420f76
parente850a064cee3c291d8e122a6284bb0ea1a59787f
drawbox: Respect thickness parameter

The drawbox video filter is drawing lines one pixel thinner than
requested. The default thickness is 4 pixel but in fact the lines
drawn by default are only 3 pixel wide.

Change the comparisons in the code to fix this off-by-one bug. Also
change the default thickness from 4 to 3 to minimize the unexpected
changes from the user's perspective.

As I was already touching these lines, I also removed the "maximum" in
the thickness parameter description, as I don't think it was adding
any value and I even found it confusing.

Reviewed-by: Andrey Utkin <andrey.krieger.utkin@gmail.com>
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
doc/filters.texi
libavfilter/vf_drawbox.c