commit afdeb189e97033b54cef44a7490d89d2013cb4e5
author Sergey Fedoseev <fedoseev.sergey@gmail.com> Tue Sep 10 21:11:10 2019 +0500
committer Gregory P. Smith <greg@krypto.org> Tue Sep 10 17:11:10 2019 +0100
tree 1fee64e732137f1436be397e491dba58e628fb27
parent 9cbb97b29eac4b23e916a3233f26b60ac69e335b

Remove unneeded assignment in PyBytes_Concat() (GH-15274)

The `wb.len = -1` assignment is unneeded since its introduction in 161d695fb0455ce52530d4f43a9eac4c738f64bb as `PyObject_GetBuffer` always fills it in.

Objects/bytesobject.c

diff --git a/Objects/bytesobject.c b/Objects/bytesobject.c
index 72f9cd2..e1f5ee2 100644
--- a/Objects/bytesobject.c
+++ b/Objects/bytesobject.c
@@ -2935,7 +2935,6 @@
         Py_ssize_t oldsize;
         Py_buffer wb;
 
-        wb.len = -1;
         if (PyObject_GetBuffer(w, &wb, PyBUF_SIMPLE) != 0) {
             PyErr_Format(PyExc_TypeError, "can't concat %.100s to %.100s",
                          Py_TYPE(w)->tp_name, Py_TYPE(*pv)->tp_name);