← Back to index | Original Bugzilla link

Bug 17667 – regex([r".", r"[\(\{[\]\}\)]"]);

Status
RESOLVED
Resolution
FIXED
Severity
major
Priority
P1
Component
phobos
Product
D
Version
D2
Platform
x86_64
OS
Linux
Creation time
2017-07-19T22:25:53Z
Last change time
2018-01-05T13:30:40Z
Assigned to
Dmitry Olshansky
Creator
Nikolay (unDEFER) Krivchenkov

Comments

Comment #0 by undefer — 2017-07-19T22:25:53Z
The code demonstrating not working: https://dpaste.dzfl.pl/72a39b6bb9eb
Comment #1 by dmitry.olsh — 2017-07-25T13:15:55Z
This is a peculiar bug in that the first pattern is accepts invalid. There is unbalanced grouping of []. It interestingly goes undetected only to explode when combined with another pattern. Fix pending.
Comment #2 by dmitry.olsh — 2017-07-25T13:48:25Z
https://github.com/dlang/phobos/pull/5656
Comment #3 by github-bugzilla — 2017-08-07T09:48:29Z
Commits pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/42c0d79e5a405e12bae3b6ae642c9b2a1e2bf06e Fix issue 17667 - regex([r".", r"[\(\{[\]\}\)]"]); https://github.com/dlang/phobos/commit/284e8eee1c8783340d92e4107303a5f54c9284a9 Merge pull request #5656 from DmitryOlshansky/issue-17667 Fix issue 17667 - regex([r".", r"[\(\{[\]\}\)]"]); merged-on-behalf-of: Dmitry Olshansky <[email protected]>
Comment #4 by github-bugzilla — 2017-08-16T13:13:52Z
Commits pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/42c0d79e5a405e12bae3b6ae642c9b2a1e2bf06e Fix issue 17667 - regex([r".", r"[\(\{[\]\}\)]"]); https://github.com/dlang/phobos/commit/284e8eee1c8783340d92e4107303a5f54c9284a9 Merge pull request #5656 from DmitryOlshansky/issue-17667
Comment #5 by github-bugzilla — 2018-01-05T13:30:40Z
Commits pushed to dmd-cxx at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/42c0d79e5a405e12bae3b6ae642c9b2a1e2bf06e Fix issue 17667 - regex([r".", r"[\(\{[\]\}\)]"]); https://github.com/dlang/phobos/commit/284e8eee1c8783340d92e4107303a5f54c9284a9 Merge pull request #5656 from DmitryOlshansky/issue-17667