jenkins-bot submitted this change.

View Change

Approvals: Xqt: Looks good to me, approved Matěj Suchánek: Looks good to me, approved jenkins-bot: Verified
[IMPR] Split initializer and put getting whitelist to its own method

Decrease code complexity of initializer

Change-Id: I7a4245d061a281524e5ba9b5ac619b38566c9e28
---
M scripts/casechecker.py
1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/scripts/casechecker.py b/scripts/casechecker.py
index f744990..4e76a00 100755
--- a/scripts/casechecker.py
+++ b/scripts/casechecker.py
@@ -194,8 +194,10 @@
.format(ascii=ascii_letters,
local=self.localLtr,
bad=badPtrnStr))
+ self.get_whitelist()

- # Get whitelist
+ def get_whitelist(self):
+ """Get whitelist."""
self.knownWords = set()
self.seenUnresolvedLinks = set()


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

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: I7a4245d061a281524e5ba9b5ac619b38566c9e28
Gerrit-Change-Number: 626849
Gerrit-PatchSet: 3
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: D3r1ck01 <xsavitar.wiki@aol.com>
Gerrit-Reviewer: Matěj Suchánek <matejsuchanek97@gmail.com>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged