Fix TypeIdentifier class
Bug #1056787 reported by
Markos Zaharioudakis
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Zorba |
Fix Released
|
Critical
|
Rodolfo Ochoa | ||
Bug Description
This class needs fixing to properly represent the XQuery sequence types. This has to be done in zorba 3.0 because it involves c++ API changes.
Related branches
lp:~zorba-coders/zorba/type-api
- Markos Zaharioudakis: Approve
-
Diff: 48770 lines (+15441/-12337)118 files modifieddoc/java/examples/Test_Zorba.java (+9/-9)
include/zorba/api_shared_types.h (+1/-2)
include/zorba/collection.h (+1/-1)
include/zorba/identtypes.h (+0/-72)
include/zorba/static_context.h (+22/-21)
include/zorba/typeident.h (+189/-131)
modules/http-client/json/http-client.xq.src/http_response_parser.cpp (+1/-0)
src/api/CMakeLists.txt (+1/-2)
src/api/collectionimpl.cpp (+5/-5)
src/api/collectionimpl.h (+1/-1)
src/api/dynamiccontextimpl.cpp (+5/-12)
src/api/identtypesimpl.cpp (+0/-67)
src/api/sequencetype.cpp (+799/-0)
src/api/staticcontextimpl.cpp (+32/-50)
src/api/staticcontextimpl.h (+6/-7)
src/api/typeidentimpl.cpp (+0/-397)
src/api/unmarshaller.cpp (+12/-0)
src/api/unmarshaller.h (+8/-2)
src/compiler/codegen/plan_visitor.cpp (+7/-7)
src/compiler/expression/expr.cpp (+3/-3)
src/compiler/expression/expr_base.cpp (+6/-6)
src/compiler/expression/expr_type.cpp (+32/-32)
src/compiler/expression/flwor_expr.cpp (+5/-5)
src/compiler/parser/jsoniq_parser.cpp (+5612/-4219)
src/compiler/parser/jsoniq_parser.hpp (+46/-29)
src/compiler/parser/jsoniq_scanner.cpp (+446/-443)
src/compiler/parser/parser.y (+39/-39)
src/compiler/parser/xquery_parser.cpp (+5552/-4152)
src/compiler/parser/xquery_parser.hpp (+46/-29)
src/compiler/parser/xquery_scanner.cpp (+440/-449)
src/compiler/parsetree/parsenode_print_xml_visitor.cpp (+1/-1)
src/compiler/parsetree/parsenode_print_xqdoc_visitor.cpp (+1/-1)
src/compiler/parsetree/parsenode_print_xquery_visitor.cpp (+3/-3)
src/compiler/parsetree/parsenode_visitor.h (+1/-1)
src/compiler/parsetree/parsenodes.cpp (+15/-15)
src/compiler/parsetree/parsenodes.h (+52/-52)
src/compiler/rewriter/rules/flwor_rules.cpp (+5/-5)
src/compiler/rewriter/rules/fold_rules.cpp (+6/-6)
src/compiler/rewriter/rules/index_join_rule.cpp (+9/-9)
src/compiler/rewriter/rules/nodeid_rules.cpp (+7/-7)
src/compiler/rewriter/rules/type_rules.cpp (+1/-1)
src/compiler/rewriter/tools/dataflow_annotations.cpp (+5/-5)
src/compiler/translator/translator.cpp (+40/-40)
src/functions/func_accessors_impl.cpp (+2/-2)
src/functions/func_booleans_impl.cpp (+5/-5)
src/functions/func_jsoniq_functions_impl.cpp (+2/-2)
src/functions/func_numerics_impl.cpp (+5/-5)
src/functions/func_parse_fragment_impl.cpp (+0/-1)
src/functions/func_sequences_impl.cpp (+12/-12)
src/functions/func_strings_impl.cpp (+1/-1)
src/functions/function.cpp (+2/-2)
src/functions/signature.cpp (+92/-92)
src/runtime/core/arithmetic_impl.cpp (+2/-2)
src/runtime/core/path_iterators.cpp (+4/-4)
src/runtime/core/sequencetypes.cpp (+22/-22)
src/runtime/core/sequencetypes.h (+12/-10)
src/runtime/indexing/index_ddl.cpp (+1/-1)
src/runtime/json/jsoniq_functions_impl.cpp (+1/-1)
src/runtime/sequences/sequences_impl.cpp (+2/-2)
src/runtime/store/maps_impl.cpp (+2/-2)
src/types/casting.cpp (+15/-15)
src/types/root_typemanager.cpp (+45/-46)
src/types/root_typemanager.h (+11/-11)
src/types/schema/EventSchemaValidator.cpp (+1/-1)
src/types/schema/revalidateUtils.cpp (+1/-1)
src/types/schema/schema.cpp (+5/-5)
src/types/schema/validate.cpp (+9/-12)
src/types/typeconstants.h (+0/-10)
src/types/typeimpl.cpp (+23/-23)
src/types/typeimpl.h (+18/-22)
src/types/typemanager.h (+17/-17)
src/types/typemanagerimpl.cpp (+156/-309)
src/types/typemanagerimpl.h (+15/-17)
src/types/typeops.cpp (+55/-273)
src/types/typeops.h (+15/-24)
swig/Collection.h (+2/-2)
swig/Collection.i (+2/-2)
swig/SequenceType.h (+183/-0)
swig/SequenceType.i (+370/-0)
swig/StaticContext.h (+6/-5)
swig/StaticContext.i (+10/-10)
swig/TypeIdentifier.h (+0/-102)
swig/TypeIdentifier.i (+0/-197)
swig/csharp/CMakeLists.txt (+3/-3)
swig/java/CMakeLists.txt (+8/-8)
swig/java/ZorbaInputWrapper.java (+2/-2)
swig/java/ZorbaOutputWrapper.java (+1/-1)
swig/java/ZorbaReaderWrapper.java (+2/-2)
swig/java/ZorbaWriterWrapper.java (+2/-2)
swig/php/CMakeLists.txt (+3/-3)
swig/python/CMakeLists.txt (+5/-3)
swig/ruby/CMakeLists.txt (+5/-5)
swig/xqj/ZorbaXQCollection.java (+12/-12)
swig/xqj/ZorbaXQCollectionManager.java (+7/-7)
swig/xqj/ZorbaXQConnection.java (+193/-188)
swig/xqj/ZorbaXQDataSource.java (+2/-2)
swig/xqj/ZorbaXQDocumentManager.java (+4/-4)
swig/xqj/ZorbaXQExpression.java (+27/-27)
swig/xqj/ZorbaXQItem.java (+5/-5)
swig/xqj/ZorbaXQItemType.java (+317/-237)
swig/xqj/ZorbaXQMetaData.java (+3/-3)
swig/xqj/ZorbaXQPreparedExpression.java (+36/-36)
swig/xqj/ZorbaXQResultItem.java (+2/-2)
swig/xqj/ZorbaXQResultSequence.java (+6/-6)
swig/xqj/ZorbaXQResultSequenceScrollable.java (+6/-6)
swig/xqj/ZorbaXQSequence.java (+14/-14)
swig/xqj/ZorbaXQSequenceType.java (+1/-1)
swig/xqj/ZorbaXQStaticCollectionManager.java (+7/-7)
swig/xqj/ZorbaXQStaticContext.java (+12/-12)
swig/xqj/ZorbaXQXmlDataManager.java (+4/-4)
swig/xqj/managers/Api_test.java (+1/-1)
swig/xqj/managers/Tests.java (+12/-12)
swig/xqj/tck/xqj_test.bat.in (+11/-7)
swig/xqj/tck/xqj_test.sh.in (+6/-6)
swig/xqj/tck/zorba.properties (+1/-1)
swig/zorba_api.i (+2/-2)
test/unit/context_item.cpp (+13/-13)
test/unit/staticcollectionmanager.cpp (+108/-85)
Revision history for this message
| Chris Hillery (ceejatec) wrote | #1 |
| tags: | added: incompatible-change |
| Changed in zorba: | |
| importance: | Medium → Critical |
| Changed in zorba: | |
| assignee: | Markos Zaharioudakis (markos-za) → Rodolfo Ochoa (rodolfo-ochoa) |
| Changed in zorba: | |
| status: | New → Fix Committed |
| Changed in zorba: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
I think this bug needs some more information. What changes are required? For what reasons?
Matthias has suggested that this class should be renamed; is that also true? What should it be called?