comparison contrib/check-code.py @ 43551:313e3a279828

cleanup: remove pointless r-prefixes on double-quoted strings This is only double-quoted strings. I'll do single-quoted strings as a second step. These had existed because our source transformer didn't turn r"" into b"", so we had tagged some strings as r-strings to get "native" strings on both Pythons. Now that the transformer is gone, we can dispense with this nonsense. Methodology: I ran hg locate 'set:added() or modified() or clean()' | egrep '.*\.py$' | xargs egrep --color=never -n -- \[\^a-z\]r\"\[\^\"\\\\\]\*\"\[\^\"\] in an emacs grep-mode buffer, and then used a keyboard macro to iterate over the results and remove the r prefix as needed. # skip-blame removing unneeded r prefixes left over from Python 3 migration. Differential Revision: https://phab.mercurial-scm.org/D7305
author Augie Fackler <augie@google.com>
date Thu, 07 Nov 2019 13:18:19 -0500
parents 24a07347aa60
children c102b704edb5
comparison
equal deleted inserted replaced
43550:c093cc6e6c99 43551:313e3a279828
279 # transform plain test rules to unified test's 279 # transform plain test rules to unified test's
280 for i in [0, 1]: 280 for i in [0, 1]:
281 for tp in testpats[i]: 281 for tp in testpats[i]:
282 p = tp[0] 282 p = tp[0]
283 m = tp[1] 283 m = tp[1]
284 if p.startswith(r'^'): 284 if p.startswith('^'):
285 p = r"^ [$>] (%s)" % p[1:] 285 p = "^ [$>] (%s)" % p[1:]
286 else: 286 else:
287 p = r"^ [$>] .*(%s)" % p 287 p = "^ [$>] .*(%s)" % p
288 utestpats[i].append((p, m) + tp[2:]) 288 utestpats[i].append((p, m) + tp[2:])
289 289
290 # don't transform the following rules: 290 # don't transform the following rules:
291 # " > \t" and " \t" should be allowed in unified tests 291 # " > \t" and " \t" should be allowed in unified tests
292 testpats[0].append((r'^( *)\t', "don't use tabs to indent")) 292 testpats[0].append((r'^( *)\t', "don't use tabs to indent"))