Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Grammar productions missing spec conditionality #1574

Open
johnlumley opened this issue Nov 13, 2024 · 1 comment · May be fixed by #1576
Open

Grammar productions missing spec conditionality #1574

johnlumley opened this issue Nov 13, 2024 · 1 comment · May be fixed by #1576
Labels
Editorial Minor typos, wording clarifications, example fixes, etc. PR Pending A PR has been raised to resolve this issue XPath An issue related to XPath

Comments

@johnlumley
Copy link

The XPath grammar currently projects the following productions, which appear to be only relevant to XQuery, are not reachable from the XPath root, and either lack an if="xquery40" condition, or wrongly have xpath40 in the condition:

ExtendedFieldDeclaration, ParamListWithDefaults,ParamWithDefault

In fact, the grammar doesn't allow default values for arguments of inline functions. Is this the case?
@michaelhkay
Copy link
Contributor

the grammar doesn't allow default values for arguments of inline functions
Correct. Issue #1175 proposes the feature, but it's not currently in the spec.

@michaelhkay michaelhkay linked a pull request Nov 14, 2024 that will close this issue
Open
@michaelhkay michaelhkay added XPath An issue related to XPath Editorial Minor typos, wording clarifications, example fixes, etc. PR Pending A PR has been raised to resolve this issue labels Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Editorial Minor typos, wording clarifications, example fixes, etc. PR Pending A PR has been raised to resolve this issue XPath An issue related to XPath
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

1574 Mark some productions as XQuery only michaelhkay/qtspecs
2 participants
@michaelhkay @johnlumley