Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug(coffeescript) fix freezing bug due to badly behaved regex #2376

Merged
merged 6 commits into from
Feb 6, 2020

Conversation

joshgoebel
Copy link
Member

@joshgoebel joshgoebel commented Jan 31, 2020

Closes #2375

@joshgoebel joshgoebel requested a review from egor-rogov January 31, 2020 16:09
@joshgoebel joshgoebel added language bug hotfix bad bug, should fix and release ASAP labels Jan 31, 2020
@joshgoebel
Copy link
Member Author

@marcoscaceres Or you could approve also if you have time to review. Only pinging you since I consider freeze issues pretty serious but I'd still like to have more than just my eyeballs on this.

@joshgoebel joshgoebel added this to the 9.18.1 milestone Jan 31, 2020
@joshgoebel
Copy link
Member Author

Actually I just realized I can both release a new stable version and hold this open for review at the same time, so that's what I'm going to do. :-) The power of Git.

@joshgoebel joshgoebel requested a review from allejo February 4, 2020 16:58
@joshgoebel
Copy link
Member Author

This has been live in 9.18.1 for 5 days now with no reported issue. Going to merge and close this, but if anyone still seems an issue with it later we can always review.

@joshgoebel joshgoebel merged commit 2430e37 into highlightjs:master Feb 6, 2020
@egor-rogov
Copy link
Collaborator

Looks sane. Sorry I'm late.

@joshgoebel
Copy link
Member Author

Better late than never, so I've heard. :-p

@joshgoebel joshgoebel deleted the coffeescript_regex_freeze branch February 15, 2020 13:11
taufik-nurrohman pushed a commit to taufik-nurrohman/highlight.js that referenced this pull request Feb 18, 2020
…ghtjs#2376)

* bug(coffeescript) fix freezing bug due to badly behaved regex
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug hotfix bad bug, should fix and release ASAP language
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(coffeescript) Freezing, poorly behaved regex to match regex.
2 participants