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

ag0aep6g@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ag0aep6g@gmail.com

--- Comment #1 from ag0aep6g@gmail.com ---
This would be a serious breaking change. The documentation says that the function allocates and copies.

--

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

--- Comment #2 from sigod <sigod.mail@gmail.com> ---
It's meaningless for dynamic arrays.

Currently, if you change you code from ranges to arrays you have to remove use of `array()` or it just adds hidden cost.

--

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

--- Comment #3 from ag0aep6g@gmail.com ---
(In reply to sigod from comment #2)
> It's meaningless for dynamic arrays.

Currently std.array.array guarantees one level of duplication. So when I call it on an int[], I can rely on the new array being independent from the original one. I can alter elements without affecting the original. I can cast it to immutable(int)[] without running into undefined behavior when the original is altered.

I'm not saying that this is the best behavior for a function called "array", but that's how it's documented and how it works. Changing it now would be a serious breaking change.

--

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

--- Comment #4 from sigod <sigod.mail@gmail.com> ---
(In reply to ag0aep6g from comment #3)
> (In reply to sigod from comment #2)
> > It's meaningless for dynamic arrays.
> 
> Currently std.array.array guarantees one level of duplication. So when I call it on an int[], I can rely on the new array being independent from the original one. I can alter elements without affecting the original. I can cast it to immutable(int)[] without running into undefined behavior when the original is altered.

One of the first things that I learned in D is that you have `dup` and `idup` "properties" for this. I bet no one uses `array()` for duplicating arrays.

> I'm not saying that this is the best behavior for a function called "array", but that's how it's documented and how it works. Changing it now would be a serious breaking change.

Function called `array` for a reason. Because it gives you array where you have only an input range. It doesn't make sense to do anything if you already have an array. In normal code you will never call `array` for array.

--

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

--- Comment #5 from ag0aep6g@gmail.com ---
(In reply to sigod from comment #4)
> I bet no one uses `array()` for duplicating
> arrays.

I don't think betting on these things is a good course of action. The function is documented to allocate a new array. Changing that is a breaking change.

> Function called `array` for a reason. Because it gives you array where you have only an input range. It doesn't make sense to do anything if you already have an array. In normal code you will never call `array` for array.

Here's a little generic function that relies on std.array.array's current behavior:
----
immutable(ElementType!R)[] toImmutableArray(R)(R range)
    if (isInputRange!R && !hasIndirections!(ElementType!R))
{
    import std.array: array;
    import std.exception: assumeUnique;
    return assumeUnique(range.array);
}
----

--

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

--- Comment #6 from ag0aep6g@gmail.com ---
(In reply to ag0aep6g from comment #5)
> Here's a little generic function that relies on std.array.array's current behavior:
> ----
> immutable(ElementType!R)[] toImmutableArray(R)(R range)
>     if (isInputRange!R && !hasIndirections!(ElementType!R))
> {
>     import std.array: array;
>     import std.exception: assumeUnique;
>     return assumeUnique(range.array);
> }
> ----

Missing imports:
----
import std.range: ElementType, isInputRange;
import std.traits: hasIndirections;
----

--

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

Jack Stouffer <jack@jackstouffer.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jack@jackstouffer.com

--- Comment #7 from Jack Stouffer <jack@jackstouffer.com> ---
(In reply to sigod from comment #4)
> I bet no one uses `array()` for duplicating
> arrays.

Maybe, maybe not. But I'd bet $500 that people rely on the functionality and don't know it. If this is changed you'll have people suddenly wondering why their arrays are getting filled garbage data.

With a function as popular as std.array.array, I'd say this change is way too dangerous.

--

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

--- Comment #8 from sigod <sigod.mail@gmail.com> ---
I'm not betting on anything. It was a figure of speech.

--

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

--- Comment #9 from Jack Stouffer <jack@jackstouffer.com> ---
(In reply to sigod from comment #8)
> I'm not betting on anything. It was a figure of speech.

The best way to call someone's bluff is to put money on the table. If you were sure that no one was using this for duplication, then you would have taken my bet.

--

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

--- Comment #10 from sigod <sigod.mail@gmail.com> ---
(In reply to Jack Stouffer from comment #9)
> (In reply to sigod from comment #8)
> > I'm not betting on anything. It was a figure of speech.
> 
> The best way to call someone's bluff is to put money on the table. If you were sure that no one was using this for duplication, then you would have taken my bet.

I don't gamble. Especially when it's impossible to win.

(Let's end off-topic here.)

--