PythonCharmers/python-future

Type not in (int, long) converted to not in (int, int)

Open

#265 ouverte le 19 janv. 2017

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Python (327 forks)batch import
0.19bughelp wantedtype-conversion

Métriques du dépôt

Stars
 (1 172 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Currently,

if type(time) not in (int, long): is converted to: if type(time) not in (int, int) (version 0.15.2)

This does not seem correct.

Following http://python3porting.com/differences.html I have instead changed this to:

if sys.version_info < (3,):
    integer_types = (int, long)
else:
    integer_types = (int,)

and then: if type(time) not in integer_types:

It would be excellent if this was built into futurize.

Guide contributeur

Direction de recherche
Examinez le code source du script `futurize` pour trouver où la conversion des types dans les expressions `not in` se produit. Le problème est que `long` est incorrectement remplacé par `int` au lieu d'être traité correctement. Cherchez un mappage de types ou une règle de transformation qui gère les types intégrés.
Stack technique
python
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Python
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60