[FFmpeg-cvslog] r9601 - trunk/libavcodec/mpegidct.c

[Diego Biurrun]

On Thu, Jul 12, 2007 at 10:12:43AM +0200, Reimar D?ffinger wrote:
> On Thu, Jul 12, 2007 at 02:36:20AM +0200, Michael Niedermayer wrote:
> > On Thu, Jul 12, 2007 at 12:21:16AM +0100, M?ns Rullg?rd wrote:
> > > "Guillaume POIRIER" <poirierg at gmail.com> writes:
> > > > On 7/12/07, M?ns Rullg?rd <mans at mansr.com> wrote:
> > > >
> > > >> > +/* idct.c, inverse fast discrete cosine transform                           */
> > > >> > +
> > > >> > +/* Copyright (C) 1996, MPEG Software Simulation Group. All Rights Reserved. */
> > > >>
> > > >> NOOOOOOOOOOOOOOOOOOOOOOOoooooooooooooooooooooooooooooo
> > > >>
> > > >> Why oh why did you do this?
> > > >
> > > > Probably because Ivan wants its SVN committer privileges taken away...
> > > > I see no other reasonable explanation ;-)
> > > 
> > > Yeah, I'd be happy to revoke all his rights.  He is clearly not
> > > following our usual code of conduct.
> > 
> > ivan did follow the rules, he asked me and i said ok
> 
> Unless someone wants to stay anonymous such large changes should be
> posted on -dev first, even you can easily miss a problem in those and
> nobody likes not being asked and having things suddenly appear without a
> good reason (like anonymity would be).

100% agreed.  Michael himself complained back when Mike committed
Kostya's patches and new decoders without submitting them for
review on ffmpeg-devel first...

Diego