swiftlang/swift

[SR-5058] Redundant Conformance Diagnostic should have an associated fixit

Open

#47.634 aberto em 31 de mai. de 2017

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)Swift (10.719 forks)batch import
bugcompilerdiagnostics qualitygood first issue

Métricas do repositório

Stars
 (69.989 stars)
Métricas de merge de PR
 (Mesclagem média 7d 6h) (556 fundiu PRs em 30d)

Description
Previous ID SR-5058
Radar rdar://32431756
Original Reporter @CodaFi
Type Bug
Status In Progress
Resolution

Attachment: Download
Votes 1
Component/s Compiler
Labels Bug, DiagnosticsQoI, StarterBug
Assignee interfere (JIRA)
Priority Medium

md5: 5985872320450fd433971928ef7a5b14

Issue Description:

The redundant conformance diagnostic can point at the protocol that is redundant, but doesn’t actually offer to remove it.

protocol Foo {}
struct Bar {}

extension Bar: Foo {}
extension Bar: Foo {} // redundant conformance of 'Bar' to protocol 'Foo'

Guia do colaborador

Direção de pesquisa
A questão é adicionar uma sugestão de correção (fixit) para o diagnóstico de conformidade redundante. Procure a emissão do diagnóstico no código fonte do compilador Swift, provavelmente em lib/Sema ou lib/AST. A mensagem de diagnóstico é sobre conformidade redundante, e a sugestão de correção deve propor a remoção da declaração de conformidade duplicada. Investigue as sugestões de correção existentes no compilador (por exemplo, em DiagnosticConsumers do LLVM) para entender a API para adicionar sugestões de correção. A questão está rotulada como StarterBug e envolve uma alteração localizada.
Pilha de tecnologia
swift
Domain
tooling
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Swift languageSwift compiler basicsDiagnostic fixit API
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40