[FFmpeg-devel] [PATCH 2/3] lavf/srtenc: honor subtitle position side data.

Clément Bœsch ubitux at gmail.com
Thu Dec 20 16:21:12 CET 2012


On Sat, Dec 01, 2012 at 12:43:27AM +0100, Clément Bœsch wrote:
> ---
>  libavformat/srtenc.c | 18 +++++++++++++++++-
>  1 file changed, 17 insertions(+), 1 deletion(-)
> 
> diff --git a/libavformat/srtenc.c b/libavformat/srtenc.c
> index 97b297e..57be568 100644
> --- a/libavformat/srtenc.c
> +++ b/libavformat/srtenc.c
> @@ -22,6 +22,7 @@
>  #include "avformat.h"
>  #include "internal.h"
>  #include "libavutil/log.h"
> +#include "libavutil/intreadwrite.h"
>  
>  /* TODO: add options for:
>     - character encoding;
> @@ -63,6 +64,17 @@ static int srt_write_packet(AVFormatContext *avf, AVPacket *pkt)
>  
>      if (write_ts) {
>          int64_t s = pkt->pts, e, d = pkt->duration;
> +        int size, x1 = -1, y1 = -1, x2 = -1, y2 = -1;
> +        const uint8_t *p;
> +
> +        av_packet_split_side_data(pkt);
> +        p = av_packet_get_side_data(pkt, AV_PKT_DATA_SUBTITLE_POSITION, &size);
> +        if (p && size == 16) {
> +            x1 = AV_RL32(p     );
> +            y1 = AV_RL32(p +  4);
> +            x2 = AV_RL32(p +  8);
> +            y2 = AV_RL32(p + 12);
> +        }
>  
>          if (d <= 0)
>              /* For backward compatibility, fallback to convergence_duration. */
> @@ -73,12 +85,16 @@ static int srt_write_packet(AVFormatContext *avf, AVPacket *pkt)
>              return 0;
>          }
>          e = s + d;
> -        avio_printf(avf->pb, "%d\n%02d:%02d:%02d,%03d --> %02d:%02d:%02d,%03d\n",
> +        avio_printf(avf->pb, "%d\n%02d:%02d:%02d,%03d --> %02d:%02d:%02d,%03d",
>                         srt->index,
>                         (int)(s / 3600000),      (int)(s / 60000) % 60,
>                         (int)(s /    1000) % 60, (int)(s %  1000),
>                         (int)(e / 3600000),      (int)(e / 60000) % 60,
>                         (int)(e /    1000) % 60, (int)(e %  1000));
> +        if (p)
> +            avio_printf(avf->pb, "  X1:%03d X2:%03d Y1:%03d Y2:%03d",
> +                        x1, y1, x2, y2);
> +        avio_printf(avf->pb, "\n");
>      }
>      avio_write(avf->pb, pkt->data, pkt->size);
>      if (write_ts)

Fixed and applied.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121220/995dd362/attachment.asc>


More information about the ffmpeg-devel mailing list