python / python/cpython

It is awkward to check whether an obj is of a particular heap type (defined in an extension module)

Abierto
#133,161 3 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

topic-C-API type-feature
Lenguaje dominante
Python
Estrellas
77.2k
Forks
35.9k
Métricas de merge de PR
Métricas de PR pendientes

Descripción

I wanted to check whether an object is of type array and ended up doing this:

    PyObject *exc = PyErr_GetRaisedException();
    PyObject *module = PyType_GetModuleByDef(Py_TYPE(lhs), &arraymodule);
    if (module == NULL) {
        if (!PyErr_Occurred() || PyErr_ExceptionMatches(PyExc_TypeError)) {
            /* lhs is not an array instance - ignore the TypeError (if any) */
            PyErr_SetRaisedException(exc);
            return 0;
        }
        else {
            _PyErr_ChainExceptions1(exc);
            return -1;
        } 
    }   
    PyErr_SetRaisedException(exc);
    return array_Check(lhs, get_array_state(module));

CC @ericsnowcurrently @encukou

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza con las llamadas de la API de C mostradas en el issue, especialmente PyType_GetModuleByDef y el ejemplo de array_Check, para comprender la ruta actual de comprobación de tipos. Revisa las convenciones relacionadas de la API de C de CPython y determina una forma concreta y ampliamente aplicable de comprobar los tipos heap; se considera terminado cuando exista una dirección de API acordada con la documentación y las pruebas adecuadas.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
c, python
Área
api
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.