Skip to content

regex.pcre: small fixes#27341

Merged
medvednikov merged 1 commit into
vlang:masterfrom
JalonSolov:fix_pcre
Jun 5, 2026
Merged

regex.pcre: small fixes#27341
medvednikov merged 1 commit into
vlang:masterfrom
JalonSolov:fix_pcre

Conversation

@JalonSolov
Copy link
Copy Markdown
Collaborator

  • implements \xHH and \XHHHH handling (as already documented).

  • small README change to document using max_stack_depth instead of change_stack_depth, with an example.

  • ensure no duplicate named groups

  • ensure no mid-rune problems in UTF-8 text

  • tests added

@JalonSolov JalonSolov requested a review from penguindark June 4, 2026 17:08
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: db8614dc09

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread vlib/regex/pcre/regex.v
Copy link
Copy Markdown
Member

@penguindark penguindark left a comment

Choose a reason for hiding this comment

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

Good for me, tnx

@medvednikov medvednikov merged commit 2f74099 into vlang:master Jun 5, 2026
76 of 83 checks passed
@JalonSolov JalonSolov deleted the fix_pcre branch June 5, 2026 17:57
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.

3 participants