eclipse-jdt/eclipse.jdt.core

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

Open

#2.176 aberto em 18 de mar. de 2024

Ver no GitHub
 (5 comments) (2 reactions) (0 assignees)Java (183 forks)auto 404
enhancementhelp wanted

Métricas do repositório

Stars
 (233 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Investigue a infraestrutura existente de avisos no Eclipse JDT Core (por exemplo, para 'comparação nula' ou 'conversão desnecessária'). Identifique onde adicionar um novo aviso opcional para '==' em tipos wrapper e implemente no compilador/visitante AST. Teste com vários tipos wrapper e valores em cache.
Pilha de tecnologia
java
Domain
developer experiencetooling
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaEclipse JDT Core internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60