A brace inside an f-string interpolation comment silently changes the resulting value
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 76/100
Hướng nghiên cứu
Start in src/lexer.c at the interpolation boundary scanner around lines 399 and 422, then read the comment and f-string rules in docs/SPEC.md. Reproduce the issue and compare it with the existing #334/#344 quoted-character cases. Add regression coverage for braces in comments, preserving nested braces and quoted # or braces, and verify the evaluated output is exactly 3\n.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
A closing brace inside an f-string interpolation's comment terminates interpolation before the comment ends. The program succeeds but treats the remaining expression as literal text, changing its value.
Reproduce
Save and run this exact two-line program:
print of f"{1 # }
+ 2}"
Actual: exit 0, empty stderr, stdout shown as an escaped string:
"1\n + 2}\n"
Expected: 3\n. The } after # belongs to the comment; the actual expression is 1 + 2.
Paired control: change only the comment text from } to comment:
print of f"{1 # comment
+ 2}"
This exits 0 and prints 3\n. The ordinary parenthesized-expression control also handles the same brace-containing comment correctly:
print of (1 # }
+ 2)
It exits 0 and prints 3\n. This is not a request to introduce multiline expressions: the paired f-string control already accepts and evaluates one.
Cause and contract
The interpolation boundary scanner extracts the expression before recursively tokenizing it. It skips quoted string literals but has no comment state, and counts every exposed {/}. A } in the comment ends extraction; the following newline and + 2} become f-string literal text. The inner tokenizer recognizes the comment only after the boundary was chosen incorrectly.
The lexical contract says comments extend from # to end of line, and f-strings interpolate expressions. Boundary recognition must honor those lexical states.
Regression acceptance
- The reproducer prints exactly
3\n, matching both controls. - Test opening and closing braces within comments, including multiple brace characters.
- Retain genuine nested-expression brace handling and quoted
#/brace characters;#inside a string must not start a comment. - Assert exact output after evaluation, not parse success alone: the current defective program exits successfully.
The existing #334/#344 cases cover braces inside quoted strings, not braces inside comments. No matching open or closed report was found.
Verification
Confirmed on a fresh default release build of b91768e23c5a874a64e76e4af9ab291e6aa49983, with inherited EIGS_* variables cleared. Each reproducer and paired control was executed; no runtime or test source was changed. This is a current correctness defect, not a proposed v1 language restriction.
- Ngôn ngữ chính
- C
- Star
- 3
- Fork
- 7
- Merge trung bình
- 4 giờ 15 phút
- Pull request đã merge (30 ngày)
- 106
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của InauguralSystems/EigenScript
-
area:lint-tooling bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
InauguralSystems/EigenScript#1340 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:stdlib found-by:code-review kind:silent-wrong
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
InauguralSystems/EigenScript#1338 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:lint-tooling found-by:critic kind:docs-drift
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
InauguralSystems/EigenScript#1335 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:ci found-by:critic kind:gate-defect
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
InauguralSystems/EigenScript#1311 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enrolment: decide test_gc_runner_controls.py (exempt vs enrol) and whether floors need a ratchetĐang mởarea:gates found-by:critic kind:decision
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
InauguralSystems/EigenScript#1280 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của InauguralSystems/EigenScript
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
microsoft/ebpf-for-windows#5604 ·
Maintainer thường phản hồi trong vòng 3 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Update OPENEXR_IMATH_TAGĐang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 84/100
AcademySoftwareFoundation/openexr#2683 ·
Maintainer thường phản hồi trong vòng 1 ngày