[FFmpeg-devel] [RFC] XSUB subtitle decoder

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


Hi

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?

yes
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