Skip to content

Add recursive descent XQuery parser (opt-in via feature flag) #6220

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

Draft
joewiz wants to merge 4 commits into
base: develop
Choose a base branch
from

[test] Remove duplicate orderby-empty-ordering-spec tests

245adae
Select commit
Loading
Failed to load commit list.
Draft

Add recursive descent XQuery parser (opt-in via feature flag) #6220

[test] Remove duplicate orderby-empty-ordering-spec tests
245adae
Select commit
Loading
Failed to load commit list.
Codacy Production / Codacy Static Code Analysis required action Apr 14, 2026 in 0s

90 new issues (0 max.) of at least severity.

Codacy Here is an overview of what got changed by this pull request:

Issues
======
- Added 90
           

Complexity increasing per file
==============================
- exist-core/src/test/java/org/exist/xquery/parser/next/XQueryParserTest.java  240
- exist-core/src/test/java/org/exist/xquery/parser/next/NativeParserIntegrationTest.java  47
- exist-core/src/test/java/org/exist/xquery/parser/next/ParserBenchmark.java  4
- exist-core/src/main/java/org/exist/xquery/parser/next/Token.java  11
- exist-core/src/main/java/org/exist/xquery/parser/next/XQUFExpressions.java  32
- taskings/grammar-dispatch-audit.py  70
- exist-core/src/test/java/org/exist/xquery/parser/next/XQueryLexerTest.java  62
- exist-core/src/main/java/org/exist/xquery/parser/next/FTExpressions.java  103
- exist-core/src/main/java/org/exist/xquery/parser/next/ParseError.java  14
- exist-core/src/main/java/org/exist/xquery/parser/next/XQueryLexer.java  275
- exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java  15
- exist-core/src/test/java/org/exist/xquery/parser/next/LexerBenchmark.java  9
- exist-core/src/main/java/org/exist/xquery/parser/next/XQ4Expressions.java  31
- exist-core/src/main/java/org/exist/xquery/XQueryContext.java  25
         

Clones added
============
- exist-core/src/test/java/org/exist/xquery/FunctionTypeInElementContentTest.java  10
- exist-core/src/test/java/org/exist/xquery/parser/next/NativeParserIntegrationTest.java  1
- exist-core/src/test/java/org/exist/xquery/parser/next/ParserBenchmark.java  2
- exist-core/src/main/java/org/exist/xquery/parser/next/FTExpressions.java  6

See the complete overview on Codacy

Annotations

Check notice on line 203 in exist-core/src/main/java/org/exist/xquery/XQuery.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/XQuery.java#L203 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 2799 in exist-core/src/main/java/org/exist/xquery/XQueryContext.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/XQueryContext.java#L2799 

Unnecessary use of fully qualified name 'org.exist.xquery.parser.next.XQueryParser' due to existing same package import 'org.exist.xquery.*'

Check notice on line 2800 in exist-core/src/main/java/org/exist/xquery/XQueryContext.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/XQueryContext.java#L2800 

Unnecessary use of fully qualified name 'org.exist.xquery.parser.next.XQueryParser' due to existing same package import 'org.exist.xquery.*'

Check notice on line 2818 in exist-core/src/main/java/org/exist/xquery/XQueryContext.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/XQueryContext.java#L2818 

Unnecessary use of fully qualified name 'java.util.Iterator' due to existing import 'java.util.*'

Check warning on line 39 in exist-core/src/main/java/org/exist/xquery/parser/next/FTExpressions.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/FTExpressions.java#L39 

Class cannot be instantiated and does not provide any static methods or fields

Check notice on line 78 in exist-core/src/main/java/org/exist/xquery/parser/next/FTExpressions.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/FTExpressions.java#L78 

Document empty method body

Check notice on line 50 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L50 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 51 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L51 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 52 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L52 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 53 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L53 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 54 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L54 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 55 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L55 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 56 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L56 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 57 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L57 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 58 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L58 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 59 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L59 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 60 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L60 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 61 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L61 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 62 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L62 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 63 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L63 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 64 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L64 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 65 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L65 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 66 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L66 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 67 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L67 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.

Check notice on line 68 in exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

exist-core/src/main/java/org/exist/xquery/parser/next/Keywords.java#L68 

Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.
View more details on Codacy Production