OpenXRay / OpenXRay/xray-16

Bug in _types.h

Open
#195 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Code Quality Help wanted
Dominant language
C++
Stars
3.6k
Forks
536
Avg merge
6d 1h
Merged PRs (30d)
3

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

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at src/xrCore/_types.h:31 and compare the min-value determination with the C++ std::numeric_limits documentation linked in the issue. Verify the behavior for signed integer types, then run the relevant existing tests or build checks; done means the type helpers correctly represent minimum values without breaking other supported types.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
game-dev
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.