Chage FFI::cdef() return FFI\CDef object
Nadie ha tomado este issue todavía.
- #8585 de @chopins — cerrado sin fusionar
- Lenguaje dominante
- C
- Estrellas
- 40.4k
- Forks
- 8.2k
- Merge medio
- 2 d 15 h
- PR fusionados (30 d)
- 103
Descripción
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
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comience por el punto de entrada FFI::cdef(), zend_ffi_handlers.get_method y la discusión del issue vinculado #8585. Determine el impacto en la compatibilidad de devolver FFI\CDef y pasarlo a métodos no estáticos; el trabajo requiere un diseño de API acordado, no una propuesta sin resolver.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- c, php
- Área
- backend
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100