Skip to content

fixed persisting infinite loop on $"#18

Open
clockworked247 wants to merge 1 commit into
mbrevoort:masterfrom
clockworked247:master
Open

fixed persisting infinite loop on $"#18
clockworked247 wants to merge 1 commit into
mbrevoort:masterfrom
clockworked247:master

Conversation

@clockworked247
Copy link
Copy Markdown

Tested with the following XQuery strings:

$"

"^/([\w\d_-])/?(update|create|delete).?(\w)/?$"

let $pattern := fn:concat("^$")

"\uxxxxx"

fn:analyze-string("http://nssdc.gsfc.nasa.gov/ontologies/spacecraft#launchDate"\i\c*$")/s:match/fn:string()

matches($path, "^/[^/?#]+/documents/?$")
Try to change "$path" to a new string with double-quotes. Chrome freaks
out, rendering the UI frozen and eventually bringing up the Kill

interface.

Tested with the following XQuery strings:

$"
---
"^/([\w\d_\-]*)/?(update|create|delete)\.?(\w*)/?$"
---
let $pattern := fn:concat("^$")
---
"\uxxxxx"
---

fn:analyze-string("http://nssdc.gsfc.nasa.gov/ontologies/spacecraft#launchDate"\i\c*$")/s:match/fn:string()
---
matches($path, "^/[^/?#]+/documents/?$")
Try to change "$path" to a new string with double-quotes. Chrome freaks
out, rendering the UI frozen and eventually bringing up the Kill
interface.
---
@clockworked247
Copy link
Copy Markdown
Author

Hey Mike,

I went ahead and tried my hand at fixing the persisting $" issue with the XQuery mode parser. The change appears to fix the described uses of that string.

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