Improve type safety of internal classes
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
Hướng nghiên cứu
Trước tiên, hãy lập danh mục các lớp nội bộ và các phép ép kiểu từ lớp cơ sở sang lớp dẫn xuất bị ảnh hưởng bởi issue, sau đó truy vết nơi các đối tượng đó đi vào công cụ instrumentation. So sánh các mẫu kế thừa và giao diện hiện tại với cách tiếp cận kế thừa ảo và QI trên lớp dẫn xuất được đề xuất; được xem là hoàn tất khi các phép ép kiểu không an toàn liên quan đã được xử lý và các đường đi bị ảnh hưởng vẫn tương thích.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
There are a number of places where we cast from a base class to a derived class in our internal code, which could cause problems if a caller passes us an instance of an interface that we did not create internally. I suggest we:
-
Update our code to use virtual inheritance (instead of multiple inheritance). This will help us catch locations where we cast from a base class.
-
QI for the derived class. This will give us type safety.
- Ngôn ngữ chính
- C++
- Star
- 104
- Fork
- 53
- Merge trung bình
- 3 giờ 34 phút
- Pull request đã merge (30 ngày)
- 1
Chuẩn bị môi trường
- Không có Dockerfile hay 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 microsoft/CLRInstrumentationEngine
-
documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
microsoft/CLRInstrumentationEngine#169 · 2 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
microsoft/CLRInstrumentationEngine#526 · 1 reaction ·
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
Tất cả issue của microsoft/CLRInstrumentationEngine
Issue tương tự
-
torch_ops/logprob.cu does not compile with the serving container's nvcc (13.3.73); check_torch_ops.py cannot run as shippedCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 Dưới một giờ Mức phù hợp với người mới 72/100
ashhart/TensorFold#535 ·
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 66/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 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
agent:Windows bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/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 72/100
Maintainer thường phản hồi trong vòng 1 ngày