[Ffmpeg-devel] [PATCH] Add Dirac support to ffmpeg]
Fri Sep 16 17:50:12 CEST 2005
At 11:37 02/09/2005, you wrote:
>On Tue, Aug 30, 2005 at 04:25:54PM +0100, Andrew Kennedy wrote:
> > Hi,
> > I've amended the patch in line with your recommendations. As part of the
> > changes, encoding should now only work with the -2 strict option.
>thanks, but it seems i keep finding issues every time i want to apply it
>so i have to annoy you again with a list of recommandet changes
> > +static int dirac_encode_frame(AVCodecContext *avccontext,
> > + unsigned char *frame,
> > + int buf_size, void *data)
> > + avccontext->coded_frame->pts= 0;
>this obviously is wrong, 2 frames cant have the same pts, why do you
>set it at all?
This seems to have been set to avoid problems when encoding to container
formats such as avi. When removed, non-monotone errors are thrown up before
writing the frame. So this line of code is really more of a short-term fix.
As to a cleaner implementation, I'm not sure which approach to take. Any
More information about the ffmpeg-devel