[issue35557] Allow lowercase hexadecimal characters in base64.b16decode()

2018-12-22 Thread Dylan Houlihan
Dylan Houlihan added the comment: Karthikeyan, Thank you for taking the time to respond so thoroughly. In particular, in the future I'll be more careful to qualify and quantify potential performance improvements using `timeit` or `perf`. That being said, as I mentioned the primary motivatio

[issue35557] Allow lowercase hexadecimal characters in base64.b16decode()

2018-12-22 Thread Dylan Houlihan
Change by Dylan Houlihan : -- type: performance -> behavior ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue35557] Allow lowercase hexadecimal characters in base64.b16decode()

2018-12-22 Thread Dylan Houlihan
Change by Dylan Houlihan : -- versions: -Python 3.4, Python 3.5, Python 3.6, Python 3.7 ___ Python tracker ___ ___ Python-bugs-list

[issue35557] Allow lowercase hexadecimal characters in base64.b16decode()

2018-12-21 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: This is compatibility breaking change. Furthermore, it changes the purposed behavior that was from the initial version (4c904d1bf71d01bc22fbb123493f975050560e9c). Since currently there is an option which allows to accept lowercase hexadecimal characters,

[issue35557] Allow lowercase hexadecimal characters in base64.b16decode()

2018-12-21 Thread Karthikeyan Singaravelan
Karthikeyan Singaravelan added the comment: Thanks for the report. A couple of points as below : * This changes the interface of the function by removing a parameter. Thus it will break compatibility with Python 2 and also earlier versions of Python 3. Removing a parameter in the signature

[issue35557] Allow lowercase hexadecimal characters in base64.b16decode()

2018-12-21 Thread Dylan Houlihan
Change by Dylan Houlihan : -- keywords: +patch pull_requests: +10512 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-

[issue35557] Allow lowercase hexadecimal characters in base64.b16decode()

2018-12-21 Thread Dylan Houlihan
New submission from Dylan Houlihan : Currently, the `base64` method `b16decode` does not decode a hexadecimal string with lowercase characters by default. To do so requires passing `casefold=True` as the second argument. I propose a change to the `b16decode` method to allow it to accept hexad