On Mar 2, 2012, at 8:19 AM, Max Leske wrote:

> I need to calculate CRC for Git and couldn't find it in the system packages 
> (I guess I'd find it in Cryptography). Since Pharo has a system package 
> called "System-Hashing" I was wondering if anyone sees a reason why CRC 
> should not be part of that package. That would also allow the Zip 
> implementation to use the system CRC function (at the moment Zip implements 
> its own CRC check). 
> 
Yes, that would be nice. But people will yell at you for making Pharo bloated 
and going in the completely wrong direction and
that you should and never ever add any code.

> Another thing: going through Sytem-Hashing I noticed that there are two 
> implementations for SHA1: "SHA1" and "SecureHashAlgorithm". They both have 
> the exact same class comment but vary slightly in the messages they implement 
> (although most of them are present in both classes). Can we remove one?

Yes. But for sure someone will yell at you if you remove it, because the system 
should be stable and not force people to 
change their code. 

        Marcus

--
Marcus Denker -- http://marcusdenker.de


Reply via email to