swiftlang/swift

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

Open

#47.634 geöffnet am 31. Mai 2017

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Swift (10.719 Forks)batch import
bugcompilerdiagnostics qualitygood first issue

Repository-Metriken

Stars
 (69.989 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 7T 6h) (556 gemergte PRs in 30 T)

Beschreibung
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'

Contributor Guide

Research-Richtung
Das Problem besteht darin, einen Fixit für die redundante Konformitätsdiagnose hinzuzufügen. Suchen Sie nach der Ausgabe der Diagnose im Swift Compiler Quellcode, wahrscheinlich in lib/Sema oder lib/AST. Die Diagnosemeldung betrifft redundante Konformität, und der Fixit sollte vorschlagen, die doppelte Konformitätsdeklaration zu entfernen. Untersuchen Sie vorhandene Fixits im Compiler (z. B. in LLVMs DiagnosticConsumers), um die API zum Hinzufügen von Fixits zu verstehen. Das Problem ist mit StarterBug gekennzeichnet und betrifft eine lokalisierte Änderung.
Tech Stack
swift
Domain
tooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Swift languageSwift compiler basicsDiagnostic fixit API
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40