[FFmpeg-devel] [PATCH]Fix bug in setting the pts when decoding or transcoding Dirac video wrapped in MPEG-TS using libschroedingerdec.c

Michael Niedermayer michaelni
Sat Dec 11 19:48:30 CET 2010


On Tue, Dec 07, 2010 at 12:31:02PM +1100, Anuradha Suraparaju wrote:
> Hi,
> 
> 
> On Mon, Nov 29, 2010 at 2:03 PM, Anuradha Suraparaju
> <asuraparaju at gmail.com>wrote:
> 
> >
> >
> > On Fri, Nov 26, 2010 at 5:19 PM, Reimar D?ffinger <
> > Reimar.Doeffinger at gmx.de> wrote:
> >
> >> On Fri, Nov 26, 2010 at 10:17:09AM +1100, Anuradha Suraparaju wrote:
> >> > +            pktts->pts = avccontext->reordered_opaque;
> >>
> >> While it often will be pts in some form, I'd prefer if you didn't call it
> >> that.
> >> IMO it just risks confusing people into using it in the incorrect ways you
> >> did before.
> >>
> >
> > Noted. modified patch attached.
> >
> >
> Any comments/updates re this patch.

david? you are listed as maintainer ...

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

Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101211/4b99a7b6/attachment.pgp>



More information about the ffmpeg-devel mailing list