← Back to index | Original Bugzilla link

Bug 12890 – std.array index based replace

Status: RESOLVED
Resolution: FIXED
Severity: enhancement
Priority: P1
Component: phobos
Product: D
Version: D2
Platform: All
OS: All
Creation time: 2014-06-11T15:14:00Z
Last change time: 2014-08-17T17:41:51Z
Keywords: pull
Assigned to: nobody
Creator: safety0ff.bugz

Comments

Comment #0 by safety0ff.bugz — 2014-06-11T15:14:22Z

Currently there is some 3+ year old code commented out in std.array: https://github.com/D-Programming-Language/phobos/blob/master/std/array.d#L1921 and: https://github.com/D-Programming-Language/phobos/blob/master/std/array.d#L2076 This code is for replace with signature: T[] replace(T, Range)(T[] subject, size_t from, size_t to, Range stuff) Either we remove the commented out code or uncomment is and verify that it is still correct.

Comment #1 by safety0ff.bugz — 2014-06-11T22:34:45Z

https://github.com/D-Programming-Language/phobos/pull/2240

Comment #2 by github-bugzilla — 2014-08-17T17:41:50Z

Commits pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/c4f1c43366c79f4ff9ddfadbc0a8c943e0cb5c94 Fix issue 12890 - index based replace https://github.com/D-Programming-Language/phobos/commit/68833f0b9a1720f5764cc7f33c6be9b3a06f9d54 Merge pull request #2240 from Safety0ff/fix12890 Fix issue 12890 - index based replace