dotnet/roslyn

NullReferenceException when calling SyntaxFactory.ParseArgumentList

Aperta

#58.668 aperta il 6 gen 2022

Vedi su GitHubGuida contributor
 (0 commenti) (1 reazione) (1 assegnatario)C# (4257 fork)batch import
Area-CompilersBugLanguage-VBhelp wanted

Metriche repository

Star
 (20.414 stelle)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

Version Used: Nuget Microsoft.CodeAnalysis.VisualBasic Versions=4.0.1, 3.11.0

What are we trying to do? Parse an argument list (and recieving an unexpected exception (presumably because the input is invalid).)

  1. Reference Microsoft.CodeAnalysis.VisualBasic package in a project
  2. Call SyntaxFactory.ParseArgumentList("42")

Expected Behavior: Correct parsing of the argument. At least a more understandable exception like ArgumentException or similar, but not a direct bubbling of an internal exception.

Actual Behavior: NullReferenceExceptionwhen calling a static method.

The current implementation is quite misleading. As soon as one changes the passed string to contain parenthesis (turn "42" into "(42)") the parsing works. Which could be acceptable behavior. But simply passing on the internal exception is not. It should be something else.

Guida contributor

Direzione di ricerca
Analizza l'implementazione interna di SyntaxFactory.ParseArgumentList per individuare dove si verifica il riferimento nullo e aggiungi una corretta validazione degli argomenti.
Tech stack
csharp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#Roslyn
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70