Home Original page

[Python-Dev] PEP 409 and the stdlib

R. David Murray rdmurray at bitdance.com
Mon May 20 16:50:03 CEST 2013 
On Mon, 20 May 2013 07:12:07 -0700, Ethan Furman <ethan at 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
More information about the Python-Dev mailing list