OpenXRay/xray-16

Bug in _types.h

オープン

#195 opened on 2018/04/21

GitHub で見るコントリビューターガイド
 (4 件のコメント) (0 件のリアクション) (0 人の担当者)C++ (519 件のフォーク)auto 404
Code QualityHelp wanted

Repository metrics

Stars
 (3,524 個のスター)
PR merge metrics
 (PR metrics pending)

説明

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

コントリビューターガイド

調査方針
types.h ファイル、特に31行目を調べて、現在のstd::numeric limitsの使用法を理解してください。符号付き型の最小値が std::numeric limits<int>::max()ではなくstd::numeric limits<int>::min()であるべきか確認してください。符号付き型のtype zeroをstd::numeric limits<int>::lowest()に置き換えることを検討してください。正しい使用法を確認するために、std::numeric limitsのcppreferenceドキュメントを参照してください。
技術スタック
cpp
領域
game dev
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C++
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
65