[Ffmpeg-cvslog] r5592 - trunk/libavutil/md5.c

Diego Biurrun diego
Mon Jul 3 23:40:59 CEST 2006


On Mon, Jul 03, 2006 at 11:22:57AM +0200, Michael Niedermayer wrote:
> 
> On Mon, Jul 03, 2006 at 11:16:14AM +0200, lu_zero wrote:
> > Author: lu_zero
> > Date: Mon Jul  3 11:16:13 2006
> > New Revision: 5592
> > 
> > Modified:
> >    trunk/libavutil/md5.c
> > 
> > Log:
> > big endian fix
> > 
> > --- trunk/libavutil/md5.c	(original)
> > +++ trunk/libavutil/md5.c	Mon Jul  3 11:16:13 2006
> > @@ -69,7 +69,11 @@
> >          }\
> >          a = b + (( a << t ) | ( a >> (32 - t) ));
> >  
> > +#ifdef WORDS_BIGENDIAN
> > +static void body(uint32_t ABCD[4], uint32_t X[16]){
> > +#else
> >  static void body(uint32_t ABCD[4], const uint32_t X[16]){
> > +#endif
> 
> iam against this, either make it const or not but not different depending on
> endianness
> 
> and md5.c is maintained, this is a development policy violation, you should
> have sent a patch / have asked first on ffmpeg-dev, if then noone awnsers
> then you can commit your patch but dont just change other developers code
> at will

Well, I think you could say that Luca is the big-endian maintainer, so I
don't think he overstepped his privileges in this case...

Diego




More information about the ffmpeg-cvslog mailing list