diff options
author | Bram Moolenaar <Bram@vim.org> | 2013-06-23 12:51:32 +0200 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2013-06-23 12:51:32 +0200 |
commit | 6c85e7f3be87fa4a669e8f080d527a3982e8e96c (patch) | |
tree | b2b247ced220135504ef4781d1a134632eaa9bda | |
parent | 3dbcd0c7ad93533f46981ebafe6710c1e03b1be2 (diff) | |
download | vim-git-6c85e7f3be87fa4a669e8f080d527a3982e8e96c.tar.gz |
updated for version 7.3.1226v7.3.1226
Problem: Python: duplicate code.
Solution: Share code between OutputWrite() and OutputWritelines(). (ZyX)
-rw-r--r-- | src/if_py_both.h | 50 | ||||
-rw-r--r-- | src/testdir/test86.ok | 2 | ||||
-rw-r--r-- | src/testdir/test87.ok | 2 | ||||
-rw-r--r-- | src/version.c | 2 |
4 files changed, 26 insertions, 30 deletions
diff --git a/src/if_py_both.h b/src/if_py_both.h index 9acb3105b..164e2a42a 100644 --- a/src/if_py_both.h +++ b/src/if_py_both.h @@ -281,15 +281,15 @@ writer(writefn fn, char_u *str, PyInt n) } } - static PyObject * -OutputWrite(OutputObject *self, PyObject *args) + static int +write_output(OutputObject *self, PyObject *string) { - Py_ssize_t len = 0; - char *str = NULL; - int error = self->error; + Py_ssize_t len = 0; + char *str = NULL; + int error = self->error; - if (!PyArg_ParseTuple(args, "et#", ENC_OPT, &str, &len)) - return NULL; + if (!PyArg_Parse(string, "et#", ENC_OPT, &str, &len)) + return -1; Py_BEGIN_ALLOW_THREADS Python_Lock_Vim(); @@ -298,44 +298,37 @@ OutputWrite(OutputObject *self, PyObject *args) Py_END_ALLOW_THREADS PyMem_Free(str); + return 0; +} + + static PyObject * +OutputWrite(OutputObject *self, PyObject *string) +{ + if (write_output(self, string)) + return NULL; + Py_INCREF(Py_None); return Py_None; } static PyObject * -OutputWritelines(OutputObject *self, PyObject *args) +OutputWritelines(OutputObject *self, PyObject *seq) { - PyObject *seq; PyObject *iterator; PyObject *item; - int error = self->error; - - if (!PyArg_ParseTuple(args, "O", &seq)) - return NULL; if (!(iterator = PyObject_GetIter(seq))) return NULL; while ((item = PyIter_Next(iterator))) { - char *str = NULL; - PyInt len; - - if (!PyArg_Parse(item, "et#", ENC_OPT, &str, &len)) + if (write_output(self, item)) { - PyErr_SetString(PyExc_TypeError, _("writelines() requires list of strings")); Py_DECREF(iterator); Py_DECREF(item); return NULL; } Py_DECREF(item); - - Py_BEGIN_ALLOW_THREADS - Python_Lock_Vim(); - writer((writefn)(error ? emsg : msg), (char_u *)str, len); - Python_Release_Vim(); - Py_END_ALLOW_THREADS - PyMem_Free(str); } Py_DECREF(iterator); @@ -360,8 +353,8 @@ OutputFlush(PyObject *self UNUSED) static struct PyMethodDef OutputMethods[] = { /* name, function, calling, doc */ - {"write", (PyCFunction)OutputWrite, METH_VARARGS, ""}, - {"writelines", (PyCFunction)OutputWritelines, METH_VARARGS, ""}, + {"write", (PyCFunction)OutputWrite, METH_O, ""}, + {"writelines", (PyCFunction)OutputWritelines, METH_O, ""}, {"flush", (PyCFunction)OutputFlush, METH_NOARGS, ""}, {"__dir__", (PyCFunction)OutputDir, METH_NOARGS, ""}, { NULL, NULL, 0, NULL} @@ -3009,7 +3002,8 @@ TabListItem(PyObject *self UNUSED, PyInt n) return NULL; } -/* Window object +/* + * Window object */ typedef struct diff --git a/src/testdir/test86.ok b/src/testdir/test86.ok index a4f1ac405..05d194338 100644 --- a/src/testdir/test86.ok +++ b/src/testdir/test86.ok @@ -444,7 +444,7 @@ sys.stdout.attr = None:AttributeError:('invalid attribute',) sys.stdout.write(None):TypeError:('coercing to Unicode: need string or buffer, NoneType found',) >> OutputWriteLines sys.stdout.writelines(None):TypeError:("'NoneType' object is not iterable",) -sys.stdout.writelines([1]):TypeError:('writelines() requires list of strings',) +sys.stdout.writelines([1]):TypeError:('coercing to Unicode: need string or buffer, int found',) > VimCommand vim.command(1):TypeError:('must be string, not int',) > VimToPython diff --git a/src/testdir/test87.ok b/src/testdir/test87.ok index 1d4044519..414dc9e42 100644 --- a/src/testdir/test87.ok +++ b/src/testdir/test87.ok @@ -433,7 +433,7 @@ sys.stdout.attr = None:(<class 'AttributeError'>, AttributeError('invalid attrib sys.stdout.write(None):(<class 'TypeError'>, TypeError("Can't convert 'NoneType' object to str implicitly",)) >> OutputWriteLines sys.stdout.writelines(None):(<class 'TypeError'>, TypeError("'NoneType' object is not iterable",)) -sys.stdout.writelines([1]):(<class 'TypeError'>, TypeError('writelines() requires list of strings',)) +sys.stdout.writelines([1]):(<class 'TypeError'>, TypeError("Can't convert 'int' object to str implicitly",)) >>> Testing *Iter* using sys.stdout.writelines(%s) sys.stdout.writelines(FailingIter()):(<class 'NotImplementedError'>, NotImplementedError()) sys.stdout.writelines(FailingIterNext()):(<class 'NotImplementedError'>, NotImplementedError()) diff --git a/src/version.c b/src/version.c index 4d2e7a962..8e33ab621 100644 --- a/src/version.c +++ b/src/version.c @@ -729,6 +729,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1226, +/**/ 1225, /**/ 1224, |