dotnet/roslyn

Microsoft.CodeAnalysis.VisualBasic.SymbolDisplay.ToDisplayString() does not include ByRef for C# ParameterSymbols

Open

#14 683 ouverte le 22 oct. 2016

Voir sur GitHub
 (1 commentaire) (1 réaction) (0 assignés)C# (4 257 forks)batch import
Area-CompilersBughelp wanted

Métriques du dépôt

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

Description

Microsoft.CodeAnalysis.VisualBasic.SymbolDisplay.ToDisplayString() returns F(Integer) rather than F(ByRef Integer) for:

class C
{
    void F(ref int i) { }
}

See Microsoft.CodeAnalysis.VisualBasic.UnitTests.SymbolDisplayTests.RefReturn.

Guide contributeur

Direction de recherche
Étudiez la méthode VisualBasic.SymbolDisplay.ToDisplayString() pour comprendre pourquoi elle omet 'ByRef' pour les paramètres ref de C#. Le test 'Microsoft.CodeAnalysis.VisualBasic.UnitTests.SymbolDisplayTests.RefReturn' fournit un point de départ. Examinez la logique d'affichage des symboles pour la gestion des modificateurs de paramètres et assurez vous que ref est inclus lors de l'affichage des méthodes C# au format VB.
Stack technique
csharp
Domaine
tooling
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.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Visual Basic syntax
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70