swiftlang/swift

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

Aperta

#48.814 aperta il 1 nov 2017

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)Swift (10.719 fork)batch import
IRGencompilergood first issueimprovement

Metriche repository

Star
 (69.989 stelle)
Metriche merge PR
 (Merge medio 8g 17h) (510 PR mergiate in 30 g)

Descrizione
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.

Guida contributor

Direzione di ricerca
Esamina le due funzioni GlobalPropertyOpt::isVisibleExternally() e FunctionLivenessComputation::isVisibleExternally() nel codice sorgente del compilatore Swift. Probabilmente si trovano in file relativi a IRGen e SILOptimizer. Identifica la logica duplicata e crea un nuovo metodo nella classe ValueDecl che incapsuli il controllo del livello di accesso. Sostituisci quindi il codice duplicato in entrambe le funzioni con chiamate a questo nuovo metodo. Assicurati che i test continuino a passare.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80