swiftlang/swift

Fill out unimplemented expression cases in the ASTPrinter

Open

#61.601 geöffnet am 17. Okt. 2022

Auf GitHub ansehen
 (20 Kommentare) (5 Reaktionen) (0 zugewiesene Personen)Swift (10.719 Forks)batch import
ASTPrintercompilerexpressionsgood first issue

Repository-Metriken

Stars
 (69.989 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T 17h) (510 gemergte PRs in 30 T)

Beschreibung

In #40691 we added initial support for printing Expr nodes in the ASTPrinter, and #64667 filled out most of the rest. However there are still some expressions left unimplemented, including:

  • All the ImplicitConversionExpr subclasses, we should stamp out their definitions as printing their sub-expression, using a macro
  • InterpolatedStringLiteralExpr (this is a tricky case since we'd need to reconstruct the string literal from the TapExpr, or otherwise preserve its original form)

These are partially implemented, with outstanding FIXMEs:

  • ClosureExpr
  • CaptureListExpr
  • KeyPathExpr

Some extras tests would be good for:

  • SubscriptExpr (would be great to have a test exercising some other call patterns, e.g multiple arguments, argument labels)
  • TupleElementExpr
  • ObjCSelectorExpr
  • DynamicSubscriptExpr
  • EnumIsCaseExpr
  • MakeTemporarilyEscapableExpr
  • EditorPlaceholderExpr
  • DotSyntaxBaseIgnoredExpr

Contributor Guide

Research-Richtung
Untersuche die vorhandene ASTPrinter Implementierung, insbesondere den Expr Drucker im Swift Compiler. Sieh dir an, wie andere Ausdruckstypen gedruckt werden (z. B. in #40691 und #64667). Für ImplicitConversionExpr Unterklassen verstehe das Makromuster. Für InterpolatedStringLiteralExpr recherchiere, wie TapExpr verwendet wird. Implementiere dann die fehlenden Fälle und füge Tests hinzu.
Tech Stack
swiftcpp
Domain
toolingdeveloper experience
Issue Type
Funktion
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.
Über 1 Woche
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.
Klar
Voraussetzungen
SwiftC++GitCompiler Development
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50