Patchwork D6997: contrib: fix check-code to be able to detect missing _() with bytestrings

login
register
mail settings
Submitter phabricator
Date Oct. 6, 2019, 4:06 p.m.
Message ID <8959bce6bd2000db00d6ed9fd4545ca8@localhost.localdomain>
Download mbox | patch
Permalink /patch/42031/
State Not Applicable
Headers show

Comments

phabricator - Oct. 6, 2019, 4:06 p.m.
Closed by commit rHGe65e7290041e: contrib: fix check-code to be able to detect missing _() with bytestrings (authored by durin42).
This revision was automatically updated to reflect the committed changes.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D6997?vs=16888&id=16908

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D6997/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D6997

AFFECTED FILES
  contrib/check-code.py

CHANGE DETAILS




To: durin42, #hg-reviewers, indygreg
Cc: mercurial-devel

Patch

diff --git a/contrib/check-code.py b/contrib/check-code.py
--- a/contrib/check-code.py
+++ b/contrib/check-code.py
@@ -508,7 +508,7 @@ 
          [ \t\n#]*
          (?# any strings/comments might precede a string, which
            # contains translatable message)
-         ((['"]|\'\'\'|""")[ \npq%bAPMxno]*(['"]|\'\'\'|""")[ \t\n#]+)*
+         b?((['"]|\'\'\'|""")[ \npq%bAPMxno]*(['"]|\'\'\'|""")[ \t\n#]+)*
          (?# sequence consisting of below might precede translatable message
            # - formatting string: "% 10s", "%05d", "% -3.2f", "%*s", "%%" ...
            # - escaped character: "\\", "\n", "\0" ...