[3.6] bpo-30605: Fix compiling binary regexs with BytesWarnings enabled. (GH-2016) (#2214)
Running our unit tests with `-bb` enabled triggered this failure..
(cherry picked from commit 171b9a354e816eebc6d4c3a8553303942e9c5025)
diff --git a/Lib/sre_parse.py b/Lib/sre_parse.py
index 608f9a2..b505e11 100644
--- a/Lib/sre_parse.py
+++ b/Lib/sre_parse.py
@@ -736,7 +736,7 @@
if not first or subpattern:
import warnings
warnings.warn(
- 'Flags not at the start of the expression %s%s' % (
+ 'Flags not at the start of the expression %r%s' % (
source.string[:20], # truncate long regexes
' (truncated)' if len(source.string) > 20 else '',
),
diff --git a/Lib/test/test_re.py b/Lib/test/test_re.py
index e88d0b3..a6cbbd0 100644
--- a/Lib/test/test_re.py
+++ b/Lib/test/test_re.py
@@ -1346,7 +1346,7 @@
self.assertTrue(re.match(p, lower_char))
self.assertEqual(
str(warns.warnings[0].message),
- 'Flags not at the start of the expression %s' % p
+ 'Flags not at the start of the expression %r' % p
)
self.assertEqual(warns.warnings[0].filename, __file__)
@@ -1355,10 +1355,22 @@
self.assertTrue(re.match(p, lower_char))
self.assertEqual(
str(warns.warnings[0].message),
- 'Flags not at the start of the expression %s (truncated)' % p[:20]
+ 'Flags not at the start of the expression %r (truncated)' % p[:20]
)
self.assertEqual(warns.warnings[0].filename, __file__)
+ # bpo-30605: Compiling a bytes instance regex was throwing a BytesWarning
+ with warnings.catch_warnings():
+ warnings.simplefilter('error', BytesWarning)
+ p = b'A(?i)'
+ with self.assertWarns(DeprecationWarning) as warns:
+ self.assertTrue(re.match(p, b'a'))
+ self.assertEqual(
+ str(warns.warnings[0].message),
+ 'Flags not at the start of the expression %r' % p
+ )
+ self.assertEqual(warns.warnings[0].filename, __file__)
+
with self.assertWarns(DeprecationWarning):
self.assertTrue(re.match('(?s).(?i)' + upper_char, '\n' + lower_char))
with self.assertWarns(DeprecationWarning):
diff --git a/Misc/ACKS b/Misc/ACKS
index c2ebf7f..1f3a0cc 100644
--- a/Misc/ACKS
+++ b/Misc/ACKS
@@ -1663,6 +1663,7 @@
Gerald S. Williams
Jason Williams
John Williams
+Roy Williams
Sue Williams
Carol Willing
Steven Willis
diff --git a/Misc/NEWS b/Misc/NEWS
index 22f980c..a258f52 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -51,6 +51,9 @@
Library
-------
+- bpo-30605: re.compile() no longer raises a BytesWarning when compiling a
+ bytes instance with misplaced inline modifier. Patch by Roy Williams.
+
- [Security] bpo-29591: Update expat copy from 2.1.1 to 2.2.0 to get fixes
of CVE-2016-0718 and CVE-2016-4472. See
https://sourceforge.net/p/expat/bugs/537/ for more information.