jenkins-bot merged this change.

View Change

Approvals: Lokal Profil: Looks good to me, approved jenkins-bot: Verified
[cleanup] Simplify range checking in proofreadpage.py

Change-Id: I3cc966b44222aa2b48475b0980d6b35ceda2a2fc
---
M pywikibot/proofreadpage.py
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/pywikibot/proofreadpage.py b/pywikibot/proofreadpage.py
index d05f932..4a72548 100644
--- a/pywikibot/proofreadpage.py
+++ b/pywikibot/proofreadpage.py
@@ -953,9 +953,7 @@
if end is None:
end = self.num_pages

- if not ((1 <= start <= self.num_pages)
- and (1 <= end <= self.num_pages)
- and (start <= end)):
+ if not (1 <= start <= end <= self.num_pages):
raise ValueError('start=%s, end=%s are not in valid range (%s, %s)'
% (start, end, 1, self.num_pages))


To view, visit change 512107. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I3cc966b44222aa2b48475b0980d6b35ceda2a2fc
Gerrit-Change-Number: 512107
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: John Vandenberg <jayvdb@gmail.com>
Gerrit-Reviewer: Lokal Profil <andre.costa@wikimedia.se>
Gerrit-Reviewer: jenkins-bot (75)