OpenXRay/xray-16

Bug in _types.h

Aperta

#195 aperta il 21 apr 2018

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)C++ (519 fork)auto 404
Code QualityHelp wanted

Metriche repository

Star
 (3524 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

https://github.com/OpenXRay/xray-16/blob/70f130c6fc719293c3b6abb1ccacbb084eaee6d1/src/xrCore/_types.h#L31 Is there problem with determination min values here ? For example -std::numeric_limits::max() != std::numeric_limits::min() also for signed types. http://en.cppreference.com/w/cpp/types/numeric_limits Also can replace type_zero to std::numeric_limits::lowest() ?

Guida contributor

Direzione di ricerca
Esamina il file types.h, in particolare la riga 31, per capire l'uso corrente di std::numeric limits. Verifica se i valori minimi per i tipi signed sono definiti erroneamente usando std::numeric limits<int>::max() invece di std::numeric limits<int>::min(). Considera di sostituire type zero con std::numeric limits<int>::lowest() per i tipi signed. Consulta la documentazione cppreference per std::numeric limits per confermare l'uso corretto.
Tech stack
cpp
Dominio
game dev
Tipo issue
Bug
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.
65