Skip to content

bpo-30021: Add examples for re.escape().#1048

Merged
serhiy-storchaka merged 3 commits into
python:masterfrom
serhiy-storchaka:re-escape-examples
Apr 13, 2017
Merged

bpo-30021: Add examples for re.escape().#1048
serhiy-storchaka merged 3 commits into
python:masterfrom
serhiy-storchaka:re-escape-examples

Conversation

@serhiy-storchaka
Copy link
Copy Markdown
Member

No description provided.

@mention-bot
Copy link
Copy Markdown

@serhiy-storchaka, thanks for your PR! By analyzing the history of the files in this pull request, we identified @birkenfeld, @tiran and @ezio-melotti to be potential reviewers.

Copy link
Copy Markdown
Member

@berkerpeksag berkerpeksag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I'd probably use an f-string in the second example, but that would make backporting it to 2.7 and 3.5 harder.

@serhiy-storchaka
Copy link
Copy Markdown
Member Author

It is hard to use f-strings with regular expressions since { and } are special in both microlanguages.

@serhiy-storchaka serhiy-storchaka merged commit 8fc7bc2 into python:master Apr 13, 2017
@serhiy-storchaka serhiy-storchaka deleted the re-escape-examples branch April 13, 2017 16:17
serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this pull request Apr 13, 2017
And fix the parameter name.

(cherry picked from commit 8fc7bc2)
serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this pull request Apr 13, 2017
And fix the parameter name.

(cherry picked from commit 8fc7bc2)
serhiy-storchaka added a commit to serhiy-storchaka/cpython that referenced this pull request Apr 13, 2017
And fix the parameter name.

(cherry picked from commit 8fc7bc2)
serhiy-storchaka added a commit that referenced this pull request Apr 13, 2017
And fix the parameter name.

(cherry picked from commit 8fc7bc2)
serhiy-storchaka added a commit that referenced this pull request Apr 13, 2017
And fix the parameter name.

(cherry picked from commit 8fc7bc2)
serhiy-storchaka added a commit that referenced this pull request Apr 13, 2017
And fix the parameter name.

(cherry picked from commit 8fc7bc2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants