summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVictor Stinner <vstinner@python.org>2024-10-09 15:12:11 (GMT)
committerGitHub <noreply@github.com>2024-10-09 15:12:11 (GMT)
commit6a39e96ab8ebc1144f713988ac6fe439e4476488 (patch)
tree20f4c0367c472d07ca66c3ea15fa5c98c70af70d
parent3ee474f5683110e153fdd0cbd2024f99d6c124e5 (diff)
downloadcpython-6a39e96ab8ebc1144f713988ac6fe439e4476488.zip
cpython-6a39e96ab8ebc1144f713988ac6fe439e4476488.tar.gz
cpython-6a39e96ab8ebc1144f713988ac6fe439e4476488.tar.bz2
gh-115754: Use Py_GetConstant(Py_CONSTANT_EMPTY_BYTES) (#125195)
Replace PyBytes_FromString("") and PyBytes_FromStringAndSize("", 0) with Py_GetConstant(Py_CONSTANT_EMPTY_BYTES).
-rw-r--r--Modules/_ctypes/_ctypes.c4
-rw-r--r--Modules/_io/textio.c2
-rw-r--r--Modules/mmapmodule.c4
-rw-r--r--Modules/zlibmodule.c4
-rw-r--r--Objects/bytesobject.c2
-rw-r--r--Parser/action_helpers.c2
6 files changed, 9 insertions, 9 deletions
diff --git a/Modules/_ctypes/_ctypes.c b/Modules/_ctypes/_ctypes.c
index 951e691..9453ed3 100644
--- a/Modules/_ctypes/_ctypes.c
+++ b/Modules/_ctypes/_ctypes.c
@@ -4732,7 +4732,7 @@ Array_subscript(PyObject *myself, PyObject *item)
char *dest;
if (slicelen <= 0)
- return PyBytes_FromStringAndSize("", 0);
+ return Py_GetConstant(Py_CONSTANT_EMPTY_BYTES);
if (step == 1) {
return PyBytes_FromStringAndSize(ptr + start,
slicelen);
@@ -5418,7 +5418,7 @@ Pointer_subscript(PyObject *myself, PyObject *item)
char *dest;
if (len <= 0)
- return PyBytes_FromStringAndSize("", 0);
+ return Py_GetConstant(Py_CONSTANT_EMPTY_BYTES);
if (step == 1) {
return PyBytes_FromStringAndSize(ptr + start,
len);
diff --git a/Modules/_io/textio.c b/Modules/_io/textio.c
index 439e26c..68d1636 100644
--- a/Modules/_io/textio.c
+++ b/Modules/_io/textio.c
@@ -559,7 +559,7 @@ _io_IncrementalNewlineDecoder_getstate_impl(nldecoder_object *self)
Py_DECREF(state);
}
else {
- buffer = PyBytes_FromString("");
+ buffer = Py_GetConstant(Py_CONSTANT_EMPTY_BYTES);
flag = 0;
}
flag <<= 1;
diff --git a/Modules/mmapmodule.c b/Modules/mmapmodule.c
index 99a85e9..e1c26e1 100644
--- a/Modules/mmapmodule.c
+++ b/Modules/mmapmodule.c
@@ -486,7 +486,7 @@ mmap_read_line_method(mmap_object *self,
remaining = (self->pos < self->size) ? self->size - self->pos : 0;
if (!remaining)
- return PyBytes_FromString("");
+ return Py_GetConstant(Py_CONSTANT_EMPTY_BYTES);
start = self->data + self->pos;
if (safe_memchr(&eol, start, '\n', remaining) < 0) {
@@ -1274,7 +1274,7 @@ mmap_subscript(mmap_object *self, PyObject *item)
CHECK_VALID(NULL);
if (slicelen <= 0)
- return PyBytes_FromStringAndSize("", 0);
+ return Py_GetConstant(Py_CONSTANT_EMPTY_BYTES);
else if (step == 1)
return _safe_PyBytes_FromStringAndSize(self->data + start, slicelen);
else {
diff --git a/Modules/zlibmodule.c b/Modules/zlibmodule.c
index c5aaf22..78dcce7 100644
--- a/Modules/zlibmodule.c
+++ b/Modules/zlibmodule.c
@@ -267,12 +267,12 @@ newcompobject(PyTypeObject *type)
self->eof = 0;
self->is_initialised = 0;
self->zdict = NULL;
- self->unused_data = PyBytes_FromStringAndSize("", 0);
+ self->unused_data = Py_GetConstant(Py_CONSTANT_EMPTY_BYTES);
if (self->unused_data == NULL) {
Py_DECREF(self);
return NULL;
}
- self->unconsumed_tail = PyBytes_FromStringAndSize("", 0);
+ self->unconsumed_tail = Py_GetConstant(Py_CONSTANT_EMPTY_BYTES);
if (self->unconsumed_tail == NULL) {
Py_DECREF(self);
return NULL;
diff --git a/Objects/bytesobject.c b/Objects/bytesobject.c
index 58a4fee..bf58e55 100644
--- a/Objects/bytesobject.c
+++ b/Objects/bytesobject.c
@@ -1638,7 +1638,7 @@ bytes_subscript(PyObject *op, PyObject* item)
&stop, step);
if (slicelength <= 0) {
- return PyBytes_FromStringAndSize("", 0);
+ return Py_GetConstant(Py_CONSTANT_EMPTY_BYTES);
}
else if (start == 0 && step == 1 &&
slicelength == PyBytes_GET_SIZE(self) &&
diff --git a/Parser/action_helpers.c b/Parser/action_helpers.c
index 1972c60..24b817c 100644
--- a/Parser/action_helpers.c
+++ b/Parser/action_helpers.c
@@ -1541,7 +1541,7 @@ _PyPegen_concatenate_strings(Parser *p, asdl_expr_seq *strings,
}
if (bytes_found) {
- PyObject* res = PyBytes_FromString("");
+ PyObject* res = Py_GetConstant(Py_CONSTANT_EMPTY_BYTES);
/* Bytes literals never get a kind, but just for consistency
since they are represented as Constant nodes, we'll mirror