jenkins-bot merged this change.

View Change

Approvals: Dvorapa: Looks good to me, but someone else must approve Framawiki: Looks good to me, approved jenkins-bot: Verified
Use compiled regex object instead of re.sub

Change-Id: I231eadde4b6b17955ff4fbf0fd66d6bdfd252faa
---
M scripts/solve_disambiguation.py
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/solve_disambiguation.py b/scripts/solve_disambiguation.py
index e921b4c..dbbd6f0 100755
--- a/scripts/solve_disambiguation.py
+++ b/scripts/solve_disambiguation.py
@@ -965,8 +965,8 @@
(len(new_page_title) <= len(link_text)) and
(firstcap(link_text[:len(new_page_title)]) ==
firstcap(new_page_title)) and
- (re.sub(self.trailR, '',
- link_text[len(new_page_title):]) == '') and
+ (self.trailR.sub(
+ '', link_text[len(new_page_title):]) == '') and
(not section)
):
newlink = "[[%s]]%s" \

To view, visit change 438062. To unsubscribe, visit settings.

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I231eadde4b6b17955ff4fbf0fd66d6bdfd252faa
Gerrit-Change-Number: 438062
Gerrit-PatchSet: 1
Gerrit-Owner: Matěj Suchánek <matejsuchanek97@gmail.com>
Gerrit-Reviewer: Dvorapa <dvorapa@seznam.cz>
Gerrit-Reviewer: Framawiki <framawiki@tools.wmflabs.org>
Gerrit-Reviewer: John Vandenberg <jayvdb@gmail.com>
Gerrit-Reviewer: Zoranzoki21 <zorandori4444@gmail.com>
Gerrit-Reviewer: jenkins-bot <>