diasurgical/DevilutionX

Prefer scoped enums where possible

Aperta

#2300 aperta il 4 lug 2021

Vedi su GitHubGuida contributor
 (4 commenti) (1 reazione) (0 assegnatari)C++ (743 fork)batch import
help wanted

Metriche repository

Star
 (7255 stelle)
Metriche merge PR
 (Merge medio 2g 12h) (15 PR mergiate in 30 g)

Descrizione

Many of the unscoped enums used in devilutionX can use enum class instead. This gives stronger type safety for things like Direction and allows unambiguous constructors/conversion functions in related classes (e.g. Displacement).

This does make it more difficult to cast back to an integral type so the usage of some enums (such as _item_indexes) will need to be evaluated to see if it's worth changing those from unscoped to scoped.

Guida contributor

Direzione di ricerca
Controlla ogni enum senza scope nel codebase e valuta se possono essere convertiti in enum class senza modifiche sostanziali, prestando attenzione ai cast interi.
Tech stack
cpp
Dominio
desktop
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70