[FFmpeg-devel] [PATCH 4/7] avcodec/ass: output missing fields in AVSubtitles and output files

[James Darnley]

On 2014-10-11 19:59, Clément Bœsch wrote:
> Fixes Ticket #3207
> ---
>  libavcodec/ass.c                |  29 +++++++++--
>  libavcodec/version.h            |   2 +-
>  tests/ref/fate/sub-aqtitle      |  72 +++++++++++++--------------
>  tests/ref/fate/sub-charenc      | 106 ++++++++++++++++++++--------------------
>  tests/ref/fate/sub-jacosub      |  28 +++++------
>  tests/ref/fate/sub-microdvd     |  28 +++++------
>  tests/ref/fate/sub-movtext      |  12 ++---
>  tests/ref/fate/sub-mpl2         |  14 +++---
>  tests/ref/fate/sub-mpsub        |  48 +++++++++---------
>  tests/ref/fate/sub-mpsub-frames |  10 ++--
>  tests/ref/fate/sub-pjs          |  12 ++---
>  tests/ref/fate/sub-realtext     |  16 +++---
>  tests/ref/fate/sub-sami         |  24 ++++-----
>  tests/ref/fate/sub-srt          |  80 +++++++++++++++---------------
>  tests/ref/fate/sub-subviewer    |  12 ++---
>  tests/ref/fate/sub-subviewer1   |  26 +++++-----
>  tests/ref/fate/sub-vplayer      |  12 ++---
>  tests/ref/fate/sub-webvtt       |  36 +++++++-------
>  18 files changed, 294 insertions(+), 273 deletions(-)

These test filetypes are input files, right?  And you're encoding to
SSA/ASS, right?  The output certainly looks like SSA.  I'm just checking
that these changes are not unrelated.

Other than that it looks fine and seems to agree with what's in the ticket.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 603 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141014/8165eeeb/attachment.asc>