[FFmpeg-devel] [PATCH] Add support for parsing the Display Definition Segment in dvbsubdec.c

Michael Niedermayer michaelni
Fri Apr 23 15:41:09 CEST 2010


On Fri, Apr 23, 2010 at 09:05:03AM +0100, Ian Caulfield wrote:
> On 23 April 2010 05:05, Mark Kendall <mark.kendall at gmail.com> wrote:
> > The attached patch parses the Display Definition Segment in DVB
> > subtitle packets (see EN 300 743 V1.3.1).
> >
> > The Display Definition Segment explicitly indicates to the decoder the
> > display size for which the subtitles were generated, allowing standard
> > definition subtitles to be displayed on high definition streams.
> >
> > This extends AVSubtitleRect to hold the new display information and,
> > if a Display Definition Segment is not seen, defaults to 720x576 (per
> > the standard).
> 
> Would it be more appropriate to fill in the width and height fields of
> AVCodecContext?

agree, 
with documentation added to width/height and where the x/y position is
documented

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100423/b7c346c6/attachment.pgp>



More information about the ffmpeg-devel mailing list