python / python/cpython

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

Aberta
#133,161 3 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

topic-C-API type-feature
Linguagem predominante
Python
Estrelas
77.2k
Forks
35.9k
Métricas de merge de PRs
Métricas de PR pendentes

Descrição

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

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Comece pelas chamadas da C API mostradas na issue, especialmente PyType_GetModuleByDef e o exemplo de array_Check, para entender o caminho atual de verificação de tipos. Revise as convenções relacionadas da C API do CPython e determine uma forma concreta e amplamente aplicável de verificar heap types; considera-se concluído quando houver uma direção de API acordada, com documentação e testes apropriados.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
c, python
Domínio
api
Tipo de issue
Funcionalidade
Dificuldade
5/5
Tempo estimado
Mais de uma semana
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.