dotnet/roslyn

Fix VB Parsing of Format Specifiers

Ouverte

#58 043 ouverte le 30 nov. 2021

Voir sur GitHubGuide contributeur
 (16 commentaires) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-CompilersBugLanguage-VBhelp wanted

Métriques du dépôt

Stars
 (20 414 étoiles)
Métriques de merge PR
 (Merge moyen 6j 17h) (256 PRs mergées en 30 j)

Description

VB has the same parsing "bug" as detailed in https://github.com/dotnet/roslyn/issues/57750, but unlike C# it does not support interpolated string handlers, so there are no runtime impacts to the parsing issue. We should update the VB parser to be in sync with the C# parser and the runtime here, but it's not high priority so the team is not going to focus on it. We'd accept a community PR to implement this fix.

Guide contributeur

Direction de recherche
Examinez la correction C# dans l'issue #57750 pour comprendre le changement d'analyse. Localisez la logique correspondante de l'analyseur VB dans la base de code Roslyn, probablement dans src/Compilers/VisualBasic/Portable/Parsing. Implémentez la même mise à jour pour VB, en veillant à ce qu'il n'y ait aucun impact à l'exécution car VB ne dispose pas de gestionnaires de chaînes interpolées. Testez le changement avec les tests d'analyse VB existants.
Stack technique
csharp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75