comparison hgext/phabricator.py @ 41532:bd3f03d8cc9f

global: use raw strings for regular expressions with escapes Escape sequences like \w, \s, and \d are technically invalid in str/bytes. This became a deprecation warning in Python 3.6 (https://bugs.python.org/issue27364). Python 3.8 bumps it to a SyntaxWarning (https://bugs.python.org/issue32912), which is non-silent by default. This commit changes a number of regular expressions to use br'' so regular expression special sequences don't need \\ literals. This fixes roughly half of the SyntaxWarning we see in the code base with Python 3.8. Differential Revision: https://phab.mercurial-scm.org/D5815
author Gregory Szorc <gregory.szorc@gmail.com>
date Mon, 04 Feb 2019 08:54:30 -0800
parents 876494fd967d
children 570e62f1dcf2
comparison
equal deleted inserted replaced
41531:14983ac4a764 41532:bd3f03d8cc9f
253 return None 253 return None
254 repophid = encoding.strtolocal(query[r'data'][0][r'phid']) 254 repophid = encoding.strtolocal(query[r'data'][0][r'phid'])
255 repo.ui.setconfig(b'phabricator', b'repophid', repophid) 255 repo.ui.setconfig(b'phabricator', b'repophid', repophid)
256 return repophid 256 return repophid
257 257
258 _differentialrevisiontagre = re.compile(b'\AD([1-9][0-9]*)\Z') 258 _differentialrevisiontagre = re.compile(br'\AD([1-9][0-9]*)\Z')
259 _differentialrevisiondescre = re.compile( 259 _differentialrevisiondescre = re.compile(
260 b'^Differential Revision:\s*(?P<url>(?:.*)D(?P<id>[1-9][0-9]*))$', re.M) 260 br'^Differential Revision:\s*(?P<url>(?:.*)D(?P<id>[1-9][0-9]*))$', re.M)
261 261
262 def getoldnodedrevmap(repo, nodelist): 262 def getoldnodedrevmap(repo, nodelist):
263 """find previous nodes that has been sent to Phabricator 263 """find previous nodes that has been sent to Phabricator
264 264
265 return {node: (oldnode, Differential diff, Differential Revision ID)} 265 return {node: (oldnode, Differential diff, Differential Revision ID)}