[FFmpeg-devel] [PATCH] Bluray Subtitle Support, v7 (Updated patch to v11)

Stephen Backway stev391
Mon Aug 24 13:11:36 CEST 2009


On Mon, 2009-08-24 at 12:30 +0200, Diego Biurrun wrote:
> On Mon, Aug 24, 2009 at 08:08:18PM +1000, Stephen Backway wrote:
> > 
> > --- libavcodec/pgssubdec.c	(revision 0)
> > +++ libavcodec/pgssubdec.c	(revision 0)
> > @@ -0,0 +1,467 @@
> > +
> > +        if (run > 0 && pixel_count + run <= sub->rects[0]->w * sub->rects[0]->h) {
> > +            memset(sub->rects[0]->pict.data[0] + pixel_count, color, run);
> > +            pixel_count += run;
> > +        } else if (run == 0) {
> 
> nit: (!run)
> 
> > +            /*
> > +             * New Line. Check if correct pixels decoded,
> > +             * if not display warning and adjust bitmap
> > +             * pointer to correct new line position.
> > +             */
> 
> nit: That could easily fit on two lines.
> 
> Diego
Both fixed,

No other changes in attached patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bluray_subtitles_ffmpeg_19689_v11.1.diff
Type: text/x-patch
Size: 17643 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090824/7022240b/attachment.bin>



More information about the ffmpeg-devel mailing list