aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorErik Olofsson <eaj.olofsson@gmail.com>2013-11-07 21:07:10 +0100
committerMichael Niedermayer <michaelni@gmx.at>2013-11-12 03:15:33 +0100
commit3b1dcdec7d85f25b39fc90aed026419d394b78a5 (patch)
tree43b6c8b9b828f77a16381d973a6bf57179d16353
parent355cea8ddc9bcf9e485dd4fdc9ffe7a491450c97 (diff)
downloadffmpeg-3b1dcdec7d85f25b39fc90aed026419d394b78a5.tar.gz
libavcodec/xsubenc.c: Fix bounding box coordinates
Fix coordinates of the lower right corner of the text area. Note that the coordinates are redundant as the size and position of the text area are known. Many decoders (including Sony Playstation 3, VLC and FFmpeg) ignore the redundant coordinates. Some hardware decoders need them for correct playback. Verified on Philips DVD player models HTS7201 and DVP3380. Fixes ticket #3031 Signed-off-by: Erik Olofsson <eaj.olofsson@gmail.com> Reviewed-by: Reimar Döffinger <Reimar.Doeffinger@gmx.de> Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r--libavcodec/xsubenc.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/libavcodec/xsubenc.c b/libavcodec/xsubenc.c
index 9dc80ee4af..bb2cdb0529 100644
--- a/libavcodec/xsubenc.c
+++ b/libavcodec/xsubenc.c
@@ -166,8 +166,8 @@ static int xsub_encode(AVCodecContext *avctx, unsigned char *buf,
bytestream_put_le16(&hdr, height);
bytestream_put_le16(&hdr, h->rects[0]->x);
bytestream_put_le16(&hdr, h->rects[0]->y);
- bytestream_put_le16(&hdr, h->rects[0]->x + width);
- bytestream_put_le16(&hdr, h->rects[0]->y + height);
+ bytestream_put_le16(&hdr, h->rects[0]->x + width -1);
+ bytestream_put_le16(&hdr, h->rects[0]->y + height -1);
rlelenptr = hdr; // Will store length of first field here later.
hdr+=2;