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

Support deeply nested variable names #137

Merged
merged 1 commit into from Sep 26, 2017

Conversation

Projects
None yet
1 participant
@50Wliu
Member

50Wliu commented Sep 25, 2017

Changes the optional match for accessors to a multiple match, and changes the non-capturing group to an atomic group to avoid potential catastrophic backtracking.

Fixes #135

@50Wliu

This comment has been minimized.

Show comment
Hide comment
@50Wliu

50Wliu Sep 26, 2017

Member

Merging, when I have more time I'll try to work out the comprehensive solution.

Member

50Wliu commented Sep 26, 2017

Merging, when I have more time I'll try to work out the comprehensive solution.

@50Wliu 50Wliu merged commit fd91d0c into master Sep 26, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@50Wliu 50Wliu deleted the wl-multiple-accessors branch Sep 26, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment