OpenXRay/xray-16

Bug in _types.h

Ouverte

#195 ouverte le 21 avr. 2018

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)C++ (519 forks)auto 404
Code QualityHelp wanted

Métriques du dépôt

Stars
 (3 524 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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() ?

Guide contributeur

Direction de recherche
Examinez le fichier types.h, en particulier la ligne 31, pour comprendre l'utilisation actuelle de std::numeric limits. Vérifiez si les valeurs minimales pour les types signés sont définies incorrectement en utilisant std::numeric limits<int>::max() au lieu de std::numeric limits<int>::min(). Envisagez de remplacer type zero par std::numeric limits<int>::lowest() pour les types signés. Consultez la documentation cppreference pour std::numeric limits afin de confirmer l'utilisation correcte.
Stack technique
cpp
Domaine
game dev
Type d'issue
Bug
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.
Active
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.
65