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


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...