This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author gregory.p.smith
Recipients christian.heimes, gregory.p.smith, pitrou, python-dev
Date 2013-10-22.20:03:06
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <CAGE7PNKkhuxsdY+924BiKQHKgcxfpAtbHvFpC1cZ_aaB2_J2Ow@mail.gmail.com>
In-reply-to <1376832556.53.0.90130150517.issue18742@psf.upfronthosting.co.za>
Content
>
> Have you had a chance to read http://www.python.org/dev/peps/pep-0452/ ,
too?

Overall pep 452 looks good but one thing popped out at me:

        >>> import hashlib
        >>> from Crypto.Hash import MD5
        >>> m = MD5.new()
        >>> isinstance(m, hashlib.CryptoHash)
        True

This suggests that there is an ABC hashlib.CryptoHash that non hashlib
implementations sound use but that is not spelled out anywhere as required
or recommended for third party hash implementations or why using it is a
good thing. I believe that is what you want to do with the ABC being
defined in this issue so I'd mention that explicitly in the pep.
History
Date User Action Args
2013-10-22 20:03:06gregory.p.smithsetrecipients: + gregory.p.smith, pitrou, christian.heimes, python-dev
2013-10-22 20:03:06gregory.p.smithlinkissue18742 messages
2013-10-22 20:03:06gregory.p.smithcreate