[Ffmpeg-cvslog] CVS: ffmpeg/libavcodec ac3enc.c,1.29,1.30

Michael Niedermayer michaelni
Sat Apr 8 02:19:45 CEST 2006


Hi

On Fri, Apr 07, 2006 at 07:12:17PM -0400, Justin Ruggles wrote:
> Corey Hickey wrote:
> > I would expect that the change is intended and the regresssion results
> > should be altered correspondingly. Justin, please comment. I'll patch
> > the results tomorrow if I don't hear otherwise.
> 
> Yes, that would be the expected result for any 44100/22050/11025 Hz
> files.  I see you have updated regression results already...great!
> 
> For future reference, if I submit a patch for approval which would
> knowingly break regression tests, should I include an update in the
> patch or should I just mention it and leave it to the admins if/when the
> patch is accepted?

always include a diff for the regression test checksums


[...]

-- 
Michael

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is





More information about the ffmpeg-cvslog mailing list