[Issue 17668] regex(q"<[^]>")

2018-01-05 Thread d-bugmail--- via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=17668

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

https://github.com/dlang/phobos/commit/ba15802664b806ffc2a56e2ef1cec4f6ce523dac
Fix issue 17668 - assert failure regex(q"<[^]>")

https://github.com/dlang/phobos/commit/bcb17a6d88c5107fc9df388b6e1a1fe84f6bf90c
Merge pull request #5657 from DmitryOlshansky/issue-17668

--

[Issue 17668] regex(q"<[^]>")

2017-08-16 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=17668

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

https://github.com/dlang/phobos/commit/ba15802664b806ffc2a56e2ef1cec4f6ce523dac
Fix issue 17668 - assert failure regex(q"<[^]>")

https://github.com/dlang/phobos/commit/bcb17a6d88c5107fc9df388b6e1a1fe84f6bf90c
Merge pull request #5657 from DmitryOlshansky/issue-17668

--

[Issue 17668] regex(q"<[^]>")

2017-08-07 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=17668

github-bugzi...@puremagic.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--

[Issue 17668] regex(q"<[^]>")

2017-08-07 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=17668

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

https://github.com/dlang/phobos/commit/ba15802664b806ffc2a56e2ef1cec4f6ce523dac
Fix issue 17668 - assert failure regex(q"<[^]>")

https://github.com/dlang/phobos/commit/bcb17a6d88c5107fc9df388b6e1a1fe84f6bf90c
Merge pull request #5657 from DmitryOlshansky/issue-17668

Fix issue 17668 - assert failure regex(q"<[^]>")
merged-on-behalf-of: Dmitry Olshansky
<dmitryolshan...@users.noreply.github.com>

--

[Issue 17668] regex(q"<[^]>")

2017-07-26 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=17668

--- Comment #1 from Dmitry Olshansky  ---
https://github.com/dlang/phobos/pull/5657

--

[Issue 17668] regex(q"<[^]>")

2017-07-20 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=17668

Nikolay (unDEFER) Krivchenkov  changed:

   What|Removed |Added

 CC||unde...@gmail.com

--

[Issue 17668] regex(q"<[^]>")

2017-07-20 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=17668

Dmitry Olshansky  changed:

   What|Removed |Added

 CC||dmitry.o...@gmail.com
   Assignee|nob...@puremagic.com|dmitry.o...@gmail.com

--

Re: regex(q"<[^]>")

2017-07-19 Thread unDEFER via Digitalmars-d 
On Wednesday, 19 July 2017 at 16:59:24 UTC, Dmitry Olshansky 
wrote:

On Monday, 17 July 2017 at 17:00:10 UTC, unDEFER wrote:

Please file at:
issues.dlang.org

Thanks!

---
Dmitry Olshansky


Thank you,
https://issues.dlang.org/show_bug.cgi?id=17668
https://issues.dlang.org/show_bug.cgi?id=17667

[Issue 17668] New: regex(q"<[^]>")

2017-07-19 Thread via Digitalmars-d-bugs 
https://issues.dlang.org/show_bug.cgi?id=17668

  Issue ID: 17668
   Summary: regex(q"<[^]>")
   Product: D
   Version: D2
  Hardware: x86_64
OS: Linux
Status: NEW
  Severity: normal
  Priority: P1
 Component: phobos
  Assignee: nob...@puremagic.com
  Reporter: unde...@gmail.com

regex(q"<[^]>") leads to assertion, but user input may lead only to exception.

--

Re: regex(q"<[^]>")

2017-07-19 Thread Dmitry Olshansky via Digitalmars-d 

On Monday, 17 July 2017 at 17:00:10 UTC, unDEFER wrote:

Hello!
The code in the header leads to assertion!
But the user inputed data don't must leads to any assertions!


Please file at:
issues.dlang.org

Thanks!

---
Dmitry Olshansky

Re: regex(q"<[^]>")

2017-07-18 Thread Temtaime via Digitalmars-d 

On Tuesday, 18 July 2017 at 16:34:39 UTC, unDEFER wrote:

On Tuesday, 18 July 2017 at 08:56:12 UTC, Anton Fediushin wrote:
Not a bug, but I think that `regex()` should fail with a nice 
exception, not silently fail.


Yes, exception, not assert.


The forum is not a bugtracker

Re: regex(q"<[^]>")

2017-07-18 Thread unDEFER via Digitalmars-d 

On Tuesday, 18 July 2017 at 08:56:12 UTC, Anton Fediushin wrote:
Not a bug, but I think that `regex()` should fail with a nice 
exception, not silently fail.


Yes, exception, not assert.

Re: regex(q"<[^]>")

2017-07-18 Thread Anton Fediushin via Digitalmars-d 

On Monday, 17 July 2017 at 17:00:10 UTC, unDEFER wrote:

Hello!
The code in the header leads to assertion!
But the user inputed data don't must leads to any assertions!


This regular expression is invalid.
[bar] - Matches 'b' or 'a' or 'r'
[^bar] - Matches everything but 'b' or 'a' or 'r'.

So, [^] - is an invalid regex.
^ in this case should be escaped:
[\^] - matches '^'.

Not a bug, but I think that `regex()` should fail with a nice 
exception, not silently fail.

Re: regex(q"<[^]>")

2017-07-17 Thread unDEFER via Digitalmars-d 

On Monday, 17 July 2017 at 20:43:29 UTC, unDEFER wrote:

Sorry, fixed in the newest DMD also as the other bug in regex...


Oh, no. Not fixed. Fixed only other bug.

Re: regex(q"<[^]>")

2017-07-17 Thread unDEFER via Digitalmars-d 

Sorry, fixed in the newest DMD also as the other bug in regex...

regex(q"<[^]>")

2017-07-17 Thread unDEFER via Digitalmars-d 

Hello!
The code in the header leads to assertion!
But the user inputed data don't must leads to any assertions!