]> granicus.if.org Git - python/commitdiff
bpo-30246: fix several error messages which only mention bytes in struct (#1421)
authorXiang Zhang <angwerzx@126.com>
Thu, 14 Sep 2017 02:33:26 +0000 (10:33 +0800)
committerGitHub <noreply@github.com>
Thu, 14 Sep 2017 02:33:26 +0000 (10:33 +0800)
Modules/_struct.c

index 69a1e996d05d3ded56e6d8a8e0050ca0ed9be18a..04d7f8e1871925f5f22817cecbd63df4e696757f 100644 (file)
@@ -1453,7 +1453,8 @@ Struct___init___impl(PyStructObject *self, PyObject *format)
     if (!PyBytes_Check(format)) {
         Py_DECREF(format);
         PyErr_Format(PyExc_TypeError,
-                     "Struct() argument 1 must be a bytes object, not %.200s",
+                     "Struct() argument 1 must be a str or bytes object, "
+                     "not %.200s",
                      Py_TYPE(format)->tp_name);
         return -1;
     }
@@ -1535,7 +1536,7 @@ Struct_unpack_impl(PyStructObject *self, Py_buffer *buffer)
     assert(self->s_codes != NULL);
     if (buffer->len != self->s_size) {
         PyErr_Format(StructError,
-                     "unpack requires a bytes object of length %zd",
+                     "unpack requires a buffer of %zd bytes",
                      self->s_size);
         return NULL;
     }
@@ -1708,8 +1709,8 @@ Struct_iter_unpack(PyStructObject *self, PyObject *buffer)
     }
     if (iter->buf.len % self->s_size != 0) {
         PyErr_Format(StructError,
-                     "iterative unpacking requires a bytes length "
-                     "multiple of %zd",
+                     "iterative unpacking requires a buffer of "
+                     "a multiple of %zd bytes",
                      self->s_size);
         Py_DECREF(iter);
         return NULL;