← Back to index | Original Bugzilla link

Bug 13314 – BinaryHeap assumes Store has dup property

Status: RESOLVED
Resolution: FIXED
Severity: normal
Priority: P1
Component: phobos
Product: D
Version: D2
Platform: All
OS: All
Creation time: 2014-08-17T22:00:00Z
Last change time: 2017-01-16T23:25:17Z
Assigned to: nobody
Creator: safety0ff.bugz

Comments

Comment #0 by safety0ff.bugz — 2014-08-17T22:00:30Z

Example: import std.range; import std.container; void main() { int[] arr = [1,2,3,4,5]; auto heap = (&arr).refRange.heapify; }

Comment #1 by lucia.mcojocaru — 2016-12-02T20:43:58Z

PR https://github.com/dlang/phobos/pull/4929

Comment #2 by github-bugzilla — 2016-12-07T03:54:33Z

Commits pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/fba056a8df961138859561a6f4ea3d9447efbd6d fix issue 13314 https://github.com/dlang/phobos/commit/9eb35e11b39ed54fea8b6f4fc18503c86bddc05a Merge pull request #4929 from somzzz/issue_13314 fix issue 13314 - BinaryHeap assumes Store has dup property

Comment #3 by github-bugzilla — 2017-01-07T03:02:45Z

Commits pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/fba056a8df961138859561a6f4ea3d9447efbd6d fix issue 13314 https://github.com/dlang/phobos/commit/9eb35e11b39ed54fea8b6f4fc18503c86bddc05a Merge pull request #4929 from somzzz/issue_13314

Comment #4 by github-bugzilla — 2017-01-16T23:25:17Z

Commits pushed to newCTFE at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/fba056a8df961138859561a6f4ea3d9447efbd6d fix issue 13314 https://github.com/dlang/phobos/commit/9eb35e11b39ed54fea8b6f4fc18503c86bddc05a Merge pull request #4929 from somzzz/issue_13314