Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Fixes #87: Do not cache search results if regular expression contains \G #88

Merged
merged 1 commit into from
Sep 19, 2018
Merged

Fixes #87: Do not cache search results if regular expression contains \G #88

merged 1 commit into from
Sep 19, 2018

Conversation

alexdima
Copy link
Contributor

@alexdima alexdima commented Sep 19, 2018

See #87

  • Disables caching if the regular expression contains \G.
  • Adds a regression test.

(fyi @daviwil @maxbrunsfeld)

Copy link
Contributor

@maxbrunsfeld maxbrunsfeld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for tracking this down @alexandrudima. I just left one suggestion.

src/onig-reg-exp.cc Show resolved Hide resolved
@maxbrunsfeld maxbrunsfeld merged commit bfb1753 into atom:master Sep 19, 2018
@maxbrunsfeld
Copy link
Contributor

This change is in v7.0.2.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants