[mythtv] SSA/ASS sub support(with patches)

[Mark Kendall]

On 19 November 2010 20:18, Shawn King <shawnking at optusnet.com.au> wrote:
> FFMPEG has now gotten ASS/SSA sub decoders is there any plans on adding
> support?
>
> attached is some patches that i have been using for a few days now w/out
> issues it doesnt support any SSA override codes but most other stuff is in
> there(myth-03-SubtitleScreenSSASubSupport.patch is very wip atm)
>
> the ffmpeg-* patches is the backported ASS/SSA subtitle code with 1
> bugfix(ASS dialog line wasn't null terminated not sure if FFMPEG guys know
> about it yet)
> myth-01-AVFormatFontSupport.patch adds support for loading embedded fonts
> into mythtv's fontmanager for use in the subs
> myth-02-AVFormatSubtitleHeaderSupport.patch adds support for getting the
> subtitle headers
> and myth-03-SubtitleScreenSSASubSupport.patch adds all the code for
> displaying the subs and is quite messy(and prob wrong at parts)
>
> it isnt good enough to be considered for commiting i know but it is a start

To be honest - looks like a pretty good start:)

Clearly there is a decision to be made about backporting the ffmpeg
changes (and I'd suggest it would be preferable to have them broken
out into the original ffmpeg versions and separate additional fixes -
to help tracking) - but I have no idea when the next ffmpeg merge is
planned.

The decoder stuff looks straightforward enough - though somebody else
will need to comment on the font code.

As far as the subtitlescreen patch goes :-

 - I'd suggest just rolling SSASubUtils into SubtitleScreen
 - look at using the the Qt::Alignment flags directly rather than
adding the new alignment enums
 - there is some extra code needed to handle the fontstretch property,
otherwise the fonts may not look correct with XVideo rendering.

regards

Mark