Skip to content

feat: regexp engine configuration#480

Draft
stevenh wants to merge 1 commit into
masterfrom
feat/regexp-option
Draft

feat: regexp engine configuration#480
stevenh wants to merge 1 commit into
masterfrom
feat/regexp-option

Conversation

@stevenh

@stevenh stevenh commented Dec 6, 2022

Copy link
Copy Markdown
Collaborator

Add support for ECMAScript compatible regular expression engine via regexp2 using functional options e.g.

vm := New(RegExp(regexp2.Creator{}))

This also adds function option support for the existing options:

  • Debugger - Deprecates: SetDebuggerHandler
  • RandomSource - Deprecates: SetRandomSource.
  • Stack - Deprecates: SetStackLimit.
  • StackTrace - Deprecates: SetStackTraceLimit.

@stevenh stevenh force-pushed the feat/regexp-option branch 2 times, most recently from e33f36c to 893429c Compare December 6, 2022 19:13
@stevenh

stevenh commented Dec 6, 2022

Copy link
Copy Markdown
Collaborator Author

Thoughts @andig ?

@andig

andig commented Dec 6, 2022

Copy link
Copy Markdown
Contributor

No idea, sorry. I‘m more a drive-by user 🤠

Add support for ECMAScript compatible regular expression engine via
regexp2 using functional options e.g.

vm := New(RegExp(regexp2.Creator{}))

This also adds function option support for the existing options:
* Debugger - Deprecates: SetDebuggerHandler
* RandomSource - Deprecates: SetRandomSource.
* Stack - Deprecates: SetStackLimit.
* StackTrace - Deprecates: SetStackTraceLimit.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants