ruby / ruby/fiddle

Unreachable code path in Fiddle::ValueUtil#wrap_arg

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

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

Ngôn ngữ chính
Ruby
Star
195
Fork
53
Merge trung bình
23 giờ 57 phút
Pull request đã merge (30 ngày)
1

Mô tả

In Fiddle::ValueUtil#wrap_arg there is a long case-when for arg with an unreachable code path.
When arg and ty are of Array type, and ty[0] is TYPE_CHAR, there is another condition, for arg to be String at Line 101.
Even if this condition is fixed, val is undefined at Line 102 for this method.
https://github.com/ruby/fiddle/blob/04238cefed14a433f93909bd93dbe1d0a4efc47e/lib/fiddle/value.rb#L95-L106

Should Lines 100-103 be removed?

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

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

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.

Hướng nghiên cứu

Bắt đầu trong lib/fiddle/value.rb tại Fiddle::ValueUtil#wrap_arg, đặc biệt là các dòng 95-106 được liên kết trong issue. Theo dõi các nhánh Array và TYPE_CHAR để xác nhận điều kiện không thể đạt tới và val chưa được định nghĩa, sau đó xác định liệu các dòng 100-103 có nên được xóa hay không; hoàn tất khi đường dẫn chết được xử lý mà không thay đổi việc wrapping các đối số hợp lệ.

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

Đánh giá

Công nghệ
ruby
Lĩnh vực
backend
Loại issue
Tái cấu trúc
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
52/100

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.