scan_int_tokens line/col fields are untested
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ó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 78/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- c
- Lĩnh vực
- testing-qa
Hướng nghiên cứu
Bắt đầu tại src/builtins.c, ở các bộ đếm token_line và token_col bên trong scan_int_tokens, sau đó đọc khối SIT1-SIT12 trong tests/test_string_math.eigs để xem kiểu assertion đang dùng. Thêm assertion cho line và col đối với một token nằm sau một dòng comment bị bỏ qua và một token có thụt lề, theo cùng kiểu với các kiểm tra ST3/ST4/ST6/ST7 đã bị xóa. Hoàn thành khi test chạy đạt trên mã hiện tại và thất bại khi token_col bị dịch chuyển có chủ đích đi một đơn vị.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Found by the blind critic on #1677 round 4. This predates #1677 and is not caused by it.
scan_int_tokens computes each row's line (index 1) and col (index 2) with its own counter loop in src/builtins.c (token_line / token_col). That code is separate from the removed scan_tokens. No test asserts those two fields. tests/test_string_math.eigs SIT1-SIT12 check the text, is_int, the value and the offset slice, but not line or col. The ST1-ST8 line/col checks only ever exercised scan_tokens, and #1677 deleted them along with it.
Done when
tests/test_string_math.eigsasserts line and col for at least one token after a skipped comment line and one indented token, mirroring the old ST3/ST4/ST6/ST7.- A planted off-by-one in
token_colmakes the test fail.
- Ngôn ngữ chính
- C
- Star
- 3
- Fork
- 7
- Merge trung bình
- 8 giờ 27 phút
- Pull request đã merge (30 ngày)
- 245
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:gfx bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
InauguralSystems/EigenScript#1684 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:stdlib bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
InauguralSystems/EigenScript#1679 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:consumer
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
InauguralSystems/EigenScript#1658 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:stdlib kind:silent-wrong
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
InauguralSystems/EigenScript#1652 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:stdlib kind:silent-wrong
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
InauguralSystems/EigenScript#1642 ·
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 76/100
osam-cologne/archlinux-proaudio#909 ·
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 84/100
ARM-software/CMSIS-DSP#354 ·
-
good first issue help wanted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/100
Maintainer thường phản hồi trong vòng 1 ngày
-
documentation good first issue help wanted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
ai-quokka-wannabe/link#40 ·
-
[BUG] Sentinel sends CONFIG REWRITE to instances started without a config fileCó thể đã có người làm @jdymitarai đã nhận hôm nay. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 2 ngày