Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

f-strings do not show warnings about invalid escapes #105800

Closed
pablogsal opened this issue Jun 14, 2023 · 1 comment
Closed

f-strings do not show warnings about invalid escapes #105800

pablogsal opened this issue Jun 14, 2023 · 1 comment
Labels
3.12 bugs and security fixes 3.13 bugs and security fixes

Comments

@pablogsal
Copy link
Member

pablogsal commented Jun 14, 2023 •
edited by bedevere-bot
Loading

In 3.12 and 3.13, f-strings are not warning about invalid escapes that get warnings in real strings. This seems like a bug.

>>> f'\?'
'\\?'
>>> len(f'\?')
2
>>> '\?'
<stdin>:1: SyntaxWarning: invalid escape sequence '\?'
'\\?'

_Originally posted by @terryjreedy in https://github.com/python/cpython/issues/105784#issuecomment-1591828755_
            

<!-- gh-linked-prs -->
### Linked PRs
* gh-105801
* gh-105806
<!-- /gh-linked-prs -->
@bedevere-bot bedevere-bot mentioned this issue Jun 14, 2023
Merged
pablogsal added a commit to pablogsal/cpython that referenced this issue Jun 14, 2023
@pablogsal
pythongh-105800: Issue SyntaxWarning in f-strings for invalid escape …
8a4b38e 
…sequences
@AlexWaygood AlexWaygood added 3.12 bugs and security fixes 3.13 bugs and security fixes labels Jun 14, 2023
@bedevere-bot bedevere-bot mentioned this issue Jun 15, 2023
Merged
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jun 15, 2023
@pablogsal @miss-islington
pythongh-105800: Issue SyntaxWarning in f-strings for invalid escape …
e3a46f8 
…sequences (pythonGH-105801)

(cherry picked from commit 12b6d84)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
pablogsal added a commit that referenced this issue Jun 15, 2023
@pablogsal
gh-105800: Issue SyntaxWarning in f-strings for invalid escape sequen…
12b6d84 
…ces (#105801)
@pablogsal
Copy link
Member Author

CC: @terryjreedy

@pablogsal pablogsal changed the title f-strings are not warning about invalid escapes f-strings do not warning about invalid escapes Jun 15, 2023
@pablogsal pablogsal changed the title f-strings do not warning about invalid escapes f-strings do not show warnings about invalid escapes Jun 15, 2023
pablogsal added a commit that referenced this issue Jun 15, 2023
@miss-islington @pablogsal
[3.12] gh-105800: Issue SyntaxWarning in f-strings for invalid escape…
638c2ba 
… sequences (GH-105801) (#105806)

Co-authored-by: Pablo Galindo Salgado <[email protected]>
@mattip mattip mentioned this issue Jun 15, 2023
Closed
carljm added a commit to carljm/cpython that referenced this issue Jun 15, 2023
@carljm
Merge branch 'main' into modulelocals
166aca9 
* main: (57 commits)
  pythongh-105831: Fix NEWS blurb from pythongh-105828 (python#105833)
  pythongh-105820: Fix tok_mode expression buffer in file & readline tokenizer (python#105828)
  pythongh-105751, test_ctypes: Remove disabled tests (python#105826)
  pythongh-105821: Use a raw f-string in test_httpservers.py (python#105822)
  pythongh-105751: Remove platform usage in test_ctypes (python#105819)
  pythongh-105751: Reenable disable test_ctypes tests (python#105818)
  pythongh-105751: Remove dead code in test_ctypes (python#105817)
  More reorganisation of the typing docs (python#105787)
  Improve docs for `typing.dataclass_transform` (python#105792)
  pythonGH-89812: Churn `pathlib.Path` test methods (python#105807)
  pythongh-105800: Issue SyntaxWarning in f-strings for invalid escape sequences (python#105801)
  pythongh-105751: Cleanup test_ctypes imports (python#105803)
  pythongh-105481: add HAS_JUMP flag to opcode metadata (python#105791)
  pythongh-105751: test_ctypes avoids the operator module (pythonGH-105797)
  pythongh-105751: test_ctypes: Remove @need_symbol decorator (pythonGH-105798)
  pythongh-104909: Implement conditional stack effects for macros (python#105748)
  pythongh-75905: Remove test_xmlrpc_net: skipped since 2017 (python#105796)
  pythongh-105481: Fix types and a bug for pseudos (python#105788)
  Update DSL docs for cases generator (python#105753)
  pythonGH-77273: Better bytecodes for f-strings (pythonGH-6132)
  ...
@pdiakumis pdiakumis mentioned this issue Jun 13, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3.12 bugs and security fixes 3.13 bugs and security fixes
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pablogsal @AlexWaygood