[Ffmpeg-devel] [PATCH] mjpeg cleanup and again interlaced fix
Sun Apr 15 03:44:27 CEST 2007
On Sat, Apr 14, 2007 at 06:55:07PM +0200, Baptiste Coudurier wrote:
> Michael Niedermayer wrote:
> > Hi
> > On Sat, Apr 14, 2007 at 06:16:40PM +0200, Baptiste Coudurier wrote:
> >> Michael Niedermayer wrote:
> >>> Hi
> >>> On Sat, Apr 14, 2007 at 02:36:07PM +0200, Baptiste Coudurier wrote:
> >>> [...]
> >>>>>>>> you also should know what will happen with mov style w/h CORRECT cropping,
> >>>>>>>> aren't you also mov.c maintainer ?
> >>>>>>> well i think mjpeg after the 2nd patch from you will break with mov
> >>>>>>> cropping, which would mean your 2nd patch is buggy
> >>>>>> Well, it might break for cropping, but it is IMHO less buggy than
> >>>>>> before, patch addresses an issue, and does not address the other issue,
> >>>>>> but still height *2 is wrong.
> >>>>> i dont know if its written in any docs but we have always rejected bugfixes
> >>>>> which knowingly introduce other bugs
> >>>> Why don't you send patches to common parts, which I will know will
> >>>> introduce other bugs ? So you can experience that "rejected" feeling.
> >>> iam maintainer of the common parts also you are free to complain if i break
> >>> something
> >> I did. You voluntary still ignore it and don't fix.
> > i dont know what exactly you mean, could you tell me the subj/date of the
> > mail where you complained?
> Also now you got serveral bug reports related to code you maintain,
> let's start counting time before the fix.
you can stop counting ive fixed the bugs, which where in my code (the last
remaining one is introduced by a patch by someone else and simply reversing
that patch will fix that one too)
PS: i still dont understand what your 3rd patch did even with looking at the
ODML spec and correcting the bug which was a 1 line change
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: not available
More information about the ffmpeg-devel