From 1647ce7e60bb467a9cd3ada06179902443bc7623 Mon Sep 17 00:00:00 2001 From: Nozomu Kaneko Date: Sat, 15 Dec 2012 16:06:20 +0900 Subject: [PATCH] rather than use two-ifs, do it in one if-elif-else branch. --- sphinx/ext/refcounting.py | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/sphinx/ext/refcounting.py b/sphinx/ext/refcounting.py index 1d2a353a16..c76d25d260 100644 --- a/sphinx/ext/refcounting.py +++ b/sphinx/ext/refcounting.py @@ -79,14 +79,12 @@ class Refcounts(dict): continue elif entry.result_type not in ("PyObject*", "PyVarObject*"): continue - rc = _('Return value: ') if entry.result_refs is None: - rc += _("Always NULL.") + rc = _('Return value: Always NULL.') + elif entry.result_refs: + rc = _('Return value: New reference.') else: - if entry.result_refs: - rc += _("New reference.") - else: - rc += _("Borrowed reference.") + rc = _('Return value: Borrowed reference.') node.insert(0, refcount(rc, rc))