Web lists-archives.com

Re: [PATCH 4/5] Makefile: add USE_SHA1DC knob




On Thu, Mar 16, 2017 at 10:24:32PM -0700, Junio C Hamano wrote:

> Jeff King <peff@xxxxxxxx> writes:
> 
> > On Thu, Mar 16, 2017 at 03:43:13PM -0700, Junio C Hamano wrote:
> >
> >> Jeff King <peff@xxxxxxxx> writes:
> >> 
> >> > +ifdef USE_SHA1DC
> >> > +	LIB_OBJS += sha1dc/sha1.o
> >> > +	LIB_OBJS += sha1dc/ubc_check.o
> >> > +	BASIC_CFLAGS += -DSHA1_SHA1DC
> >> 
> >> The name of this CPP symbol is one difference between this and
> >> Linus's version.  Wouldn't "-DSHA1_DC" make more sense?
> >
> > I'm fine with either. Somehow SHA1_DC felt too short, but it doesn't
> > really matter in practice.
> 
> I'm fine with either, too.  It was just double SHA1 felt a bit
> strange, when the naming convention was SHA1_ followed by the
> characteristic attribute of the implementation (e.g. came from
> Mozilla, etc.) and I thought "Detecting Collision" was the notable
> characteristic of this one.

Yeah, I thought "DC" by itself was funny, but I guess "BLK" by itself is
no less funny. I'm happy to re-roll it if you prefer the other.

-Peff