python / python/cpython

tokenize: `e.offset` on `IndentationError` is not set correctly

オープン
#153,837 コメント 3 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

3.14 3.15 3.16 interpreter-core topic-parser type-bug
主要言語
Python
スター
77.2k
フォーク
35.9k
PR マージ指標
PR 指標を取得中

説明

Bug report

Bug description:
import io, tokenize

input_s = """
def foo():
    x = 1
   x = "some_very_long_expression"
"""
try:
    list(tokenize.generate_tokens(io.StringIO(input_s).readline))
except IndentationError as e:
    print(f"{e.offset = }")

The line x = "some_very_long_expression" is indented with 3 spaces instead of 4, so we get an error.
We would expect the offset to be around 3, since that is where the issue is.
On python 3.12+, the offset is instead 35, equal to the length of the line + 1.

This appears to be caused by the refactor of tokenize done in 3.12 for PEP 701
Potentially related to https://github.com/python/cpython/issues/84515
Funnily enough, the exact same thing seems to have happened around 3.8 https://github.com/python/cpython/issues/75676

Tested versions:

Version e.offset
3.11.15 3
3.12.13 35
3.13.13 35
3.14.4 35
3.15.0b1 35

Testing command:
nix-shell -p python311 --run "python --version && python tmp.py"
(and replace 311 by 312, 313, etc)

CPython versions tested on:

3.15

Operating systems tested on:

Linux

Linked PRs
  • gh-154281

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

tokenize.generate_tokens から始め、提示された IndentationError の例を記載された Python バージョン全体で再現します。tokenizer が IndentationError をどのように構築するかを追跡し、offset を Python 3.11 の動作と比較します。誤ってインデントされた行が 3 付近の offset を報告し、その例に対するリグレッションテストのカバレッジがあることを完了条件とします。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
compilers
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。