Skip to content

(🐞) Column marker in error message off when line contains non ASCII character #102310

Description

@KotlinIsland
b"Ā"
👉 python test.py
  File "/test.py", line 1
    b"Ā"
         ^
SyntaxError: bytes can only contain ASCII literal characters

Here the caret is pointing to blackspace after the code

So each non-ASCII character is adding more incorrect offset:

👉 $c:temp = "b'ĀĀĀĀĀĀĀĀ'"
👉 py temp
  File "C:\temp", line 1
    b'ĀĀĀĀĀĀĀĀ'
                       ^

Expected

👉 python test.py
  File "/test.py", line 1
    b"Ā"
      ^
SyntaxError: bytes can only contain ASCII literal characters

Linked PRs

Activity

  1. arhadthedev commented on Feb 28, 2023

    @arhadthedev
    Member

    Can confirm on Python 3.12.0a5+ (heads/main:bcadcde712, Feb 26 2023, 12:54:46) [MSC v.1929 64 bit (AMD64)] on win32.

    Note: Ā here is UTF8-encoded as C4 80 (a single U+0100 LATIN CAPITAL LETTER A WITH MACRON). So incorrect row advancing on combined code points is totally ruled out until text normalization happens inside the parser.

  2. changed the title [-](🐞) Column marker in error message off for bytes with non ASCII character[/-] [+](🐞) Column marker in error message off when line contains non ASCII character[/+] on Feb 28, 2023
  3. arhadthedev commented on Feb 28, 2023

    @arhadthedev
    Member

    @benjaminp, @pablogsal (as parser experts)

    update: @gvanrossum, @lysnikolaou (as new parser experts)

  4. terryjreedy commented on Feb 28, 2023

    @terryjreedy
    Member

    In IDLE, Python 3.12.0a5+ (heads/main:0f89acf6cc, Feb 27 2023, 21:33:28) says "incomplete input" for this and "\N{a}", reported in #102312, and marks the closing quote, but this is effect of how codeop compiles.
    In REPL, I see the above and the #102312 message. I closed #102312 as duplicate.

  5. terryjreedy commented on Feb 28, 2023

    @terryjreedy
    Member

    Verified bug in installed 3.10.10, 3.11.2, and 3.12.a5.

  6. sobolevn commented on Feb 28, 2023

    @sobolevn
    Member

    What do you think about this?

    » ./python.exe ex.py
      File "/Users/sobolev/Desktop/cpython/ex.py", line 1
        x = b"Ā"
            ^^^^^
    SyntaxError: bytes can only contain ASCII literal characters
    

    This can be achieved with:
    Снимок экрана 2023-02-28 в 14 02 29

    Or do we need to highlight the symbol exactly?
    If my solution is fine, I can add a test case and send a PR.

  7. arhadthedev commented on Apr 21, 2023

    @arhadthedev
    Member

    If my solution is fine, I can add a test case and send a PR.

    @sobolevn In absence of other feedback, your solution looks better than nothing. So it's worth to create a PR and get it merged before 3.12b1.

  8. sobolevn commented on Apr 21, 2023

    @sobolevn
    Member

    Will do :)

  9. added a commit that references this issue on Apr 21, 2023
  10. added a commit that references this issue on Apr 23, 2023
  11. added a commit that references this issue on Apr 23, 2023
  12. added a commit that references this issue on Apr 23, 2023
  13. lysnikolaou commented on Apr 25, 2023

    @lysnikolaou
    Member

    Can this be closed?

    Edit: Actually, I'm closing this. Re-open in case there's more to it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    3.10 (EOL)end of life3.11only security fixes3.12only security fixesinterpreter-core(Objects, Python, Grammar, and Parser dirs)type-bugAn unexpected behavior, bug, or error

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions