www.digitalmars.com         C & C++   DMDScript  

digitalmars.D - std.regex needs help

reply Dmitry Olshansky <dmitry.olsh gmail.com> writes:
I know it's not unusual to rant about open bugs and such but it's been 4 
months and I'm helpless.

I had some nice breakthrough in std.regex performance back in May but 
failed to deliver it to the mainline because of a bug:
https://github.com/dlang/phobos/pull/4286

If anyone fluent in backend can decipher issue:
https://issues.dlang.org/show_bug.cgi?id=15989
I'd indebt.

I'm not above a workaround if that's what it takes.

---
Dmitry Olshansky
Sep 30 2016
next sibling parent reply Stefan Koch <uplink.coder googlemail.com> writes:
On Friday, 30 September 2016 at 17:29:41 UTC, Dmitry Olshansky 
wrote:

 I know it's not unusual to rant about open bugs and such but 
 it's been 4 months and I'm helpless.

 I had some nice breakthrough in std.regex performance back in 
 May but failed to deliver it to the mainline because of a bug:
 https://github.com/dlang/phobos/pull/4286

 If anyone fluent in backend can decipher issue:
 https://issues.dlang.org/show_bug.cgi?id=15989
 I'd indebt.

 I'm not above a workaround if that's what it takes.

 ---
 Dmitry Olshansky


You can workaround the issue by making ctRegex a static immutable 
instead of an enum.
Sep 30 2016
parent Walter Bright <newshound2 digitalmars.com> writes:
On 9/30/2016 11:45 AM, Stefan Koch wrote:

 You can workaround the issue by making ctRegex a static immutable instead of an
 enum.


That should work, but I wouldn't call it a workaround. It's the right way to do 
that.
Sep 30 2016
prev sibling parent Dmitry Olshansky <dmitry.olsh gmail.com> writes:
On 9/30/16 7:29 PM, Dmitry Olshansky wrote:

 I know it's not unusual to rant about open bugs and such but it's been 4
 months and I'm helpless.

 I had some nice breakthrough in std.regex performance back in May but
 failed to deliver it to the mainline because of a bug:
 https://github.com/dlang/phobos/pull/4286


Thanks to everyone involved, this is now a healthy green pull request.


---
Dmitry Olshansky
Oct 07 2016