dotnet/roslyn

Should protected abstract methods in SemanticModel be private protected instead?

Ouverte

#29 235 ouverte le 12 août 2018

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-CompilersBugConcept-APIhelp 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

I noticed that protected abstract methods on SemanticModel like GetDeclaredSymbolCore are documented in Roslyn's API reference. Since it's impossible to inherit from SemanticModel for users of the API, I think such methods should not be included in the API reference.

And I think the most reasonable way to do that is to change such methods to private protected. Do you agree?

It's possible there are more types like that in the codebase, but SemanticModel is the one where I noticed this.

Guide contributeur

Direction de recherche
Identifiez toutes les méthodes abstraites protégées dans SemanticModel et vérifiez que les changer en private protected ne casse pas les accès des sous classes internes.
Stack technique
csharp
Domaine
api
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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#Understanding of access modifiers
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85