Skip to content

Drop old catch_var from the grammar file - #1800

Draft
CaiJimmy wants to merge 3 commits into
masterfrom
jimmy/drop-catch-var-grammar
Draft

Drop old catch_var from the grammar file#1800
CaiJimmy wants to merge 3 commits into
masterfrom
jimmy/drop-catch-var-grammar

Conversation

@CaiJimmy

@CaiJimmy CaiJimmy commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

This is not part of W3C XQuery standard, and we actually don't use the variable in the translation visitor, so it's dead.

I expect a lot of old test cases updates because some of them did use this construction. (Well no test was broken because of this

Now the grammar has been aligned to the official standard:

image

@github-actions

Copy link
Copy Markdown

QT3 Test Results - Parser: jsoniq

Regression baseline: run 31613200034

🎉 No regressions found for this build.

Summary of passed tests for jsoniq
Test Suite Passing Failing Errors Skipped Total
app 1984 39 127 7 2157
array 207 0 1 1 209
fn 10024 381 232 498 11135
map 220 0 0 0 220
math 149 0 0 0 149
misc 700 77 34 60 871
op 4192 63 70 41 4366
prod 10344 569 465 829 12207
ser 321 21 1 0 343
xs 147 6 0 11 164
Total 28288 1156 930 1447 31821

Full analysis report: see analysis-jsoniq in artifacts.

Download detailed test results

@github-actions

Copy link
Copy Markdown

QT3 Test Results - Parser: xquery

Regression baseline: run 31613200034

🎉 No regressions found for this build.

Summary of passed tests for xquery
Test Suite Passing Failing Errors Skipped Total
app 2024 38 88 7 2157
array 207 0 1 1 209
fn 10078 343 216 498 11135
map 220 0 0 0 220
math 149 0 0 0 149
misc 719 64 28 60 871
op 4284 14 27 41 4366
prod 10571 442 365 829 12207
ser 343 0 0 0 343
xs 153 0 0 11 164
Total 28748 901 725 1447 31821

Full analysis report: see analysis-xquery in artifacts.

Download detailed test results

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