swiftlang/swift

[SR-6264] Unify duplicated code in GlobalPropertyOpt::isVisibleExternally() and FunctionLivenessComputation::isVisibleExternally()

Ouverte

#48 814 ouverte le 1 nov. 2017

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)Swift (10 719 forks)batch import
IRGencompilergood first issueimprovement

Métriques du dépôt

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

Description
Previous ID SR-6264
Radar None
Original Reporter @gparker42
Type Improvement
Votes 0
Component/s Compiler
Labels Improvement, IRGen, StarterBug
Assignee None
Priority Medium

md5: 7fe2948a0ff34a9d22f375125e2fe476

Issue Description:

Functions GlobalPropertyOpt::isVisibleExternally() and FunctionLivenessComputation::isVisibleExternally() have duplicated code that checks whether a declaration may be externally used based on its access level. This duplication should be fixed.

Possible solution: add a function to class ValueDecl to do the work, and call it from both places.

Guide contributeur

Direction de recherche
Examinez les deux fonctions GlobalPropertyOpt::isVisibleExternally() et FunctionLivenessComputation::isVisibleExternally() dans le code source du compilateur Swift. Elles se trouvent probablement dans des fichiers liés à IRGen et SILOptimizer. Identifiez la logique dupliquée et créez une nouvelle méthode dans la classe ValueDecl qui encapsule la vérification du niveau d'accès. Remplacez ensuite le code dupliqué dans les deux fonctions par des appels à cette nouvelle méthode. Assurez vous que les tests continuent de passer.
Stack technique
cpp
Domaine
backend
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.
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++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80