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

monarchdodra@gmail.com changed:

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

--- Comment #1 from monarchdodra@gmail.com ---
That's not fixed yet !?

--

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

safety0ff.bugz <safety0ff.bugz@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |safety0ff.bugz@gmail.com

--- Comment #2 from safety0ff.bugz <safety0ff.bugz@gmail.com> ---
I think the bug you reported is not valid.
However, there is a bug when using length to shorten the array:

import std.container;

void main() {
  Array!int a; // empty Array
  a.length = 10; // a contains 10 default ints
  a.length = 9; // Remove one, should have capacity > 0
  assert(a.capacity > 0);
}

--

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

--- Comment #3 from thedeemon <dlang@thedeemon.com> ---
Why do you think it's not valid?

It does throw runtime error and capacity is wrong after setting the length. http://dpaste.dzfl.pl/b10858af2a56

--

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

--- Comment #4 from safety0ff.bugz <safety0ff.bugz@gmail.com> ---
Ah, I misinterpreted the meaning of the capacity (unused capacity vs total
capacity.)

Perhaps the title should be "std.container.array capacity not updated when changing length" or something like that since it's not specific to increasing length.

--

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

Eduard Staniloiu <edi33416@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |trivial
                 CC|                            |edi33416@gmail.com

--

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

--- Comment #5 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/8ba7ce59ba4bbeb1e7f45fb34ecff2a02930aad9 Fix issue 13619 - array capacity not updated when changing length

https://github.com/dlang/phobos/commit/9872247505c8a62e87bb97010084406c051eed2f Merge pull request #5267 from byebye/issue_13619

Fix issue 13619 - array capacity not updated when changing length merged-on-behalf-of: Jack Stouffer <jack@jackstouffer.com>

--

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

github-bugzilla@puremagic.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--

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

--- Comment #6 from github-bugzilla@puremagic.com ---
Commits pushed to stable at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/8ba7ce59ba4bbeb1e7f45fb34ecff2a02930aad9 Fix issue 13619 - array capacity not updated when changing length

https://github.com/dlang/phobos/commit/9872247505c8a62e87bb97010084406c051eed2f Merge pull request #5267 from byebye/issue_13619

--

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

--- Comment #7 from github-bugzilla@puremagic.com ---
Commits pushed to newCTFE at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/8ba7ce59ba4bbeb1e7f45fb34ecff2a02930aad9 Fix issue 13619 - array capacity not updated when changing length

https://github.com/dlang/phobos/commit/9872247505c8a62e87bb97010084406c051eed2f Merge pull request #5267 from byebye/issue_13619

--

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

--- Comment #8 from github-bugzilla@puremagic.com ---
Commits pushed to dmd-cxx at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/8ba7ce59ba4bbeb1e7f45fb34ecff2a02930aad9 Fix issue 13619 - array capacity not updated when changing length

https://github.com/dlang/phobos/commit/9872247505c8a62e87bb97010084406c051eed2f Merge pull request #5267 from byebye/issue_13619

--