study: use checkedint as a drop-in replacement of native long

mw mingwu at gmail.com
Mon Aug 17 17:58:16 UTC 2020


On Monday, 17 August 2020 at 17:15:59 UTC, H. S. Teoh wrote:
> On Mon, Aug 17, 2020 at 05:10:01PM +0000, mw via Digitalmars-d 
> wrote:
>> 
>> https://issues.dlang.org/show_bug.cgi?id=21169
>
> You might want to consider splitting up that bug into multiple 
> smaller, more self-contained bugs.  Less intimidating that way 
> for potential contributors to fix. ;-)

Was hesitating between splitting or not, then thought user may 
want to take the full picture into consideration the best 
approach to fix the issue, e.g. in case 
one-stone-that-can-kill-multiple-birds.

(People can always submit separate a PR to fix one particular 
issue).


> Chain assignment fix: https://github.com/dlang/phobos/pull/7599

Thanks for the PR, I just added comments: does this fix also work 
for mixed native & checked chain assignment? i.e. add to unittest:

```
   long la, lb;
   Checked!long ca, cb;

   la = ca = lb = cb;  // mixed chain assign
   ca = la = cb = lb;
```



More information about the Digitalmars-d mailing list