← Back to index | Original Bugzilla link

Bug 14920 – [REG2.067.0] SList.insertAfter on uninitialized list triggers assertion in _first

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P1
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2015-08-14T10:29:00Z
Last change time
2017-07-19T17:43:44Z
Keywords
pull
Assigned to
nobody
Creator
secondaryAccount

Comments

Comment #0 by secondaryAccount — 2015-08-14T10:29:39Z
see: http://forum.dlang.org/post/[email protected] import std.container: SList; void main() { SList!int tmp; tmp.insertAfter( tmp[], 3 ); } triggers assertion in SList._first(), line 57 because the list is not initialized. I am working on a PR.
Comment #1 by ag0aep6g — 2015-08-14T14:11:36Z
Works with 2.066, fails since 2.067, making this a regression.
Comment #2 by secondaryAccount — 2015-08-14T19:16:26Z
https://github.com/D-Programming-Language/phobos/pull/3554
Comment #3 by github-bugzilla — 2015-08-16T10:57:14Z
Commits pushed to stable at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/0c28c7717437b10570677994112fc874e196d26a fix issue 14920 https://github.com/D-Programming-Language/phobos/commit/aac09abf0a8f8624e81bbac4ded38b941cc768d9 Merge pull request #3557 from tosttost/issue_14920 fix issue 14920 - [REG2.067.0] SList.insertAfter on uninitialized list triggers assertion in _first targeting stable branch
Comment #4 by github-bugzilla — 2015-09-07T16:46:47Z
Commits pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/0c28c7717437b10570677994112fc874e196d26a fix issue 14920 https://github.com/D-Programming-Language/phobos/commit/aac09abf0a8f8624e81bbac4ded38b941cc768d9 Merge pull request #3557 from tosttost/issue_14920
Comment #5 by github-bugzilla — 2017-07-19T17:43:44Z
Commits pushed to dmd-cxx at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/0c28c7717437b10570677994112fc874e196d26a fix issue 14920 https://github.com/dlang/phobos/commit/aac09abf0a8f8624e81bbac4ded38b941cc768d9 Merge pull request #3557 from tosttost/issue_14920