Skip to content

Make PROCESSING_INSTRUCTION regexp lazy#452

Closed
kddnewton wants to merge 1 commit intoSAP:masterfrom
kddnewton:patch-1
Closed

Make PROCESSING_INSTRUCTION regexp lazy#452
kddnewton wants to merge 1 commit intoSAP:masterfrom
kddnewton:patch-1

Conversation

@kddnewton
Copy link
Copy Markdown
Contributor

@kddnewton kddnewton commented Feb 10, 2024

Otherwise this fails with input like

<list><value id="all"><label>All</label></value><value id="one-one"><?group one?><label>One</label></value><value id="one-two"><?group one?><label>Two</label></value><value id="two-one"><?group two?><label>One</label></value><value id="two-two"><?group two?><label>Two</label></value></list>

where it will go to the end of the line to find the ?>

@cla-assistant
Copy link
Copy Markdown

cla-assistant bot commented Feb 10, 2024

CLA assistant check
All committers have signed the CLA.

@kddnewton
Copy link
Copy Markdown
Contributor Author

@bd82 any chance you can take a look?

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.

1 participant