On Mon, 20 May 2013 07:12:07 -0700, Ethan Furman <ethan@stoneleaf.us> wrote:
As a case in point, base64.py is currently getting a bug fix, and also contains this code:
def b32decode(s, casefold=False, map01=None): . . . for i in range(0, len(s), 8): quanta = s[i: i + 8] acc = 0 try: for c in quanta: acc = (acc << 5) + b32rev[c] except KeyError: raise binascii.Error('Non-base32 digit found') . . . else: raise binascii.Error('Incorrect padding')
Does the KeyError qualify as irrelevant noise?
I don't see that it is of benefit to suppress it.
If we're not going to suppress the originating error I think we should at least change the double trace back message as it implies two failures, instead of just one.
I don't understand what you want to do here. --David