Lookbehind in ctRegex #1485
Merged
Lookbehind in ctRegex #1485
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I think I broke this pulling #1471 . Needs a rebase :/ |
|
Closing to resolve the rebase mess... |
|
Rebased and fixed. |
|
Another reduction in tests to let poor FreeBSD 32 complete the test build... |
|
awes |
|
please rebase, I guess it clashes with #1484 which I just merged |
|
I'd suggest first merging all the commits that correspond to #1484 into a single commit, and then re-basing over head. With some luck, it might even auto-merge. |
|
I just made a patch by diffing with new master. Now it's one tiny commit, let's see how it goes with the tester. |
|
Lookin' good OT. I no longer see test runner icons. Is that only me? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is it, all tests of run-time tests on lookaround are now passing with ctRegex as well.
The pull includes pull #1484, I'll rebase once it's merged.
P.S. I've hit a curious bug with CTFE in std.array.Appender that I'll narrow down later, it almost feels like good ol' times ;)