php / php/php-src

Chage FFI::cdef() return FFI\CDef object

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

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

  • #8585 của @chopins — đã đóng, không merge
Extension: ffi Feature Status: Needs Triage
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

Now, FFI::cdef() return a FFI object, then through FFI object call C function.
The above reasons cause some method support call by statically and non-statically, and these method will need exclude in zend_ffi_handlers.get_method. if C DLL has same name function will not be call, or add new FFI method must insert to execlude list.
So suggest FFI::cdef() return object of other class, For example use FFI\CDef instead of FFI object.
FFI\CDef is not construrct and extend in PHP code, Similarly FFI\CData.
Through FFI\CDef of zend_ffi_handlers.get_method will call C function. and FFI non-statically method add one paramter receive FFI\CDef object. this is cause in compatibility.

see: #8585

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 entry point FFI::cdef(), zend_ffi_handlers.get_method và phần thảo luận trong issue được liên kết #8585. Xác định tác động về khả năng tương thích của việc trả về FFI\CDef và truyền nó cho các phương thức không static; yêu cầu để hoàn tất là một thiết kế API đã được thống nhất, không phải một đề xuất chưa được giải quyết.

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, php
Lĩnh vực
backend
Loại issue
Tính năng
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Cần làm rõ
Mức phù hợp với người mới
25/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.