Merge lp:~zorba-coders/zorba/fix_bug_987830 into lp:zorba
Proposed by
Sorin Marian Nasoi
Status: | Merged |
---|---|
Approved by: | Sorin Marian Nasoi |
Approved revision: | 10799 |
Merged at revision: | 10809 |
Proposed branch: | lp:~zorba-coders/zorba/fix_bug_987830 |
Merge into: | lp:zorba |
Diff against target: |
138 lines (+22/-11) 7 files modified
modules/ExternalModules.conf (+2/-2) modules/com/zorba-xquery/www/modules/debugger/dbgp-message-handler.xq (+1/-1) modules/com/zorba-xquery/www/modules/project_xqdoc.xq (+2/-2) modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq (+2/-2) modules/org/expath/ns/file.xq (+1/-1) src/compiler/translator/translator.cpp (+1/-1) test/rbkt/Queries/zorba/xqdoc/moduleDocumentation.xq (+13/-2) |
To merge this branch: | bzr merge lp:~zorba-coders/zorba/fix_bug_987830 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sorin Marian Nasoi | Approve | ||
Markos Zaharioudakis | Approve | ||
Review via email: mp+103370@code.launchpad.net |
Commit message
fix for lp:987830.
Description of the change
fix for lp:987830.
To post a comment you must log in.
"empty" and "none" are not valid sequence types. We should try that are modules are at least parsable by other engines as well, so we should use valid sequence types.; For "empty" use empty-sequence(). There is not a valid sequence type for "none", but I did not see it used anywhere. Do you plan to use it? where?
This also means that the code in translator.cpp should be removed.