Nick Coghlan <ncogh...@gmail.com> added the comment:

The entire change affecting the PEP 538 implementation in 
https://github.com/python/cpython/commit/9454060e84a669dde63824d9e2fcaf295e34f687#diff-8c018c3ada66d06c8e101e47a313c2c7
 needs to be reverted: the locale should be coerced before *ANY* calls are made 
to Py_DecodeLocale, as the whole point of the architecture of the PEP 
implementation was to ensure that *nothing ever gets decoded incorrectly in the 
first place*.

I just initially missed that in Victor's enthusiasm for fixing the incorrect 
decodings that can arise in the absence of locale coercion he'd *introduced* 
incorrect decoding into the locale coercion case.

----------
nosy: +ncoghlan, ned.deily
priority: normal -> release blocker

_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue34589>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to