[issue40016] Clarify flag case in `re` module docstring

Change by Terry J. Reedy : -- assignee: docs@python -> terry.reedy resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue40016] Clarify flag case in `re` module docstring

miss-islington added the comment: New changeset 0dad7486e7d7bc2e0f1b0a4f44d9c28064762be5 by Miss Islington (bot) in branch '3.7': bpo-40016: re docstring: Clarify relationship of inline and argument flags (GH-19078)

[issue40016] Clarify flag case in `re` module docstring

miss-islington added the comment: New changeset 686d508c26fafb57dfe463c4f55b20013dad1441 by Miss Islington (bot) in branch '3.8': bpo-40016: re docstring: Clarify relationship of inline and argument flags (GH-19078)

[issue40016] Clarify flag case in `re` module docstring

Change by miss-islington : -- pull_requests: +18522 pull_request: https://github.com/python/cpython/pull/19162 ___ Python tracker ___

[issue40016] Clarify flag case in `re` module docstring

Terry J. Reedy added the comment: New changeset 89a2209ae6fc5f39868621799730e16f931eb497 by Ram Rachum in branch 'master': bpo-40016: re docstring: Clarify relationship of inline and argument flags (#19078) https://github.com/python/cpython/commit/89a2209ae6fc5f39868621799730e16f931eb497

[issue40016] Clarify flag case in `re` module docstring

Change by miss-islington : -- nosy: +miss-islington nosy_count: 5.0 -> 6.0 pull_requests: +18521 pull_request: https://github.com/python/cpython/pull/19161 ___ Python tracker

[issue40016] Clarify flag case in `re` module docstring

Terry J. Reedy added the comment: The docstring is currently 103 lines. I intentionally replaced 1 line with 1 line that I believe to be more informative and kept the expansion of the other line to 3 lines. -- ___ Python tracker

[issue40016] Clarify flag case in `re` module docstring

Terry J. Reedy added the comment: The docstring line in question is (?aiLmsux) Set the A, I, L, M, S, U, or X flag for the RE (see below). This is exceptional in that other syntaxes in the special characters list use lower case only for syntax variables (m, n, name, id/name, yes, no).

[issue40016] Clarify flag case in `re` module docstring

Change by Terry J. Reedy : -- title: Clarify flag case in `re` module -> Clarify flag case in `re` module docstring ___ Python tracker ___

[issue40016] Clarify flag case in `re` module

Terry J. Reedy added the comment: The root confusion is that re compilation has several variations with two sets of indicators, each with an unhelpful exception, and each combined and used in different ways. 1. Module constants with uppercase English words or word pairs, also abbreviated

[issue40016] Clarify flag case in `re` module

Serhiy Storchaka added the comment: I apologize if I was rude. It's only because of my bad English. There were many translation options for my words suggested by Google Translator and I obviously picked up the wrong one. Improving documentation is always a good thing. But I leave the final

[issue40016] Clarify flag case in `re` module

Ram Rachum added the comment: I updated my PR to match. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue40016] Clarify flag case in `re` module

Ram Rachum added the comment: I'm gonna look past the rudeness, and I'll just say that if I was tripped up by this, after 11 years of working with Python and the re module, then people in a beginner or intermediate level could be tripped up by this as well. Here's another, simpler

[issue40016] Clarify flag case in `re` module

Serhiy Storchaka added the comment: I do not see any issue except that you was careless when read the documentation. -- ___ Python tracker ___

[issue40016] Clarify flag case in `re` module

Ram Rachum added the comment: Oops, my mistake. Any other idea how to solve this discrepancy? -- ___ Python tracker ___ ___

[issue40016] Clarify flag case in `re` module

Serhiy Storchaka added the comment: They are. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue40016] Clarify flag case in `re` module

Ram Rachum added the comment: Well, these aren't the textbook case of a constant, since they're enums, and not defined in the global namespace. -- ___ Python tracker ___

[issue40016] Clarify flag case in `re` module

Serhiy Storchaka added the comment: It is very inconvenient to use single-letter lowercase names for constants. It contradicts PEP 8: https://www.python.org/dev/peps/pep-0008/#constants -- nosy: +serhiy.storchaka ___ Python tracker

[issue40016] Clarify flag case in `re` module

Ram Rachum added the comment: As you can see I left the old uppercase enums defined, to avoid breaking backward compatibility. We could make them trigger a DeprecationWarning. -- ___ Python tracker

[issue40016] Clarify flag case in `re` module

Change by Ram Rachum : -- keywords: +patch pull_requests: +18434 stage: -> patch review pull_request: https://github.com/python/cpython/pull/19078 ___ Python tracker ___

[issue40016] Clarify flag case in `re` module

New submission from Ram Rachum : Today I was tripped up by an inconsistency in the `re` docstring. I wanted to use DOTALL as a flag inside my regex, rather than as an argument to the `compile` function. Here are two lines from the docstring: (?aiLmsux) Set the A, I, L, M, S, U, or X flag