Confusing error message
Bug #867227 reported by
William Candillon
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Zorba |
Fix Released
|
Medium
|
Nicolae Brinza |
Bug Description
The following query:
for $test in (1 to 10)
return if(true()) then {
true();
1
} else {
}
true()
outputs:
:9,1: static error [err:XPST0003]: invalid expression: syntax error, unexpected "QName", expecting "end of file";
However this message is not explicit about what's wrong in the query above.
Related branches
lp:~nbrinza/zorba/error-messages
- Cezar Andrei: Approve
- Nicolae Brinza: Approve
-
Diff: 27986 lines (+3970/-3837)13 files modifiedsrc/compiler/parser/jsoniq_parser.cpp (+1545/-1531)
src/compiler/parser/jsoniq_parser.hpp (+6/-6)
src/compiler/parser/jsoniq_scanner.cpp (+371/-385)
src/compiler/parser/parser.y (+31/-10)
src/compiler/parser/xquery_parser.cpp (+1533/-1519)
src/compiler/parser/xquery_parser.hpp (+6/-6)
src/compiler/parser/xquery_scanner.cpp (+366/-380)
test/rbkt/ExpQueryResults/zorba/parser/syntax-errors/missing-semicolon-06.xml.res (+1/-0)
test/rbkt/ExpQueryResults/zorba/parser/syntax-errors/missing-semicolon-07.xml.res (+1/-0)
test/rbkt/ExpQueryResults/zorba/parser/syntax-errors/missing-semicolon-08.xml.res (+1/-0)
test/rbkt/Queries/zorba/parser/syntax-errors/missing-semicolon-06.xq (+34/-0)
test/rbkt/Queries/zorba/parser/syntax-errors/missing-semicolon-07.xq (+34/-0)
test/rbkt/Queries/zorba/parser/syntax-errors/missing-semicolon-08.xq (+41/-0)
Changed in zorba: | |
milestone: | none → 2.7 |
Changed in zorba: | |
status: | New → Fix Committed |
Changed in zorba: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Nicolae, do you think you can hack the parser to produce a better (more meaningful) error message for such cases?