[FFmpeg-devel] [RFC] XSUB subtitle decoder

Michael Niedermayer michaelni
Sun Aug 5 15:06:20 CEST 2007


On Sun, Aug 05, 2007 at 02:35:47PM +0200, Reimar D?ffinger wrote:
> Hello,
> On Sun, Aug 05, 2007 at 02:21:19PM +0200, Reimar D?ffinger wrote:
> > On Sun, Aug 05, 2007 at 01:21:22PM +0200, Michael Niedermayer wrote:
> > > hmm, whats in AVCodecContext.width/height? it seems these get read from
> > > avi for the subtitle stream?
> > 
> > Ah, right. That is how you get the original movie size. Now the question
> > is how to handle it.
> > Does adding a video_width and video_height to AVSubtitle and if set
> > using that to rescale in ffplay.c sound like a sensible approach?

this would make the x/y/w/h rational numbers practically ...
but why not use AVCodecContext.width/height directly?

> Hmm.. though that leaves the problem that the subtitle still might not
> fit, unless we add rescaling to blend_subrect... :-(

no need to do it in blend_subrect() it can be done before blend_subrect()

though that doesnt really change it by much, but its just 2 calls to
*swscale() or so ... at least i hope so ...
swscale() here could also output the same colorspace as the video uses


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Opposition brings concord. Out of discord comes the fairest harmony.
-- Heraclitus
-------------- 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/20070805/a5618357/attachment.pgp>

More information about the ffmpeg-devel mailing list