8000 gh-133968: Use private unicode writer for json by nineteendo · Pull Request #133832 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-133968: Use private unicode writer for json #133832

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
May 31, 2025
Merged

gh-133968: Use private unicode writer for json #133832

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
9ea12ad
Use private unicode writer for json
nineteendo May 10, 2025
46df04f
Part 2
nineteendo May 10, 2025
ab1aa42
Restore fast path for integers
nineteendo May 10, 2025
d18c455
Reduce diff
nineteendo May 10, 2025
51c760f
Include necessary headers
nineteendo May 10, 2025
72ae3d0
Use PyUnicodeWriter_WriteRepr
nineteendo May 10, 2025
2a6ec43
Reduce diff
nineteendo May 13, 2025
49a92f3
Merge branch 'main' into json-private-unicode-writer
nineteendo May 13, 2025
822ea86
Reduce diff
nineteendo May 13, 2025
01c45a9
Avoid heap allocation
nineteendo May 15, 2025
566637c
Merge branch 'main' into json-private-unicode-writer
nineteendo May 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Merge branch 'main' into json-private-unicode-writer
  • Loading branch information
@nineteendo
nineteendo committed May 30, 2025
commit 566637c24ae1eb92345ee175b6a71096602fd973
10 changes: 5 additions & 5 deletions Modules/_json.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1470,13 +1470,13 @@ encoder_listencode_obj(PyEncoderObject *s, PyUnicodeWriter *writer,
int rv;

if (obj == Py_None) {
return _PyUnicodeWriter_WriteASCIIString((_PyUnicodeWriter *)writer, "null", 4);
return PyUnicodeWriter_WriteASCII(writer, "null", 4);
}
else if (obj == Py_True) {
return _PyUnicodeWriter_WriteASCIIString((_PyUnicodeWriter *)writer, "true", 4);
return PyUnicodeWriter_WriteASCII(writer, "true", 4);
}
else if (obj == Py_False) {
return _PyUnicodeWriter_WriteASCIIString((_PyUnicodeWriter *)writer, "false", 5);
return PyUnicodeWriter_WriteASCII(writer, "false", 5);
}
else if (PyUnicode_Check(obj)) {
PyObject *encoded = encoder_encode_string(s, obj);
Expand Down Expand Up @@ -1643,7 +1643,7 @@ encoder_listencode_dict(PyEncoderObject *s, PyUnicodeWriter *writer,

if (PyDict_GET_SIZE(dct) == 0) {
/* Fast path */
return _PyUnicodeWriter_WriteASCIIString((_PyUnicodeWriter *)writer, "{}", 2);
return PyUnicodeWriter_WriteASCII(writer, "{}", 2);
}

if (s->markers != Py_None) {
Expand Down Expand Up @@ -1747,7 +1747,7 @@ encoder_listencode_list(PyEncoderObject *s, PyUnicodeWriter *writer,
return -1;
if (PySequence_Fast_GET_SIZE(s_fast) == 0) {
Py_DECREF(s_fast);
return _PyUnicodeWriter_WriteASCIIString((_PyUnicodeWriter *)writer, "[]", 2);
return PyUnicodeWriter_WriteASCII(writer, "[]", 2);
}

if (s->markers != Py_None) {
Expand Down
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.
0