[Issue 14832] iota(UL) + array doesn't work

Jul 26, 2015

bearophile_hugs@eml.cc

Aug 10, 2015

Steven Schveighoffer

Aug 15, 2015

Aug 15, 2015

Aug 16, 2015

Aug 19, 2015

Aug 19, 2015

Aug 22, 2015

hsteoh@quickfur.ath.cx

Dec 17, 2022

Iain Buclaw

Dec 01

dlangBugzillaToGithub

https://issues.dlang.org/show_bug.cgi?id=14832 bearophile_hugs@eml.cc changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |normal --

https://issues.dlang.org/show_bug.cgi?id=14832 Steven Schveighoffer <schveiguy@yahoo.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |pull Assignee|nobody@puremagic.com |schveiguy@yahoo.com --- Comment #1 from Steven Schveighoffer <schveiguy@yahoo.com> --- PR: https://github.com/D-Programming-Language/phobos/pull/3544 --

https://issues.dlang.org/show_bug.cgi?id=14832 Walter Bright <bugzilla@digitalmars.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bugzilla@digitalmars.com --- Comment #2 from Walter Bright <bugzilla@digitalmars.com> --- To me this looks invalid. I see no reason to support creating arrays from ulongs. --

https://issues.dlang.org/show_bug.cgi?id=14832 Dmitry Olshansky <dmitry.olsh@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dmitry.olsh@gmail.com --- Comment #3 from Dmitry Olshansky <dmitry.olsh@gmail.com> --- Also the funny thing is that https://github.com/D-Programming-Language/phobos/pull/3544 adds support for ulong length which AFAIK only iota on ulong give and BECAUSE the element type is ulong.sizeof == 8 bytes long nobody will ever be able to call array on > ~512 million long iota. So all of that effort is leading nowhere, just drop the ulong length please. --

August 16, 2015

[Issue 14832] iota(UL) + array doesn't work

Posted by Steven Schveighoffer

Permalink

Steven Schveighoffer

Permalink

https://issues.dlang.org/show_bug.cgi?id=14832

Steven Schveighoffer <schveiguy@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|Windows                     |All

--- Comment #4 from Steven Schveighoffer <schveiguy@yahoo.com> ---
(In reply to Walter Bright from comment #2)
> To me this looks invalid. I see no reason to support creating arrays from ulongs.

You mean from ranges with ulong length? I don't see why. ulong[] is a valid type, and a reasonable request. How do I make a ulong[] from iota on 32-bit?

(In reply to Dmitry Olshansky from comment #3)
> Also the funny thing is that https://github.com/D-Programming-Language/phobos/pull/3544
> 
> adds support for ulong length which AFAIK only iota on ulong give and BECAUSE the element type is ulong.sizeof == 8 bytes long nobody will ever be able to call array on > ~512 million long iota.

The point is not to make arrays of larger than size_t elements, the point is to make arrays out of small ranges that happen to use ulong as a length.

> So all of that effort is leading nowhere, just drop the ulong length please.

You mean from iota? or support from array? Why should a range that has a ulong length of 10 be disallowed?

Note that a file may have ulong length even on a 32-bit system. This would mean creating an array out of a file stream range (that reasonably supplies a ulong length) would be disallowed, even when the length is small.

I really don't understand the objections here, this code is very reasonable. It means code that makes sense on both 32-bit and 64-bit systems fails on 32-bit systems.

--

https://issues.dlang.org/show_bug.cgi?id=14832 Steven Schveighoffer <schveiguy@yahoo.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|pull | Assignee|schveiguy@yahoo.com |nobody@puremagic.com --- Comment #5 from Steven Schveighoffer <schveiguy@yahoo.com> --- PR seems to be rejected by 3 other major players, so maybe someone else will have better luck with a different solution. --

https://issues.dlang.org/show_bug.cgi?id=14832 hsteoh@quickfur.ath.cx changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |pull CC| |hsteoh@quickfur.ath.cx --- Comment #6 from hsteoh@quickfur.ath.cx --- Alternative fix: https://github.com/D-Programming-Language/phobos/pull/3571 --

https://issues.dlang.org/show_bug.cgi?id=14832 Iain Buclaw <ibuclaw@gdcproject.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P1 |P3 --

https://issues.dlang.org/show_bug.cgi?id=14832 --- Comment #7 from dlangBugzillaToGithub <robert.schadek@posteo.de> --- THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/phobos/issues/10138 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB --

Forums