php / php/php-src

Unify dns_*() function definitions for all platforms

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

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

Category: DNS Extension: standard Feature Status: Verified
Ngôn ngữ chính
C
Star
40.4k
Fork
8.1k
Merge trung bình
2 ngày 13 giờ
Pull request đã merge (30 ngày)
96

Mô tả

Description

This is a todo in the code

I'm putting it here so it gets on the radar for a todo.
If this is a wontfix then remove the todo from the code.

Changes to this code segment caused an issue #7748 with the output of gethostbyaddr() showing as a binary string in Windows.

/* TODO: Reimplement the gethostby* functions using the new winxp+ API, in dns_win32.c, then
 we can have a dns.c, dns_unix.c and dns_win32.c instead of a messy dns.c full of #ifdef
*/

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.

Hướng nghiên cứu

Bắt đầu với TODO trong ext/standard/dns.c quanh dòng 138, sau đó so sánh cách tổ chức của các tệp được tham chiếu dns.c, dns_unix.c và dns_win32.c. Xem issue #7748 về hồi quy gethostbyaddr() trên Windows. Được xem là hoàn tất khi entweder hợp nhất các định nghĩa dns_*() giữa các nền tảng mà không gây ra hồi quy đó, hoặc xóa TODO nếu dự án quyết định đây là một wontfix.

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

Đánh giá

Công nghệ
c
Lĩnh vực
networking
Loại issue
Tái cấu trúc
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
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
35/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.