rust-lang/rust-clippy

useless op on lhs of integer division or modulo

Open

#1.324 aberto em 7 de nov. de 2016

Ver no GitHub
 (2 comments) (0 reactions) (1 assignee)Rust (1.391 forks)batch import
A-lintL-correctnessL-unnecessaryT-middlegood first issue

Métricas do repositório

Stars
 (10.406 stars)
Métricas de merge de PR
 (Mesclagem média 19d 22h) (113 fundiu PRs em 30d)

Description

(x + n + y) % n or (x + n + y) / n can simply be replaced by (x + y) % n or (x + y) / n + 1

this isn't true for the following computations on unsigned integers:

(x + n - y) % n or (x + n - y) / n

as they might go negative due to the subtraction. For explicitly evading INT_MAX issues, this lint will cause false positives, but that could be evaded by not linting if INT_MAX is mentioned somewhere near the operation.

most likely, whenever this lint triggers, it is a bug in the equation, so the suggestion doesn't fix the underlying issue, but it will raise awareness of it.

Guia do colaborador

Direção de pesquisa
Investigue a logica do lint para divisao/modulo de inteiros com sinal e identifique os casos especificos em que o lint dispara incorretamente ou nao detecta. Verifique os casos extremos envolvendo INT MAX e subtracao. Considere como evitar falsos positivos quando INT MAX for mencionado proximo.
Pilha de tecnologia
rust
Domain
devtools
Tipo Issue
Bug
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-3 horas
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
RustGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75