Thread overview | |||||
---|---|---|---|---|---|
|
December 15, 2016 std.experimental.checkedint Formal Review | ||||
---|---|---|---|---|
| ||||
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. |
December 15, 2016 Re: std.experimental.checkedint Formal Review | ||||
---|---|---|---|---|
| ||||
Posted in reply to Robert burner Schadek | **2017-01-05 (AoE)** of course |
January 05, 2017 Re: std.experimental.checkedint Formal Review | ||||
---|---|---|---|---|
| ||||
Posted in reply to Robert burner Schadek | 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. |
Copyright © 1999-2021 by the D Language Foundation