eclipse-jdt/eclipse.jdt.core

RFE: issue (optional) warning when '==' is used on wrapper types

Open

#2 176 ouverte le 18 mars 2024

Voir sur GitHub
 (5 commentaires) (2 réactions) (0 assignés)Java (183 forks)auto 404
enhancementhelp wanted

Métriques du dépôt

Stars
 (233 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

I would like that Eclipse (similar to IDEA) would issue an optional warning when one uses the following construct:

Long x1 = 12345L;
Long x2 = 67890L;
if(x1 == x2)
    System.out.println("Equal");
else
    System.out.println("Not equal");

Because (as apparently we don't all know) Long is a wrapper class and using == only works for Long values between -127 and 127 due to caching (similarly for the other wrapper types).

This would at least uncover one such instance in our code base, of which I suspect to find a greater number alas.

Guide contributeur

Direction de recherche
Examinez l'infrastructure d'avertissements existante dans Eclipse JDT Core (par exemple, pour 'comparaison nulle' ou 'cast inutile'). Identifiez où ajouter un nouvel avertissement optionnel pour '==' sur les types wrapper et implémentez le dans le compilateur/visiteur AST. Testez avec différents types wrapper et valeurs mises en cache.
Stack technique
java
Domaine
developer experiencetooling
Type d'issue
Fonctionnalité
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-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaEclipse JDT Core internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60