Eliminate parse-xml#2
Bug #950621 reported by
Sorin Marian Nasoi
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Zorba |
Fix Released
|
High
|
Juan Zacarias | ||
Bug Description
ctest -R xquery_3_0
on the branch
lp:~zorba-coders/zorba/fn_envvars
shows that:
- the following functions are in the fn namespace and according to the F&O 3.0 spec they should not be there:
parse-xml#2
Related branches
lp:~zorba-coders/zorba/bug-950621
- Juan Zacarias: Approve
- Chris Hillery: Approve
-
Diff: 884 lines (+4/-795)11 files modifiedChangeLog (+1/-0)
NOTICE.txt (+0/-740)
src/functions/pregenerated/func_parsing_and_serializing.cpp (+0/-13)
src/functions/pregenerated/function_enum.h (+0/-1)
src/runtime/parsing_and_serializing/parsing_and_serializing_impl.cpp (+3/-28)
src/runtime/spec/parsing_and_serializing/parsing_and_serializing.xml (+0/-6)
test/rbkt/Queries/zorba/parsing_and_serializing/fn_parse-xml_14.xq (+0/-1)
test/rbkt/Queries/zorba/parsing_and_serializing/fn_parse-xml_15.spec (+0/-1)
test/rbkt/Queries/zorba/parsing_and_serializing/fn_parse-xml_15.xq (+0/-2)
test/rbkt/Queries/zorba/parsing_and_serializing/fn_parse-xml_16.spec (+0/-1)
test/rbkt/Queries/zorba/parsing_and_serializing/fn_parse-xml_16.xq (+0/-2)
| Changed in zorba: | |
| importance: | Undecided → Critical |
| Changed in zorba: | |
| assignee: | nobody → Juan Zacarias (juan457) |
| Changed in zorba: | |
| milestone: | none → 2.7 |
Revision history for this message
| Chris Hillery (ceejatec) wrote | #1 |
| summary: |
- XQuery 3.0 vs Zorba: parse-xml#2 should not be in FN namespace + Eliminate parse-xml#2 |
| Changed in zorba: | |
| importance: | Critical → High |
| Changed in zorba: | |
| status: | New → Fix Committed |
| Changed in zorba: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Per Matthias, this function should not even exist as its functionality is available via xml:parse(). So the entry in the spec .xml file should be removed, and the C++ implementation can be simplified as it will only need to deal with the single-argument fn:parse-xml() function now.