OpenXRay/xray-16

Bug in _types.h

Offen

#195 geöffnet am 21.04.2018

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (519 Forks)auto 404
Code QualityHelp wanted

Repository-Metriken

Stars
 (3.524 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die Datei types.h, insbesondere Zeile 31, um die aktuelle Verwendung von std::numeric limits zu verstehen. Prüfe, ob die Minimalwerte für vorzeichenbehaftete Typen fälschlicherweise mit std::numeric limits<int>::max() anstelle von std::numeric limits<int>::min() definiert sind. Erwäge, type zero für vorzeichenbehaftete Typen durch std::numeric limits<int>::lowest() zu ersetzen. Konsultiere die cppreference Dokumentation für std::numeric limits, um die korrekte Verwendung zu bestätigen.
Tech Stack
cpp
Domain
game dev
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65