]> granicus.if.org Git - python/commit
Cut disused recode_encoding logic in _PyBytes_DecodeEscape. (GH-16013)
authorGreg Price <gnprice@gmail.com>
Thu, 12 Sep 2019 18:12:22 +0000 (11:12 -0700)
committerBenjamin Peterson <benjamin@python.org>
Thu, 12 Sep 2019 18:12:22 +0000 (19:12 +0100)
commit3a4f66707e824ef3a8384827590ebaa6ca463dc0
treeed74c3415de2582188da0fa52ac3a9ffb2ac4f63
parenta44f3dc21d8b59ceeffa7dd1d9a7c05ed6be3e04
Cut disused recode_encoding logic in _PyBytes_DecodeEscape. (GH-16013)

All call sites pass NULL for `recode_encoding`, so this path is
completely untested.  That's been true since before Python 3.0.
It adds significant complexity to this logic, so it's best to
take it out.

All call sites now have a literal NULL, and that's been true since
commit 768921cf3 eliminated a conditional (`foo ? bar : NULL`) at
the call site in Python/ast.c where we're parsing a bytes literal.
But even before then, that condition `foo` had been a constant
since unadorned string literals started meaning Unicode, in commit
572dbf8f1 aka v3.0a1~1035 .

The `unicode` parameter is already unused, so mark it as unused too.
The code that acted on it was also taken out before Python 3.0, in
commit 8d30cc014 aka v3.0a1~1031 .

The function (PyBytes_DecodeEscape) is exposed in the API, but it's
never been documented.
Include/bytesobject.h
Include/longobject.h
Objects/bytesobject.c
Python/ast.c