Thread overview
[Issue 17458] [std.regex] Assertion failure in package.d
Jun 02, 2017
Vladimir Panteleev
Jul 26, 2017
Dmitry Olshansky
Jul 26, 2017
Vladimir Panteleev
Jul 26, 2017
Dmitry Olshansky
June 02, 2017
https://issues.dlang.org/show_bug.cgi?id=17458

Vladimir Panteleev <thecybershadow@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |thecybershadow@gmail.com
           Assignee|nobody@puremagic.com        |dmitry.olsh@gmail.com

--
July 26, 2017
https://issues.dlang.org/show_bug.cgi?id=17458

--- Comment #1 from Dmitry Olshansky <dmitry.olsh@gmail.com> ---
Not a bug - there is no match, before calling matches.hit one must check it for !matches.empty.

--
July 26, 2017
https://issues.dlang.org/show_bug.cgi?id=17458

--- Comment #2 from Vladimir Panteleev <dlang-bugzilla@thecybershadow.net> ---
The error message needs to be improved, then. "Assertion failure" is devoid of any information.

See also: https://github.com/dlang/phobos/pull/5578

--
July 26, 2017
https://issues.dlang.org/show_bug.cgi?id=17458

--- Comment #3 from Dmitry Olshansky <dmitry.olsh@gmail.com> ---
https://github.com/dlang/phobos/pull/5660

--
July 26, 2017
https://issues.dlang.org/show_bug.cgi?id=17458

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

https://github.com/dlang/phobos/commit/7a51c0e24eea6c02336d2aa897c608974ea2ac57 Fix issue 17458 add messages to asserts

https://github.com/dlang/phobos/commit/008a8a3ef3fab4e1f51f2bf65d3f6b42dc152f78 Merge pull request #5660 from DmitryOlshansky/issue-17458

[std.regex] Fix issue 17458 - add messages to asserts merged-on-behalf-of: Stefan Koch <UplinkCoder@users.noreply.github.com>

--
August 16, 2017
https://issues.dlang.org/show_bug.cgi?id=17458

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

https://github.com/dlang/phobos/commit/7a51c0e24eea6c02336d2aa897c608974ea2ac57 Fix issue 17458 add messages to asserts

https://github.com/dlang/phobos/commit/008a8a3ef3fab4e1f51f2bf65d3f6b42dc152f78 Merge pull request #5660 from DmitryOlshansky/issue-17458

--
January 05, 2018
https://issues.dlang.org/show_bug.cgi?id=17458

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

https://github.com/dlang/phobos/commit/7a51c0e24eea6c02336d2aa897c608974ea2ac57 Fix issue 17458 add messages to asserts

https://github.com/dlang/phobos/commit/008a8a3ef3fab4e1f51f2bf65d3f6b42dc152f78 Merge pull request #5660 from DmitryOlshansky/issue-17458

--