diasurgical/DevilutionX

Cleanup manual value clamping

Aperta

#1666 aperta il 24 apr 2021

Vedi su GitHubGuida contributor
 (2 commenti) (1 reazione) (0 assegnatari)C++ (743 fork)batch import
good first issuehelp wanted

Metriche repository

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

Descrizione

The code is littered with conditions that check that values do not go over or under the min/max range.

This is relatively easy but tedious to clean up, but doing so greatly helps the readability of the code.

An example of such cleanups in inv.cpp and a couple of other files: https://github.com/diasurgical/devilutionX/pull/1665

Guida contributor

Direzione di ricerca
Rivedi la PR collegata #1665 per comprendere lo schema utilizzato per la pulizia del clamping. Quindi cerca nella codebase (a partire da inv.cpp e altri file) i controlli manuali degli intervalli di valori min/max. Sostituiscili con le opportune funzioni di utilità.
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75