rust-lang/rust-clippy

transmute_ptr_to_ref suggestion doesn't with some outer expression types

Open

#1.754 aberto em 12 de mai. de 2017

Ver no GitHub
 (1 comment) (0 reactions) (1 assignee)Rust (1.391 forks)batch import
C-bugT-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

clippy suggests the following replacement

-                mem::transmute::<*mut c_void, &mut Box<FnMut(&Button)>>(data)(&button)
+                &mut *(data as *mut Box<FnMut(&Button)>)(&button)

but we should suggest parens around the &mut *...

bonus points if the parens only appear if necessary

Guia do colaborador

Direção de pesquisa
A questão descreve um lint do clippy `transmute ptr to ref` que sugere incorretamente substituir `mem::transmute::<*mut c void, &mut Box<FnMut(&Button)>>(data)(&button)` por `&mut *(data as *mut Box<FnMut(&Button)>)(&button)` sem parênteses. A correção envolve modificar o código de sugestão do lint no repositório do clippy (provavelmente em `clippy lints/src/transmute.rs`) para envolver `&mut *...` em parênteses quando ele aparecer em um contexto como uma chamada de função. A observação é que a sugestão é sintaticamente ambígua sem parênteses. Procure a implementação do lint `transmute ptr to ref`, examine como a sugestão é gerada (provavelmente usando `Applicability` e `span lint and sugg`), e adicione lógica para detectar se a expressão é usada como um chamado (por exemplo, dentro de uma chamada de função) e, em seguida, adicione parênteses ao redor do código sugerido. Teste a alteração com o exemplo fornecido e casos semelhantes.
Pilha de tecnologia
rust
Domain
developer experiencetooling
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.
Principalmente claro
Prerequisites
Rustclippy internal architecture
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75