• Robert burner Schadek (17/17) Dec 15 2016 It is time to formally review Andrei's checkedint.
• Robert burner Schadek (1/1) Dec 15 2016 **2017-01-05 (AoE)** of course
• Robert burner Schadek (4/4) Jan 05 2017 The review is over, there has been one comment on github "build

Robert burner Schadek <rburners gmail.com> writes:

It is time to formally review Andrei's checkedint.

Checkedint is a wrapper for integer types that allows to run code 
on over/underflows, define NaN like init states and more.

The PR can be found here:
https://github.com/dlang/phobos/pull/4613

The dub package can be found here: (1)
http://code.dlang.org/packages/checkedint_andralex

The docs can be seen through CyberShadow/DAutoTest in the PR.

Due to the holidays the review period is extended to **2016-01-05 
(AoE)**

Please browse through the comments of the PR to keep this 
discussion DRY.
Style nitpicks should be made on github.

The discussion in this thread should be about the design and 
phobos inclusion worthiness (PIW) of the module, not about where 
to place braces.


(1) Andrei, please make sure they are in sync.

Robert burner Schadek <rburners gmail.com> writes:

**2017-01-05 (AoE)** of course

Robert burner Schadek <rburners gmail.com> writes:

The review is over, there has been one comment on github "build 
error with DAutoTest:". It has been fixed.

If there are no comments until 2017-01-12 and Andrei agrees I 
will put checkedint to a vote.