Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-node_copy into lp:zorba
Review: Needs Fixing -- https://code.launchpad.net/~zorba-coders/zorba/feature-node_copy/+merge/97562 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-node_copy into lp:zorba
The function should be marked as a map and as propagating-input-nodes. Also, the must-copy-input-nodes property is not alwayts tree. It is true if the preserve mode is no-preserve. In general, I admit that must-copy-input-nodes is not a good name for this property; it should be called requires-standalone-input-trees (this is a little too long, but probably it's worth adopting in place of the current name). But the property is well documented in functions/function.cpp. -- https://code.launchpad.net/~zorba-coders/zorba/feature-node_copy/+merge/97562 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/phpapi into lp:zorba
William Candillon has proposed merging lp:~zorba-coders/zorba/phpapi into lp:zorba. Requested reviews: Matthias Brantner (matthias-brantner) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/phpapi/+merge/97612 Minor update on the PHP example page. -- https://code.launchpad.net/~zorba-coders/zorba/phpapi/+merge/97612 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'doc/php/manual/indexpage.dox' --- doc/php/manual/indexpage.dox 2010-04-28 14:13:44 + +++ doc/php/manual/indexpage.dox 2012-03-15 10:57:27 + @@ -2,15 +2,11 @@ Zorba ships with a PHP language binding that provides some basic features in order to prepare and execute queries with Zorba. - - For the moment, we provide a a class=en href=examples.htmlset of examples/a that show how this language binding can be used. - - \section note Note for garbage collected languages - Zorba requires that objects must be deleted in a certain order, which Garbage Collectors are not aware of. For that reason, - the Iterators created with XQuery.iterator() must be explicitly destroyed with iterator.destroy() before the XQuery object - is destroyed. - - \example simple.php + + The a href=http://www.zorba-xquery.com/html/entry/2011/12/27/PHP_Meets_XQuery;following tutorial/a provides an overview on how to leverage Zorba into PHP applications. + We also provide the following a class=en href=examples.htmlset of examples/a that show how the Zorba PHP API can be used. + + \example test.php This is a simple example that demonstrate how to use the Zorba XQuery Engine to create, compile, and execute queries. */ -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/phpapi into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/phpapi/+merge/97612 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
Matthias Brantner has proposed merging lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba. Requested reviews: Paul J. Lucas (paul-lucas) Markos Zaharioudakis (markos-za) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 proper return type specification for analyze-string done manually instead of automatically done by the code generation. -- https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'src/functions/func_strings_impl.cpp' --- src/functions/func_strings_impl.cpp 2012-01-11 17:30:25 + +++ src/functions/func_strings_impl.cpp 2012-03-15 15:00:28 + @@ -61,6 +61,44 @@ return NULL; } + +/*** + +/ +void populate_context_strings_impl(static_context* sctx) +{ + xqtref_t lAnalyzeStringResultType = +GENV_TYPESYSTEM.create_node_type( +store::StoreConsts::elementNode, +createQName(http://www.w3.org/2005/xpath-functions,,analyze-string-result;), +NULL, +TypeConstants::QUANT_ONE, +false, +false + ); + + { +DECL_WITH_KIND(sctx, fn_analyze_string_3_0, +(createQName(http://www.w3.org/2005/xpath-functions,,analyze-string;), +GENV_TYPESYSTEM.STRING_TYPE_QUESTION, +GENV_TYPESYSTEM.STRING_TYPE_ONE, +lAnalyzeStringResultType), +FunctionConsts::FN_ANALYZE_STRING_2); + + } + + + { +DECL_WITH_KIND(sctx, fn_analyze_string_3_0, +(createQName(http://www.w3.org/2005/xpath-functions,,analyze-string;), +GENV_TYPESYSTEM.STRING_TYPE_QUESTION, +GENV_TYPESYSTEM.STRING_TYPE_ONE, +GENV_TYPESYSTEM.STRING_TYPE_ONE, +lAnalyzeStringResultType), +FunctionConsts::FN_ANALYZE_STRING_3); + } +} + } /* vim:set et sw=2 ts=2: */ === added file 'src/functions/func_strings_impl.h' --- src/functions/func_strings_impl.h 1970-01-01 00:00:00 + +++ src/functions/func_strings_impl.h 2012-03-15 15:00:28 + @@ -0,0 +1,38 @@ +/* + * Copyright 2006-2012 The FLWOR Foundation. + * + * Licensed under the Apache License, Version 2.0 (the License); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an AS IS BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +#pragma once +#ifndef ZORBA_FUNCTIONS_STRINGS_IMPL_H +#define ZORBA_FUNCTIONS_STRINGS_IMPL_H + + +#include common/shared_types.h +#include functions/function.h +#include functions/function_impl.h +#include functions/function_consts.h + + +namespace zorba { + + +void populate_context_strings_impl(static_context* sctx); + + +} //namespace zorba + + +#endif +/* vim:set et sw=2 ts=2: */ + === modified file 'src/functions/library.cpp' --- src/functions/library.cpp 2012-03-07 15:49:25 + +++ src/functions/library.cpp 2012-03-15 15:00:28 + @@ -57,6 +57,7 @@ #include functions/func_sequences.h #include functions/func_sequences_impl.h #include functions/func_strings.h +#include functions/func_strings_impl.h #include functions/func_uris.h #include functions/func_json.h #include functions/func_var_decl.h @@ -122,6 +123,7 @@ populate_context_schema(sctx); populate_context_sctx(sctx); populate_context_strings(sctx); + populate_context_strings_impl(sctx); populate_context_uris(sctx); populate_context_sequences(sctx); populate_context_sequences_impl(sctx); === modified file 'src/functions/pregenerated/func_strings.cpp' --- src/functions/pregenerated/func_strings.cpp 2012-02-16 12:48:17 + +++ src/functions/pregenerated/func_strings.cpp 2012-03-15 15:00:28 + @@ -853,33 +853,6 @@ { -DECL_WITH_KIND(sctx, fn_analyze_string_3_0, -(createQName(http://www.w3.org/2005/xpath-functions,,analyze-string;), -GENV_TYPESYSTEM.STRING_TYPE_QUESTION, -GENV_TYPESYSTEM.STRING_TYPE_ONE, -GENV_TYPESYSTEM.ELEMENT_TYPE_ONE), -FunctionConsts::FN_ANALYZE_STRING_2); - - } - - - { - - -DECL_WITH_KIND(sctx, fn_analyze_string_3_0, -(createQName(http://www.w3.org/2005/xpath-functions,,analyze-string;), -GENV_TYPESYSTEM.STRING_TYPE_QUESTION, -GENV_TYPESYSTEM.STRING_TYPE_ONE, -GENV_TYPESYSTEM.STRING_TYPE_ONE, -GENV_TYPESYSTEM.ELEMENT_TYPE_ONE), -FunctionConsts::FN_ANALYZE_STRING_3); - - } - - - { - - DECL_WITH_KIND(sctx, fn_zorba_string_materialize,
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 -- https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-analyze_string-2012-03-15T15-01-13.07Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
Why isn't code-generation improved so it *can* be done via code-generation instead? -- https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
Why isn't code-generation improved so it *can* be done via code-generation instead? Automatic code-generation is going to be even more important (and necessary) in light of the forthcoming LLVM stuff. There's going to be even more code that ought to be generated automatically, e.g., C thunks, and lots of boilerplate LLVM C++ API code. -- https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
I have fixed the failing test, which apparently fails only in the Release build. -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 3 Pending. -- https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292 -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 -- https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions. -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292 -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97292 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681 -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
Why isn't code-generation improved so it *can* be done via code-generation instead? There are plenty of cases to specify the user-defined types. Handling each of them automatically (i.e. generate the correct create_* function call to create the type) is not worth it for the 2 functions that we have _in the core_ that require this. I don't think there will be a lot more such functions in the core with user-defined return types. Hence, doing it manually seems to be the right thing to me. -- https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-analyze_string into lp:zorba
Why isn't code-generation improved so it *can* be done via code-generation instead? Automatic code-generation is going to be even more important (and necessary) in light of the forthcoming LLVM stuff. There's going to be even more code that ought to be generated automatically, e.g., C thunks, and lots of boilerplate LLVM C++ API code. I totally agree. However, as far as I understand the issue is orthogonal because this mostly concerns the runtime code and not the code for declaring the functions in the compiler/context. -- https://code.launchpad.net/~zorba-coders/zorba/bug-analyze_string/+merge/97671 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681 -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
It seems the remote queue is stuck again, probably because I have approved it before anyone else. I'll resubmit. -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97681 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_warnings_fedora into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_warnings_fedora into lp:zorba
The PROJECT_NAMES need to be quoted because they contain spaces. That is, Zorba C-API. -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
You have been requested to review the proposed merge of lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Fixed a namespace clash by adding a -schema to the import schema prefix. -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba. === modified file 'modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq' --- modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq 2012-03-09 06:57:41 + +++ modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq 2012-03-15 17:38:21 + @@ -1263,7 +1263,8 @@ for $import in $xqdoc/xqdoc:imports//xqdoc:import[@type = schema] return concat('import schema namespace ', - $namespaces//xqdoc:namespace[@uri = $import/xqdoc:uri/text()][1]/@prefix, + concat($namespaces//xqdoc:namespace[@uri = $import/xqdoc:uri/text()][1]/@prefix, + -schema), ' = ', string($import/xqdoc:uri/text()),';'), ) -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_warnings_fedora into lp:zorba
Works great for me. -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_warnings_fedora into lp:zorba
Works great for me. It still needs one more Approve to be merged :). -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_warnings_fedora into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_warnings_fedora into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix_warnings_fedora into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 -- https://code.launchpad.net/~zorba-coders/zorba/fix_warnings_fedora/+merge/97705 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fn_envvars into lp:zorba
- the unparsed-text-lines function doesn't really stream because it materializes all the lines in a vector before returning them. Instead, the iterator should keep the istream in it's state and return the lines as they are parsed from the stream. Just one question about this, the Item returned by the function for each line should be a Stream then? -- https://code.launchpad.net/~zorba-coders/zorba/fn_envvars/+merge/88143 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 956318] [NEW] Make sure modules (external and internal) are uniform
Public bug reported: We should make sure the modules (and schemas used in them) are named with the SAME pattern. See thread: https://groups.google.com/forum/?hl=enfromgroups#!topic /zorba-dev/S10aasc-tnA ** Affects: zorba Importance: Undecided Status: New -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/956318 Title: Make sure modules (external and internal) are uniform Status in Zorba - The XQuery Processor: New Bug description: We should make sure the modules (and schemas used in them) are named with the SAME pattern. See thread: https://groups.google.com/forum/?hl=enfromgroups#!topic /zorba-dev/S10aasc-tnA To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/956318/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 956347] [NEW] Extend XQXQ module functionality by exposing the prolog
Public bug reported: adding the following two functions to the XQXQ module should allow for an arbitrary static context to be set up as required (e.g. declaring namespaces, enabling the hof option, ...). Hence, it should not be required anymore to make changes to the original query. declare %ann:sequential function prepare-library-module( $library-module-text as xs:string, $prolog as xs:string ) as empty-sequence() external; and declare %ann:sequential function prepare-main-module ( $main-module-text as xs:string, $prolog as xs:string ) as xs:anyURI external; The $prolog parameter can be an arbitrary XQuery prolog that is used to populate the static context of the module to be compiled. To do this, the StaticContext::loadProlog function of the C++ API can be used. ** Affects: zorba Importance: Low Assignee: Sorin Marian Nasoi (sorin.marian.nasoi) Status: New ** Tags: xqxq -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/956347 Title: Extend XQXQ module functionality by exposing the prolog Status in Zorba - The XQuery Processor: New Bug description: adding the following two functions to the XQXQ module should allow for an arbitrary static context to be set up as required (e.g. declaring namespaces, enabling the hof option, ...). Hence, it should not be required anymore to make changes to the original query. declare %ann:sequential function prepare-library-module( $library-module-text as xs:string, $prolog as xs:string ) as empty-sequence() external; and declare %ann:sequential function prepare-main-module ( $main-module-text as xs:string, $prolog as xs:string ) as xs:anyURI external; The $prolog parameter can be an arbitrary XQuery prolog that is used to populate the static context of the module to be compiled. To do this, the StaticContext::loadProlog function of the C++ API can be used. To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/956347/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fn_envvars into lp:zorba
- the unparsed-text-lines function doesn't really stream because it materializes all the lines in a vector before returning them. Instead, the iterator should keep the istream in it's state and return the lines as they are parsed from the stream. Just one question about this, the Item returned by the function for each line should be a Stream then? No, I don't think that's necessary. A line should be relatively short. It would probably be overhead to stream a single line. -- https://code.launchpad.net/~zorba-coders/zorba/fn_envvars/+merge/88143 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Review: Approve The change looks good. One question: is using the same prefix that the original module used likely to cause any conflicts? Are those prefixes generally unique as well as the module prefixes? -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 956460] [NEW] Zorba FOTS driver: assert-serialization bug
Public bug reported: Matthias mentioned that a test which has an obviously wrong expected result (assert-serialization![CDATA[foobar]]/assert-serialization) but isn't shown as failing. ** Affects: zorba Importance: Medium Assignee: Sorin Marian Nasoi (sorin.marian.nasoi) Status: New ** Tags: driver fots zorba -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/956460 Title: Zorba FOTS driver: assert-serialization bug Status in Zorba - The XQuery Processor: New Bug description: Matthias mentioned that a test which has an obviously wrong expected result (assert-serialization![CDATA[foobar]]/assert-serialization) but isn't shown as failing. To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/956460/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
The change looks good. One question: is using the same prefix that the original module used likely to cause any conflicts? Are those prefixes generally unique as well as the module prefixes? I think that that is what Dana meant by suggesting uniformity: as a result I have added bug lp:956318. If you are OK with this answer please set the status of this merge to 'Approve' such that the Validation Queue is able to pick it up. Thanks. -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Sounds reasonable. -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Review: Needs Information Does this also work for schema imports without a prefix? -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/fix_xqdoc_schema_import_prefix-2012-03-15T21-48-35.202Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
The attempt to merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 (message): Validation queue job fix_xqdoc_schema_import_prefix-2012-03-15T21-48-35.202Z is finished. The final status was: 7 tests did not succeed - changes not commited. Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Does this also work for schema imports without a prefix? Are you referring to: import schema default element namespace SOMETHING; ? The answer would be no, the current changes are not tested for this because we have no modules at this point using this kind of schema imports. The xqdoc printer would have to be updated and also the xqdoc2html converter in order to support this. IMHO this should not be a show stopper for these changes: If you agree please create a lp bug for me for this feature and approve this merge. -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Does this also work for schema imports without a prefix? Are you referring to: import schema default element namespace SOMETHING; ? The answer would be no, the current changes are not tested for this because we have no modules at this point using this kind of schema imports. The xqdoc printer would have to be updated and also the xqdoc2html converter in order to support this. IMHO this should not be a show stopper for these changes: If you agree please create a lp bug for me for this feature and approve this merge. Yes, that's what I mean. It looks like the xqdoc visitor could crash with this change. That's why I think the branch shouldn't be merged before it's tested. -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Does this also work for schema imports without a prefix? Are you referring to: import schema default element namespace SOMETHING; ? The answer would be no, the current changes are not tested for this because we have no modules at this point using this kind of schema imports. The xqdoc printer would have to be updated and also the xqdoc2html converter in order to support this. IMHO this should not be a show stopper for these changes: If you agree please create a lp bug for me for this feature and approve this merge. Yes, that's what I mean. It looks like the xqdoc visitor could crash with this change. That's why I think the branch shouldn't be merged before it's tested. Nope, I disagree: this merge does not introduce the problem you mentioned because it was already there :) All this change did was to make you and then me aware that we have a problem if one uses a schema import without a prefix. So my suggestion is: please open a bug for this issue and approve this merge. -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Yes, that's what I mean. It looks like the xqdoc visitor could crash with this change. That's why I think the branch shouldn't be merged before it's tested. Nope, I disagree: this merge does not introduce the problem you mentioned because it was already there :) I agree with Sorin. The problem was there already. I think Sorin should go ahead and commit the change (at least the documentation will be usable..) and then file a bug for the existing problem, and fix that after the release. Thanks Dana -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97701 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
The proposal to merge lp:~nbrinza/zorba/parse-fragment into lp:zorba has been updated. Commit Message changed to: Fully streaming parse-xml module. For more details, see: https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97701 -- https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/97701 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/fix_xqdoc_schema_import_prefix-2012-03-16T02-46-08.694Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
Validation queue job fix_xqdoc_schema_import_prefix-2012-03-16T02-46-08.694Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 -- https://code.launchpad.net/~zorba-coders/zorba/fix_xqdoc_schema_import_prefix/+merge/97708 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp