[vlc-devel] Re: [PATCH] styled subtitles support - resend

[Jean-Baptiste Kempf]

Hi,

Bernie Purcell wrote:
>  From original mail:
> This patch follows on to an earlier one I made that displayed USF 
> subtitles in styled text. This one adds support for SubViewer (excluding 
> the metadata header information for which I can't find a specification), 
> MicroDVD, SubRip and VPlayer - ie. all the generics subtitle formats 
> except SAMI.
> SAMI isn't straightforward, and the existing demux code strips out any 
> tags which might have once been present in its context. It isn't a 
> trivial job to pass the stylesheet information which can be contained in 
> that format. A major hurdle in passing it lies in the fact that its XML 
> is not well-formed.
> Because I couldn't find specs for any of these formats except SAMI (and 
> ASS and USF) I made an executive decision to restrict the list of 
> supported tags to the same ones used by Matroska in the USF DTD -- as 
> explained in the comments in the code. If anyone has any corrections to 
> apply to this assumption I'd like to hear from them, though.
> SSA/ASS is not included in this patch, but only because I haven't gotten 
> around to it yet.

Use that patch instead. Applies fine on trunk, compiles but I get a lot 
of warnings.

-- 
jb
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Subtitles.patch
Type: text/x-patch
Size: 14852 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20070328/86df76c5/attachment.bin>