eclipse-jdt/eclipse.jdt.core

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

Open

#2.176 geöffnet am 18. März 2024

Auf GitHub ansehen
 (5 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)Java (183 Forks)auto 404
enhancementhelp wanted

Repository-Metriken

Stars
 (233 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die vorhandene Warninfrastruktur in Eclipse JDT Core (z. B. für 'Null Vergleich' oder 'unnötige Cast'). Identifizieren Sie, wo eine neue optionale Warnung für '==' auf Wrappertypen hinzugefügt werden kann, und implementieren Sie sie im Compiler/AST Besucher. Testen Sie mit verschiedenen Wrappertypen und zwischengespeicherten Werten.
Tech Stack
java
Domain
developer experiencetooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaEclipse JDT Core internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60