diff options
| author | Jean Delvare <khali@linux-fr.org> | 2013-06-12 10:48:57 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michaelni@gmx.at> | 2013-06-14 03:52:32 +0200 |
| commit | cef42ded8dd099bfc8de6cfcf75c579d13e582ad (patch) | |
| tree | 0c1ec8764d580112e3384889f6d80ed81d420f76 /doc | |
| parent | e850a064cee3c291d8e122a6284bb0ea1a59787f (diff) | |
| download | ffmpeg-cef42ded8dd099bfc8de6cfcf75c579d13e582ad.tar.gz | |
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>
Diffstat (limited to 'doc')
| -rw-r--r-- | doc/filters.texi | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/doc/filters.texi b/doc/filters.texi index 0381c6a740..6e616b26b4 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -2900,7 +2900,7 @@ value @code{invert} is used, the box edge color is the same as the video with inverted luma. @item thickness, t -Set the thickness of the box edge. Default value is @code{4}. +Set the thickness of the box edge. Default value is @code{3}. @end table @subsection Examples |