dotnet/roslyn

'Use pattern-matching' should be missing on writes from local function captures

Offen

#29.512 geöffnet am 24.08.2018

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C# (4.257 Forks)batch import
Area-IDEBughelp wanted

Repository-Metriken

Stars
 (20.414 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (256 gemergte PRs in 30 T)

Beschreibung

From https://github.com/dotnet/roslyn/pull/26097#discussion_r210728805

We should look for writes that are logically within the span, but syntactically are not.

var s = field as string;
WriteField();
if (s != null) { ... }

void WriteField () => field = null;

/cc @sharwell

Contributor Guide

Research-Richtung
Untersuchen Sie die Behandlung von Schreibvorgängen aus lokalen Funktionserfassungen durch den Pattern Matching Analyzer. Verfolgen Sie die Datenflussanalyse für die Diagnose 'Use pattern matching' im Kontext lokaler Funktionen.
Tech Stack
csharp
Domain
compilers
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.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C#Roslyn internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30