From: Richard Levitte <[email protected]>

> sms>    Looking through makevms.com, I noticed this:
> sms> 
> sms> $ CONFIG_LOGICALS := 
> NO_ASM,NO_RSA,NO_DSA,NO_DH,NO_MD2,NO_MD5,NO_RIPEMD,WHRLPOOL,-
> sms>         
> NO_SHA,NO_SHA0,NO_SHA1,NO_DES/NO_MDC2;NO_MDC2,NO_RC2,NO_RC4,NO_RC5,-
> sms>         NO_IDEA,NO_BF,NO_CAST,NO_CAMELLIA,NO_SEED,NO_HMAC,NO_SSL2
> sms> 
> sms> I suspect that one could adjust this to add VAX/non-VAX conditionality
> sms> for WHRLPOOL, although it might help to spell it "WHIRLPOOL", because
> sms> the code seems to be looking for "OPENSSL_NO_WHIRLPOOL", not
> sms> "OPENSSL_NO_WHRLPOOL".
> sms> 
> sms>    Has anyone looked at this list in recent times to see if the items in
> sms> it still make sense?
> 
> I'm looking at it right now, and I'm reworking the whole thing to make
> it more comprehensible for starters ;-)

   Having already gotten it to work properly ("WHRLPOOL" ->
"NO_WHIRLPOOL"), I'd settle for getting some comments added (somewhere,
anywhere) to explain how the stuff works and/or should be used.  It
seems pretty clear that whoever stuck (the non-"NO_") "WHRLPOOL" into
that list (of "NO_*" items) could have used a little guidance.  As
usual, I see no good reason for the apparently random order in a list
like this.

>       http://antinode.info/ftp/openssl/1_0_0/test/tests.com

   Note that _I_ added the most recent defect to that procedure, because
I was misled by the (unnecessarily) deviant indenting following the line
containing "tests := -".  If you're looking for something to reform,
then a good way to start would be sucking those annoying, useless TAB
characters out of those procedures in "test/", doing which would reduce
the incentive to damage the indenting in the first place.

------------------------------------------------------------------------

   Steven M. Schweda               s...@antinode-info
   382 South Warwick Street        (+1) 651-699-9818
   Saint Paul  MN  55105-2547
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [email protected]
Automated List Manager                           [email protected]

Reply via email to