diff options
author | Victor Stinner <victor.stinner@gmail.com> | 2013-07-11 23:05:03 +0200 |
---|---|---|
committer | Victor Stinner <victor.stinner@gmail.com> | 2013-07-11 23:05:03 +0200 |
commit | b0b4a0a14f5f21613e9826f9cbb72668ee9a9f85 (patch) | |
tree | e3183c55949a753d3971a37a47cea0f5c386f28f | |
parent | c9f3150b4c9e59aa30232b633028f616005cec5d (diff) | |
download | cpython-b0b4a0a14f5f21613e9826f9cbb72668ee9a9f85.tar.gz |
Cleanup _elementtree.c
-rw-r--r-- | Modules/_elementtree.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/Modules/_elementtree.c b/Modules/_elementtree.c index 1957abfafd..a5b3ec5eae 100644 --- a/Modules/_elementtree.c +++ b/Modules/_elementtree.c @@ -529,7 +529,6 @@ subelement(PyObject *self, PyObject *args, PyObject *kwds) } elem = create_new_element(tag, attrib); - Py_DECREF(attrib); if (element_add_subelement(parent, elem) < 0) { @@ -1784,10 +1783,10 @@ element_setattro(ElementObject* self, PyObject* nameobj, PyObject* value) char *name = ""; if (PyUnicode_Check(nameobj)) name = _PyUnicode_AsString(nameobj); - - if (name == NULL) { + if (name == NULL) return -1; - } else if (strcmp(name, "tag") == 0) { + + if (strcmp(name, "tag") == 0) { Py_DECREF(self->tag); self->tag = value; Py_INCREF(self->tag); @@ -2135,15 +2134,15 @@ create_elementiter(ElementObject *self, PyObject *tag, int gettext) if (star && PyObject_RichCompareBool(tag, star, Py_EQ) == 1) tag = Py_None; - Py_XDECREF(star); + + Py_INCREF(tag); it->sought_tag = tag; it->root_done = 0; it->gettext = gettext; + Py_INCREF(self); it->root_element = self; - Py_INCREF(self); - Py_INCREF(tag); PyObject_GC_Track(it); return (PyObject *)it; |