jenkins-bot has submitted this change. ( 
https://gerrit.wikimedia.org/r/c/pywikibot/core/+/759886 )

Change subject: [IMPR] reduce similar code in date.py
......................................................................

[IMPR] reduce similar code in date.py

Change-Id: Ie0b721b54deb621373a8e057f7990bd6899c08e5
---
M pywikibot/date.py
1 file changed, 16 insertions(+), 16 deletions(-)

Approvals:
  Xqt: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/pywikibot/date.py b/pywikibot/date.py
index b6f4a42..5367237 100644
--- a/pywikibot/date.py
+++ b/pywikibot/date.py
@@ -1837,30 +1837,30 @@
 addFmt1('zh', False, makeMonthList('%d月%%d日'))

 # Walloon names depend on the day number, thus we must generate various
-# different patterns
+# different patterns:

 # For month names beginning with a consonant...
-for i in (0, 1, 2, 4, 5, 6, 8, 10, 11):
-    formats[dayMnthFmts[i]]['wa'] = eval(
-        'lambda m: multi(m, ['
-        '(lambda v: dh_dayOfMnth(v, "%dî d\' {mname}"), lambda p: p == 1), '
-        '(lambda v: dh_dayOfMnth(v, "%d d\' {mname}"), '
-        'lambda p: p in [2,3,20,22,23]), '
-        '(lambda v: dh_dayOfMnth(v, "%d di {mname}"), alwaysTrue)])'
-        .format(mname=waMonthNames[i]))
-
+_consonant_pattern = (
+    'lambda m: multi(m, ['
+    '(lambda v: dh_dayOfMnth(v, "%dî d\' {mname}"), lambda p: p == 1), '
+    '(lambda v: dh_dayOfMnth(v, "%d d\' {mname}"), '
+    'lambda p: p in [2,3,20,22,23]), '
+    '(lambda v: dh_dayOfMnth(v, "%d di {mname}"), alwaysTrue)])'
+)
 # For month names beginning with a vowel...
-for i in (3, 7, 9):
-    formats[dayMnthFmts[i]]['wa'] = eval(
-        'lambda m: multi(m, ['
-        '(lambda v: dh_dayOfMnth(v, "%dî d\' {mname}"), lambda p: p == 1), '
-        '(lambda v: dh_dayOfMnth(v, "%d d\' {mname}"), alwaysTrue)])'
-        .format(mname=waMonthNames[i]))
+_vowel_pattern = (
+    'lambda m: multi(m, ['
+    '(lambda v: dh_dayOfMnth(v, "%dî d\' {mname}"), lambda p: p == 1), '
+    '(lambda v: dh_dayOfMnth(v, "%d d\' {mname}"), alwaysTrue)])'
+)

 # Brazil uses '1añ' for the 1st of every month, and number without suffix for
 # all other days
 brMonthNames = makeMonthNamedList('br', '%s', True)
+
 for i in range(12):
+    pattern = _vowel_pattern if i in (3, 7, 9) else _consonant_pattern
+    formats[dayMnthFmts[i]]['wa'] = eval(pattern.format(mname=waMonthNames[i]))
     formats[dayMnthFmts[i]]['br'] = eval(
         'lambda m: multi(m, ['
         '(lambda v: dh_dayOfMnth(v, "%dañ {mname}"), lambda p: p == 1), '

--
To view, visit https://gerrit.wikimedia.org/r/c/pywikibot/core/+/759886
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.wikimedia.org/r/settings

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Ie0b721b54deb621373a8e057f7990bd6899c08e5
Gerrit-Change-Number: 759886
Gerrit-PatchSet: 2
Gerrit-Owner: Xqt <[email protected]>
Gerrit-Reviewer: Xqt <[email protected]>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged
_______________________________________________
Pywikibot-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to