Fine-grained: Nested class causes extra triggers to fire

Đang mở
#4,614 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
35/100
Loại issue
Lỗi
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Đình trệ
Công nghệ
python
Lĩnh vực
compilers, devtools

Hướng nghiên cứu

Start in mypy/test/testfinegrained.py with testNestedClassLeakingFromMethod and reproduce the shown debug output. Trace how nested class names containing line numbers produce triggers for both old and new names. Done means the test no longer reports unnecessary triggers while preserving the incompatible-assignment error.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug priority-2-low topic-fine-grained-incremental

This test case trigger too many triggers due to nested classes:

[case testNestedClassLeakingFromMethod]
import a

def f(o: a.A) -> None:
    o.b.x = 1

[file a.py]
class A:
    def f(self) -> None:
        class B:
            x: int
        self.b = B()

    def g(self) -> None:
        class B: pass
        self.c = B()

[file a.py.2]
class A:
    # extra line
    def f(self) -> None:
        class B:
            x: str
        self.b = B()

    def g(self) -> None:
        class B: pass
        self.c = B()
[out]
==
main:4: error: Incompatible types in assignment (expression has type "int", variable has type "str")

Here's the debug output:

mypy/test/testfinegrained.py ==== update 'a' ====
--- update single 'a' ---
triggered: ['<a.A.b>', '<a.A.c>', '<a.B@3>', '<a.B@4>', '<a.B@8>', '<a.B@9>']
process __main__.f
triggered: []

Note that since the line number is included in the class names, moving the class to another line changes the name and causes triggers for both the previous and new name to fire.

Not sure what's the best way to deal with this. If nested classes that leak from methods are rare enough this might not be a major problem.

Ngôn ngữ chính
Python
Star
20.6k
Fork
3.3k
Merge trung bình
1 ngày 18 giờ
Pull request đã merge (30 ngày)
54

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của python/mypy

Tất cả issue của python/mypy

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.