swiftlang/swift

Fill out unimplemented expression cases in the ASTPrinter

Open

#61 601 ouverte le 17 oct. 2022

Voir sur GitHub
 (20 commentaires) (5 réactions) (0 assignés)Swift (10 719 forks)batch import
ASTPrintercompilerexpressionsgood first issue

Métriques du dépôt

Stars
 (69 989 stars)
Métriques de merge PR
 (Merge moyen 8j 17h) (510 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Étudiez l'implémentation existante d'ASTPrinter, en particulier l'imprimante d'Expr dans le compilateur Swift. Regardez comment d'autres types d'expression sont imprimés (par exemple, dans #40691 et #64667). Pour les sous classes d'ImplicitConversionExpr, comprenez le modèle de macro. Pour InterpolatedStringLiteralExpr, recherchez comment TapExpr est utilisé. Ensuite, implémentez les cas manquants et ajoutez des tests.
Stack technique
swiftcpp
Domaine
toolingdeveloper experience
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Plus d'une semaine
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
SwiftC++GitCompiler Development
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50