[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-05-09 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-05-09 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-05-09 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-05-09 Thread William Candillon
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-05-08 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-05-08 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba.

Commit message:
Fixed JSON-to-XML bug.

Requested reviews:
  Paul J. Lucas (paul-lucas)

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862

Fixed JSON-to-XML bug.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'modules/json/json-csv.jq'
--- modules/json/json-csv.jq	2014-03-05 23:36:00 +
+++ modules/json/json-csv.jq	2014-05-08 18:38:36 +
@@ -141,7 +141,6 @@
  : character.
  : @error csv:INVALID_CSV_VALUE if a value of some key is not castable to
  : string.
- : field-names option is not a string.
  : @error csv:MISSING_VALUE if a missing value is detected and the
  : missing-value option is "error".
  : @error csv:EXTRA_VALUE if an extra value is detected and the

=== modified file 'src/runtime/json/snelson.cpp'
--- src/runtime/json/snelson.cpp	2014-03-20 01:59:46 +
+++ src/runtime/json/snelson.cpp	2014-05-08 18:38:36 +
@@ -176,10 +176,9 @@
   POP_ITERATOR();
   POP_STATE();
   POP_ITEM_ELEMENT();
-  if ( IN_STATE( in_object ) ) {
+  if ( IN_STATE( in_object ) )
 POP_ITEM( xml );
-POP_ITEM( json );
-  }
+  POP_ITEM( json );
   continue;
 }
 if ( IN_STATE( in_object ) ) {
@@ -198,7 +197,6 @@
   case store::JS_NULL:
 ADD_TYPE_ATTRIBUTE( "null" );
 ADD_ITEM_ELEMENT( "null" );
-POP_ITEM_ELEMENT();
 break;
 
   case store::XS_BOOLEAN:
@@ -206,7 +204,6 @@
 ADD_ITEM_ELEMENT( "boolean" );
 value_str = value_item->getBooleanValue() ? "true" : "false";
 GENV_ITEMFACTORY->createTextNode( junk_item, xml_item, value_str );
-POP_ITEM_ELEMENT();
 break;
 
   case store::XS_BYTE:
@@ -229,7 +226,6 @@
 ADD_ITEM_ELEMENT( "number" );
 value_str = value_item->getStringValue();
 GENV_ITEMFACTORY->createTextNode( junk_item, xml_item, value_str );
-POP_ITEM_ELEMENT();
 break;
 
   default:
@@ -237,8 +233,8 @@
 ADD_ITEM_ELEMENT( "string" );
 value_str = value_item->getStringValue();
 GENV_ITEMFACTORY->createTextNode( junk_item, xml_item, value_str );
-POP_ITEM_ELEMENT();
 } // switch
+POP_ITEM_ELEMENT();
 break;
 
   case store::Item::ARRAY:

=== added file 'test/rbkt/ExpQueryResults/zorba/json/json-snelson-j2x-object-07.xml.res'
--- test/rbkt/ExpQueryResults/zorba/json/json-snelson-j2x-object-07.xml.res	1970-01-01 00:00:00 +
+++ test/rbkt/ExpQueryResults/zorba/json/json-snelson-j2x-object-07.xml.res	2014-05-08 18:38:36 +
@@ -0,0 +1,10 @@
+http://john.snelson.org.uk/parsing-json-into-xquery"; type="object">
+  
+
+  
+
+  
+
+  
+  
+

=== added file 'test/rbkt/Queries/zorba/json/json-snelson-j2x-object-07.xq'
--- test/rbkt/Queries/zorba/json/json-snelson-j2x-object-07.xq	1970-01-01 00:00:00 +
+++ test/rbkt/Queries/zorba/json/json-snelson-j2x-object-07.xq	2014-05-08 18:38:36 +
@@ -0,0 +1,12 @@
+import module namespace jx = "http://zorba.io/modules/json-xml";;
+
+let $json :=
+  {
+"a" : {
+  "b" : [ { "c" : null } ]
+},
+"d" : null
+  }
+return jx:json-to-xml( $json )
+
+(: vim:set et sw=2 ts=2: :)

-- 
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-05-08 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-05-08 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/218862
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-22 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-22 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-22 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-22 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-22 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-22 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-22 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba.

Commit message:
Added missing #include  all over.

Requested reviews:
  Paul J. Lucas (paul-lucas)

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803

Added missing #include  all over.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/216803
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'src/api/item_seq_chainer.h'
--- src/api/item_seq_chainer.h	2013-08-01 11:59:51 +
+++ src/api/item_seq_chainer.h	2014-04-23 00:41:51 +
@@ -19,7 +19,7 @@
 
 #include 
 #include 
-#include 
+#include 
 
 #include 
 #include 

=== modified file 'src/runtime/base/narybase.h'
--- src/runtime/base/narybase.h	2014-02-12 02:25:45 +
+++ src/runtime/base/narybase.h	2014-04-23 00:41:51 +
@@ -17,6 +17,7 @@
 #ifndef ZORBA_RUNTIME_NARY_ITERATOR
 #define ZORBA_RUNTIME_NARY_ITERATOR
 
+#include 
 #include 
 
 #include "common/shared_types.h"

=== modified file 'src/runtime/base/unarybase.h'
--- src/runtime/base/unarybase.h	2014-02-12 02:25:45 +
+++ src/runtime/base/unarybase.h	2014-04-23 00:41:51 +
@@ -17,6 +17,8 @@
 #ifndef ZORBA_RUNTIME_UNARY_ITERATOR
 #define ZORBA_RUNTIME_UNARY_ITERATOR
 
+#include 
+
 #include "common/shared_types.h"
 
 #include "runtime/base/plan_iterator.h"

=== modified file 'src/runtime/util/handle_hashmap_item_value.h'
--- src/runtime/util/handle_hashmap_item_value.h	2013-02-07 17:24:36 +
+++ src/runtime/util/handle_hashmap_item_value.h	2014-04-23 00:41:51 +
@@ -16,6 +16,8 @@
 #ifndef ZORBA_STORE_HANDLE_HAHSET_ITEM_VALUE_H
 #define ZORBA_STORE_HANDLE_HAHSET_ITEM_VALUE_H
 
+#include 
+
 #include "zorbautils/hashmap.h"
 #include "runtime/booleans/BooleanImpl.h"
 #include "runtime/api/runtimecb.h"

=== modified file 'src/store/naive/atomic_items.h'
--- src/store/naive/atomic_items.h	2013-06-26 10:59:37 +
+++ src/store/naive/atomic_items.h	2014-04-23 00:41:51 +
@@ -17,11 +17,12 @@
 #ifndef ZORBA_STORE_ATOMIC_ITEMS_H
 #define ZORBA_STORE_ATOMIC_ITEMS_H
 
-#include 
+#include 
 #include 
 #include 
 #include 
 
+#include 
 #include 
 #ifndef ZORBA_NO_FULL_TEXT
 #include 

=== modified file 'src/store/naive/node_items.h'
--- src/store/naive/node_items.h	2013-06-29 08:38:53 +
+++ src/store/naive/node_items.h	2014-04-23 00:41:51 +
@@ -17,6 +17,7 @@
 #ifndef ZORBA_SIMPLE_STORE_NODE_ITEMS
 #define ZORBA_SIMPLE_STORE_NODE_ITEMS
 
+#include 
 #include 
 #include 
 

=== modified file 'src/store/naive/node_iterators.h'
--- src/store/naive/node_iterators.h	2013-06-15 02:57:08 +
+++ src/store/naive/node_iterators.h	2014-04-23 00:41:51 +
@@ -16,6 +16,7 @@
 #ifndef ZORBA_SIMPLE_STORE_NODE_ITERATORS
 #define ZORBA_SIMPLE_STORE_NODE_ITERATORS
 
+#include 
 #include 
 #include 
 

=== modified file 'src/store/naive/ordpath.h'
--- src/store/naive/ordpath.h	2013-02-26 04:12:43 +
+++ src/store/naive/ordpath.h	2014-04-23 00:41:51 +
@@ -17,6 +17,7 @@
 #ifndef ZORBA_STORE_ORDPATH_H
 #define ZORBA_STORE_ORDPATH_H
 
+#include 
 #include 
 
 #include 

=== modified file 'src/store/naive/simple_index_general.h'
--- src/store/naive/simple_index_general.h	2013-02-07 17:24:36 +
+++ src/store/naive/simple_index_general.h	2014-04-23 00:41:51 +
@@ -16,6 +16,7 @@
 #ifndef ZORBA_SIMPLE_STORE_INDEX_HASH_GENERAL
 #define ZORBA_SIMPLE_STORE_INDEX_HASH_GENERAL
 
+#include 
 #include 
 
 #include "simple_index.h"

=== modified file 'src/store/naive/text_node_content.h'
--- src/store/naive/text_node_content.h	2013-02-07 17:24:36 +
+++ src/store/naive/text_node_content.h	2014-04-23 00:41:51 +
@@ -17,6 +17,7 @@
 #ifndef ZORBA_SIMPLE_STORE_TEXT_NODE_CONTENT
 #define ZORBA_SIMPLE_STORE_TEXT_NODE_CONTENT
 
+#include 
 #include   /* for memset(3) */
 
 #include "shared_types.h"

=== modified file 'src/system/globalenv.h'
--- src/system/globalenv.h	2013-12-25 03:43:06 +
+++ src/system/globalenv.h	2014-04-23 00:41:51 +
@@ -17,6 +17,8 @@
 #ifndef ZORBA_GLOBALENV_H
 #define ZORBA_GLOBALENV_H
 
+#include 
+
 #include 
 #include 
 

=== modified file 'src/types/typeimpl.h'
--- src/types/typeimpl.h	2013-09-23 09:11:02 +
+++ src/types/typeimpl.h	2014-04-23 00:41:51 +
@@ -17,6 +17,8 @@
 #ifndef ZORBA_TYPEIMPL_H
 #define ZORBA_TYPEIMPL_H
 
+#include 
+
 #include "common/shared_types.h"
 
 #include "types/node_test.h"

=== modified file 'src/util/string/default_rep.h'
--- src/util/string/default_rep.h	2013-08-02 18:56:15 +
+++ src/util/string/default_rep.h	2014-04-23 00:41:51 +
@@ -17,6 +17,7 @@
 #ifndef ZORBA_RSTRING_DEFAULT_REP_H
 #define ZORBA_RSTRING_DEFAULT_REP_H
 
+#include 
 #include 
 
 #include 

=== modified file 'src/util/utf8_util.h'
--- src/util/utf8_util.h	2013-06-01 00:30:39 +
+++ src/util/utf8_util.h	2014-04-23 00:41:51 +
@@ -18,6 +18,7 @@
 #define ZORBA_UTF8_UTIL_H
 
 #include 
+#include 
 #include 
 #include 
 

=== modified file 'src/zorbaserialization/archiver.h

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-14 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-14 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-14 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-14 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-14 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-14 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-14 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba.

Commit message:
Fixed CSV quote-parsing bug.

Requested reviews:
  Paul J. Lucas (paul-lucas)

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776

Fixed CSV quote-parsing bug.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/215776
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'src/util/csv_parser.cpp'
--- src/util/csv_parser.cpp	2014-03-06 19:06:43 +
+++ src/util/csv_parser.cpp	2014-04-14 23:00:32 +
@@ -20,10 +20,22 @@
 // local
 #include "csv_parser.h"
 
+using namespace std;
+
 namespace zorba {
 
 ///
 
+inline bool peek( istream &is, char &c ) {
+  char const temp = is.peek();
+  bool const peeked = is.good();
+  if ( peeked )
+c = temp;
+  return peeked;
+}
+
+///
+
 bool csv_parser::next_value( zstring *value, bool *eol, bool *quoted ) const {
   ztd::string_appender appender( value );
   char c;
@@ -35,7 +47,9 @@
   while ( is_->get( c ) ) {
 if ( in_quote ) {
   if ( quote_esc_ == quote_ ) { // ""
-if ( c == quote_ && (c = is_->peek(), is_->good()) ) {
+if ( c == quote_ ) {
+  if ( !peek( *is_, c ) )
+break;
   if ( c != quote_ ) {
 in_quote = false;
 continue;
@@ -61,7 +75,7 @@
   }
   switch ( c ) {
 case '\r':
-  if ( ((c = is_->peek()), is_->good()) && c == '\n' )
+  if ( peek( *is_, c ) && c == '\n' )
 is_->get();
   // no break;
 case '\n':

=== added file 'test/rbkt/ExpQueryResults/zorba/csv/csv-parse-02.xml.res'
--- test/rbkt/ExpQueryResults/zorba/csv/csv-parse-02.xml.res	1970-01-01 00:00:00 +
+++ test/rbkt/ExpQueryResults/zorba/csv/csv-parse-02.xml.res	2014-04-14 23:00:32 +
@@ -0,0 +1,4 @@
+{
+  "key_1" : "value_1", 
+  "key_2" : "value_2"
+}

=== added file 'test/rbkt/Queries/zorba/csv/csv-parse-02.jq'
--- test/rbkt/Queries/zorba/csv/csv-parse-02.jq	1970-01-01 00:00:00 +
+++ test/rbkt/Queries/zorba/csv/csv-parse-02.jq	2014-04-14 23:00:32 +
@@ -0,0 +1,9 @@
+xquery version "1.0";
+
+import module namespace csv = "http://zorba.io/modules/json-csv";;
+
+let $csvString2 := """key_1"",""key_2""
+""value_1"",""value_2"""
+return csv:parse( $csvString2 )
+
+(: vim:set et sw=2 ts=2: :)

=== added file 'test/rbkt/Queries/zorba/csv/csv-parse-02.spec'
--- test/rbkt/Queries/zorba/csv/csv-parse-02.spec	1970-01-01 00:00:00 +
+++ test/rbkt/Queries/zorba/csv/csv-parse-02.spec	2014-04-14 23:00:32 +
@@ -0,0 +1,1 @@
+Serialization: indent=yes

-- 
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-02 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-02 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-02 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-02 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-02 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-02 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba.

Commit message:
Minor performance tweaks.
Fixed help message.
Fixed comment typo.

Requested reviews:
  Paul J. Lucas (paul-lucas)

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942

Minor performance tweaks.
Fixed help message.
Fixed comment typo.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'bin/zorbacmd_args.cpp'
--- bin/zorbacmd_args.cpp	2014-03-05 23:19:14 +
+++ bin/zorbacmd_args.cpp	2014-04-02 22:46:08 +
@@ -139,7 +139,7 @@
 HELP_OPT( "--execute-plan" )
   "Do not compile the query; instead load the execution plan from the file specified by the -f -q options (or by any file specified without any other argument), and execute the loaded plan.\n\n"
 
-HELP_OPT( "--external-variable, -e {=file|:=}" )
+HELP_OPT( "--external-variable, -e {=|:=}" )
   "Set the value for an externsl variable.\n\n"
 
 // f //

=== modified file 'src/runtime/base/plan_iterator.h'
--- src/runtime/base/plan_iterator.h	2014-02-27 18:36:37 +
+++ src/runtime/base/plan_iterator.h	2014-04-02 22:46:08 +
@@ -491,7 +491,7 @@
 if ( planState.profile_ ) {
   //
   // Temporaries are used here to guarantee the order in which the timers
-  // are stopped.  (If the expressions were passed as functio arguments,
+  // are stopped.  (If the expressions were passed as function arguments,
   // the order is platform/compiler-dependent.)
   //
   time::msec_type const ce( c.elapsed() );

=== modified file 'src/store/naive/json_items.cpp'
--- src/store/naive/json_items.cpp	2013-08-19 13:00:41 +
+++ src/store/naive/json_items.cpp	2014-04-02 22:46:08 +
@@ -288,7 +288,9 @@
 bool accumulate)
 {
   ASSERT_INVARIANT();
-  const char* lName = aName->getStringValue().c_str();
+  zstring zname;
+  aName->getStringValue2( zname );
+  const char *const lName = zname.c_str();
 
   Keys::iterator ite = theKeys.find(lName);
 
@@ -298,13 +300,12 @@
 
 if (getCollection() != NULL && (aValue->isStructuredItem()))
 {
-  assert(dynamic_cast(aValue.getp()));
+  assert( dynamic_cast(lValue) );
   StructuredItem* lStructuredItem = static_cast(aValue.getp());
   lStructuredItem->setCollectionTreeInfo(theCollectionInfo);
 }
 
-csize lPosition = thePairs.size();
-theKeys.insert(std::make_pair(lName, lPosition));
+theKeys.insert( std::make_pair( lName, thePairs.size() ) );
 thePairs.push_back(std::make_pair(aName.getp(), lValue));
 aName->addReference();
 lValue->addReference();
@@ -314,7 +315,7 @@
   }
   else if (accumulate)
   {
-csize lPosition = ite->second;
+size_type lPosition = ite->second;
 
 assert(thePairs[lPosition].first->getStringValue() == lName);
 
@@ -355,21 +356,20 @@
 {
   ASSERT_INVARIANT();
 
-  const char* lName = aName->getStringValue().c_str();
-  store::Item_t lValue;
+  zstring zname;
+  aName->getStringValue2( zname );
+  const char *const lName = zname.c_str();
 
   Keys::iterator lIter = theKeys.find(lName);
   if (lIter == theKeys.end())
   {
 ASSERT_INVARIANT();
-return 0;
+return NULL;
   }
-  csize lPosition = lIter->second;
+  size_type lPosition = lIter->second;
   
-  store::Item* lKey;
-
-  lKey = thePairs[lPosition].first;
-  lValue = thePairs[lPosition].second;
+  store::Item *const lKey = thePairs[lPosition].first;
+  store::Item_t lValue( thePairs[lPosition].second );
 
   if (getCollection() != NULL && (lValue->isStructuredItem()))
   {
@@ -390,7 +390,7 @@
 Keys::iterator lKeysEnd = theKeys.end();
 for (; lKeysIte != lKeysEnd; ++lKeysIte)
 {
-  csize lPos = lKeysIte->second;
+  size_type lPos = lKeysIte->second;
   if (lPos > lPosition)
   {
 lKeysIte->second = lPos - 1;
@@ -411,7 +411,9 @@
 const store::Item_t& aValue)
 {
   ASSERT_INVARIANT();
-  const char* lName = aName->getStringValue().c_str();
+  zstring zname;
+  aName->getStringValue2( zname );
+  const char *const lName = zname.c_str();
 
   Keys::const_iterator lIter = theKeys.find(lName);
   if (lIter == theKeys.end())
@@ -419,7 +421,7 @@
 ASSERT_INVARIANT();
 return 0;
   }
-  csize lPosition = lIter->second;
+  size_type lPosition = lIter->second;
 
   assert(thePairs[lPosition].first->getStringValue() == lName);
 
@@ -461,8 +463,11 @@
 const store::Item_t& aNewName)
 {
   ASSERT_INVARIANT();
-  const char* lName = aName->getStringValue().c_str();
-  const char* lNewName = aNewName->getStringValue().c_str();
+  zstring zname, znewname;
+  aName->getStringValue2( zname );
+  aNewName->getStringValue2( znewname );
+  const char *const lName = zname.c_str();
+  const char *const lNewName = znewname.c_str();
 
   Keys::const_iterator lIter = theKeys.find(lNewName);
   if (lI

Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-04-02 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/213942
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Zorba Build Bot
Validation queue result for 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848

Stage "CommitZorba" failed.

Check console output at 
http://jenkins.lambda.nu:8180/job/CommitZorba/309/console to view the results.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba.

Commit message:
1. Fixed user-reported core-dump for JSON-to-XML conversion.
2. Now also using getNameAsString() in PlanIterator.
3. Normalizing whitespace for JSound constraint error messages so they print 
better.

Requested reviews:
  Matthias Brantner (matthias-brantner)
  Paul J. Lucas (paul-lucas)

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848

1. Fixed user-reported core-dump for JSON-to-XML conversion.
2. Now also using getNameAsString() in PlanIterator.
3. Normalizing whitespace for JSound constraint error messages so they print 
better.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'src/runtime/base/plan_iterator.cpp'
--- src/runtime/base/plan_iterator.cpp	2014-02-25 00:14:59 +
+++ src/runtime/base/plan_iterator.cpp	2014-03-20 02:10:46 +
@@ -232,7 +232,7 @@
 else
   std::cout << "next (" << iter;
 
-std::cout << " = " << typeid (*iter).name()
+std::cout << " = " << iter->getNameAsString() // typeid (*iter).name()
   << ") -> "
   << "status: " << status << " -> "
   << ((status && result != NULL) ? result->show().c_str() : "null")

=== modified file 'src/runtime/json/snelson.cpp'
--- src/runtime/json/snelson.cpp	2013-12-27 15:11:16 +
+++ src/runtime/json/snelson.cpp	2014-03-20 02:10:46 +
@@ -186,6 +186,7 @@
   ADD_PAIR_ELEMENT( value_item->getStringValue() );
   PUSH_ITEM( xml );
   value_item = json_item->getObjectValue( value_item );
+  ZORBA_ASSERT( !!value_item );
   added_pair_element = true;
 }
 
@@ -246,6 +247,7 @@
 ADD_TYPE_ATTRIBUTE( "array" );
 ADD_ITEM_ELEMENT( "array" );
 PUSH_STATE( in_array );
+PUSH_ITEM( json );
 PUSH_ITERATOR( cur_iter );
 cur_iter = value_item->getArrayValues();
 cur_iter->open();

=== modified file 'src/runtime/jsound/jsound_util.cpp'
--- src/runtime/jsound/jsound_util.cpp	2014-03-14 22:38:03 +
+++ src/runtime/jsound/jsound_util.cpp	2014-03-20 02:10:46 +
@@ -1951,13 +1951,15 @@
   it->open();
   while ( it->next( item ) ) {
 ASSERT_TYPE( item, "constraint", XS_STRING );
-zstring const constraint_str( item->getStringValue() );
+zstring constraint_str( item->getStringValue() );
 try {
   unique_ptr c( new constraint( constraint_str ) );
   constraints_.push_back( c.get() );
   c.release();
 }
 catch ( ZorbaException const &e ) {
+  ascii::replace_all( constraint_str, '\n', ' ' );
+  ascii::normalize_space( constraint_str );
   throw XQUERY_EXCEPTION(
 jse::ILLEGAL_FACET_VALUE,
 ERROR_PARAMS(

=== added file 'test/rbkt/ExpQueryResults/zorba/json/json-snelson-j2x-object-06.xml.res'
--- test/rbkt/ExpQueryResults/zorba/json/json-snelson-j2x-object-06.xml.res	1970-01-01 00:00:00 +
+++ test/rbkt/ExpQueryResults/zorba/json/json-snelson-j2x-object-06.xml.res	2014-03-20 02:10:46 +
@@ -0,0 +1,8 @@
+http://john.snelson.org.uk/parsing-json-into-xquery"; type="object">
+  
+
+  1
+
+2
+  
+

=== added file 'test/rbkt/Queries/zorba/json/json-snelson-j2x-object-06.xq'
--- test/rbkt/Queries/zorba/json/json-snelson-j2x-object-06.xq	1970-01-01 00:00:00 +
+++ test/rbkt/Queries/zorba/json/json-snelson-j2x-object-06.xq	2014-03-20 02:10:46 +
@@ -0,0 +1,12 @@
+import module namespace jx = "http://zorba.io/modules/json-xml";;
+
+let $json :=
+  {
+"x" : {
+  "a" : [ 1 ],
+  "b" : 2
+}
+  }
+return jx:json-to-xml( $json )
+
+(: vim:set et sw=2 ts=2: :)

-- 
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Zorba Build Bot
Voting criteria failed for the following merge proposals:

https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848 :
Votes: {'Approve': 1}
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Description changed to:

1. Fixed user-reported core-dump for JSON-to-XML conversion.
2. Now also using getNameAsString() in PlanIterator.
3. Normalizing whitespace for JSound constraint error messages so they print 
better.

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Commit Message changed to:

1. Fixed user-reported core-dump for JSON-to-XML conversion.
2. Now also using getNameAsString() in PlanIterator.
3. Normalizing whitespace for JSound constraint error messages so they print 
better.

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Commit Message changed to:

1. Fixed user-reported core-dump for JSON-to-XML conversion.
2. Now also using getNameAsString() in PlanIterator.

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-19 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Description changed to:

1. Fixed user-reported core-dump for JSON-to-XML conversion.
2. Now also using getNameAsString() in PlanIterator.

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211848
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-14 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211150
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-14 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211150
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211150
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-14 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211150

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211150
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-14 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211150
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-14 Thread Paul J. Lucas
The parser changes are the unintentional consequences of the recent orthogonal 
parser changes on the no_commas branch.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211150
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-14 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211150
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211150
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-14 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/211150
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-11 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-11 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-11 Thread Zorba Build Bot
Validation queue result for 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509

Stage "TestZorbaUbuntu" failed.
23 tests failed (8757 total tests run).

Check test results at 
http://jenkins.lambda.nu:8180/job/TestZorbaUbuntu/478/testReport/ to view the 
results.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-11 Thread Zorba Build Bot
Validation queue result for 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509

Stage "TestZorbaUbuntu" failed.
20 tests failed (8757 total tests run).

Check test results at 
http://jenkins.lambda.nu:8180/job/TestZorbaUbuntu/479/testReport/ to view the 
results.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-11 Thread Zorba Build Bot
Validation queue result for 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509

Stage "BuildZorbaUbuntu" failed.

Check compiler output at 
http://jenkins.lambda.nu:8180/job/BuildZorbaUbuntu/542/parsed_console to view 
the results.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-11 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-11 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-11 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/210509
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Zorba Build Bot
Validation queue result for 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825

Stage "BuildZorbaUbuntu" failed.

Check compiler output at 
http://jenkins.zorba.io:8180/job/BuildZorbaUbuntu/540/parsed_console to view 
the results.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba.

Commit message:
Maintaining more state to avoid construction/allocation/destruction of 
istringstream.

Requested reviews:
  Paul J. Lucas (paul-lucas)

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825

Maintaining more state to avoid construction/allocation/destruction of 
istringstream.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'src/api/mem_streambuf.cpp'
--- src/api/mem_streambuf.cpp	2013-12-27 15:11:16 +
+++ src/api/mem_streambuf.cpp	2014-03-07 02:12:46 +
@@ -17,7 +17,6 @@
 #include "stdafx.h"
 #include   /* for memcpy(3) */
 
-#include 
 #include 
 
 #include "diagnostics/assert.h"

=== modified file 'src/runtime/csv/csv_impl.cpp'
--- src/runtime/csv/csv_impl.cpp	2014-03-06 19:06:43 +
+++ src/runtime/csv/csv_impl.cpp	2014-03-07 02:12:46 +
@@ -157,11 +157,10 @@
   return false;
 }
 
-static json::type parse_json( zstring const &s, json::token *ptoken ) {
-  mem_streambuf buf( (char*)s.data(), s.size() );
-  istringstream iss;
-  iss.ios::rdbuf( &buf );
-  json::lexer lex( iss );
+static json::type parse_json( zstring const &s, csv_parse_json_state &state,
+  json::token *ptoken ) {
+  state.set_data( s.data(), s.size() );
+  json::lexer lex( state.iss_ );
   if ( !lex.next( ptoken, false ) )
 return json::none;
   json::token::type const tt = ptoken->get_type();
@@ -221,9 +220,9 @@
 state->csv_.set_stream( item->getStream() );
   else {
 item->getStringValue2( state->string_ );
-state->mem_streambuf_.set( state->string_.data(), state->string_.size() );
-state->iss_.ios::rdbuf( &state->mem_streambuf_ );
-state->csv_.set_stream( state->iss_ );
+state->input_buf_.set( state->string_.data(), state->string_.size() );
+state->input_iss_.ios::rdbuf( &state->input_buf_ );
+state->csv_.set_stream( state->input_iss_ );
   }
 }
 
@@ -428,7 +427,7 @@
 GENV_ITEMFACTORY->createBoolean( item, false );
   else {
 json::token t;
-switch ( parse_json( *value, &t ) ) {
+switch ( parse_json( *value, state->parse_json_state_, &t ) ) {
   case json::boolean:
 GENV_ITEMFACTORY->createBoolean( item, (*value)[0] == 't' );
 break;

=== modified file 'src/runtime/csv/csv_util.h'
--- src/runtime/csv/csv_util.h	2013-08-23 00:43:04 +
+++ src/runtime/csv/csv_util.h	2014-03-07 02:12:46 +
@@ -17,10 +17,32 @@
 #ifndef ZORBA_CSV_UTIL_H
 #define ZORBA_CSV_UTIL_H
 
+// standard
+#include 
+
+// Zorba
+#include 
+
 namespace zorba {
 
 ///
 
+struct csv_parse_json_state {
+  std::istringstream iss_;
+
+  csv_parse_json_state() {
+iss_.std::ios::rdbuf( &buf_ );
+  }
+
+  void set_data( char const *s, size_t size ) {
+buf_.set( const_cast( s ), size );
+iss_.seekg( 0 );
+  }
+
+private:
+  mem_streambuf buf_;
+};
+
 namespace missing {
   enum type {
 null,

=== modified file 'src/runtime/csv/pregenerated/csv.h'
--- src/runtime/csv/pregenerated/csv.h	2014-03-06 19:06:43 +
+++ src/runtime/csv/pregenerated/csv.h	2014-03-07 02:12:46 +
@@ -50,11 +50,12 @@
   bool cast_unquoted_; //
   csv_parser csv_; //
   zstring extra_name_; //
-  std::istringstream iss_; //
+  mem_streambuf input_buf_; //
+  std::istringstream input_iss_; //
   std::vector keys_; //
   unsigned line_no_; //
-  mem_streambuf mem_streambuf_; //
   missing::type missing_; //
+  csv_parse_json_state parse_json_state_; //
   bool skip_called_; //
   zstring string_; //
   zstring value_; //

=== modified file 'src/runtime/spec/csv/csv.xml'
--- src/runtime/spec/csv/csv.xml	2014-03-06 19:06:43 +
+++ src/runtime/spec/csv/csv.xml	2014-03-07 02:12:46 +
@@ -32,11 +32,12 @@
 
 
 
-
+
+
 
 
-
 
+
 
 
 

-- 
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209825
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Zorba Build Bot
Validation queue result for 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760

Stage "CommitZorba" failed.

Check console output at 
http://jenkins.zorba.io:8180/job/CommitZorba/303/console to view the results.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Zorba Build Bot
Voting criteria failed for the following merge proposals:

https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760 :
Votes: {'Approve': 1}
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Description changed to:

1. In a couple of cases, now reusing the same zstring so the cost of growing 
the string is not paid per value.
2. Added a string_appender class that "chunks" the appending of single 
characters onto the end of strings to reduce the number of times 
zstring::append() is called.
3. Improved creating of JSON objects by using iterators rather than operator[].

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Commit Message changed to:

1. In a couple of cases, now reusing the same zstring so the cost of growing 
the string is not paid per value.
2. Added a string_appender class that "chunks" the appending of single 
characters onto the end of strings to reduce the number of times 
zstring::append() is called.
3. Improved creating of JSON objects by using iterators rather than operator[].

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Paul J. Lucas
1. In a couple of cases, now reusing the same zstring so the cost of growing 
the string is not paid per value.
2. Added a string_appender class that "chunks" the appending of single 
characters onto the end of strings to reduce the number of times 
zstring::append() is called.
3. Improved creating of JSON objects by using iterators rather than operator[].
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-06 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba.

Requested reviews:
  Paul J. Lucas (paul-lucas)

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209760
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'src/runtime/csv/csv_impl.cpp'
--- src/runtime/csv/csv_impl.cpp	2014-03-01 01:57:00 +
+++ src/runtime/csv/csv_impl.cpp	2014-03-06 19:08:12 +
@@ -365,7 +365,7 @@
   store::Item_t item;
   vector keys_copy, values;
   set keys_omit;
-  zstring value;
+  zstring *value;
   bool eol, quoted, swap_keys = false;
 
   CsvParseIteratorState *state;
@@ -381,7 +381,8 @@
 set_options( item, state );
   }
 
-  while ( state->csv_.next_value( &value, &eol, "ed ) ) {
+  while ( state->csv_.next_value( &state->value_, &eol, "ed ) ) {
+value = &state->value_;
 if ( state->keys_.size() && values.size() == state->keys_.size() &&
  state->extra_name_.empty() ) {
   //
@@ -390,13 +391,13 @@
   //
   throw XQUERY_EXCEPTION(
 csv::EXTRA_VALUE,
-ERROR_PARAMS( value, state->line_no_ ),
+ERROR_PARAMS( *value, state->line_no_ ),
 ERROR_LOC( loc )
   );
 }
 
 item = nullptr;
-if ( value.empty() ) {
+if ( value->empty() ) {
   if ( state->keys_.empty() ) {
 //
 // Header field names can never be empty.
@@ -408,7 +409,7 @@
 );
   }
   if ( quoted )
-GENV_ITEMFACTORY->createString( item, value );
+GENV_ITEMFACTORY->createString( item, *value );
   else
 switch ( state->missing_ ) {
   case missing::error:
@@ -421,15 +422,15 @@
 break;
 }
 } else if ( state->cast_unquoted_ && !quoted && !state->keys_.empty() ) {
-  if ( value == "T" || value == "Y" )
+  if ( *value == "T" || *value == "Y" )
 GENV_ITEMFACTORY->createBoolean( item, true );
-  else if ( value == "F" || value == "N" )
+  else if ( *value == "F" || *value == "N" )
 GENV_ITEMFACTORY->createBoolean( item, false );
   else {
 json::token t;
-switch ( parse_json( value, &t ) ) {
+switch ( parse_json( *value, &t ) ) {
   case json::boolean:
-GENV_ITEMFACTORY->createBoolean( item, value[0] == 't' );
+GENV_ITEMFACTORY->createBoolean( item, (*value)[0] == 't' );
 break;
   case json::null:
 GENV_ITEMFACTORY->createJSONNull( item );
@@ -437,24 +438,24 @@
   case json::number:
 switch ( t.get_numeric_type() ) {
   case json::token::integer:
-GENV_ITEMFACTORY->createInteger( item, xs_integer( value ) );
+GENV_ITEMFACTORY->createInteger( item, xs_integer( *value ) );
 break;
   case json::token::decimal:
-GENV_ITEMFACTORY->createDecimal( item, xs_decimal( value ) );
+GENV_ITEMFACTORY->createDecimal( item, xs_decimal( *value ) );
 break;
   case json::token::floating_point:
-GENV_ITEMFACTORY->createDouble( item, xs_double( value ) );
+GENV_ITEMFACTORY->createDouble( item, xs_double( *value ) );
 break;
   default:
 ZORBA_ASSERT( false );
 }
 break;
   default:
-GENV_ITEMFACTORY->createString( item, value );
+GENV_ITEMFACTORY->createString( item, *value );
 } // switch
   } // else
 } else {
-  GENV_ITEMFACTORY->createString( item, value );
+  GENV_ITEMFACTORY->createString( item, *value );
 }
 
 if ( !item.isNull() )

=== modified file 'src/runtime/csv/pregenerated/csv.h'
--- src/runtime/csv/pregenerated/csv.h	2014-01-31 21:47:54 +
+++ src/runtime/csv/pregenerated/csv.h	2014-03-06 19:08:12 +
@@ -57,6 +57,7 @@
   missing::type missing_; //
   bool skip_called_; //
   zstring string_; //
+  zstring value_; //
 
   CsvParseIteratorState();
 

=== modified file 'src/runtime/spec/csv/csv.xml'
--- src/runtime/spec/csv/csv.xml	2014-02-28 01:46:12 +
+++ src/runtime/spec/csv/csv.xml	2014-03-06 19:08:12 +
@@ -39,6 +39,7 @@
 
 
 
+
   
   
 

=== modified file 'src/store/naive/simple_item_factory.cpp'
--- src/store/naive/simple_item_factory.cpp	2013-09-17 21:12:49 +
+++ src/store/naive/simple_item_factory.cpp	2014-03-06 19:08:12 +
@@ -2366,19 +2366,20 @@
 const std::vector& names,
 const std::vector& values)
 {
+  assert( names.size() == values.size() );
+
   result = new json::SimpleJSONObject();
-
-  json::JSONObject* obj = static_cast(result.getp());
-
-  assert(names.size() == values.size());
-
-  csize numPairs = names.size();
-  for (csize i = 0; i < numPairs; ++i)
-  {
-if (!obj->add(names[i], values[i], false))
-{
-  RAISE_ERROR_NO_LOC(jerr::JNDY0003,

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-05 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-05 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-05 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-05 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-05 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-05 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-05 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba.

Commit message:
More help message clean-up.

Requested reviews:
  Paul J. Lucas (paul-lucas)

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559

More help message clean-up.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/209559
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'bin/zorbacmd_args.cpp'
--- bin/zorbacmd_args.cpp	2014-03-04 03:22:16 +
+++ bin/zorbacmd_args.cpp	2014-03-05 23:41:53 +
@@ -78,30 +78,30 @@
 
 // b //
 
-HELP_OPT( "--base-uri" )
+HELP_OPT( "--base-uri " )
   "Set the base URI property of the static context.\n\n"
 
-HELP_OPT( "--boundary-space" )
-  "Set the boundary-space policy ('strip' or 'preserve') in the static context.\n\n"
+HELP_OPT( "--boundary-space {strip|preserve}" )
+  "Set the boundary-space policy in the static context.\n\n"
 
 HELP_OPT( "--byte-order-mark" )
   "Set the byte-order-mark for the serializer.\n\n"
 
 // c //
 
-HELP_OPT( "--classpath" )
-  "JVM classpath to be used by modules using Java implementations\n\n"
+HELP_OPT( "--classpath " )
+  "Set the JVM classpath to be used by modules using Java implementations.\n\n"
 
 HELP_OPT( "--compile-only" )
   "Only compile (don't execute).\n\n"
 
-HELP_OPT( "--compile-plan," )
+HELP_OPT( "--compile-plan" )
   "Do not execute the query; just compile it and save the execution plan in the file specified with the -o option.\n\n"
 
-HELP_OPT( "--construction-mode" )
-  "Set the construction mode ('strip' or 'preserve') in the static context.\n\n"
+HELP_OPT( "--construction-mode {strip|preserve}" )
+  "Set the construction mode in the static context.\n\n"
 
-HELP_OPT( "--context-item" )
+HELP_OPT( "--context-item " )
   "Set the context item to the XML document in a given file.\n\n"
 
 // d //
@@ -111,21 +111,21 @@
   "Launch the Zorba debugger server and connect to a DBGP-enabled debugger client.\n\n"
 #endif /* ZORBA_WITH_DEBUGGER */
 
-HELP_OPT( "--debug-file" )
+HELP_OPT( "--debug-file " )
   "Sets the file to write developer debugging information to.\n\n"
 
-HELP_OPT( "--debug-stream " )
+HELP_OPT( "--debug-stream {1|cout|stdout|2|cerr|stderr}" )
   "Sets the stream to write developer debugging information to.\n\n"
 
 #ifdef ZORBA_WITH_DEBUGGER
-HELP_OPT( "--debug-host, -h" )
-  "The host where the DBGP-enabled debugger client listens for connections. Defaults to: 127.0.0.1\n\n"
+HELP_OPT( "--debug-host, -h " )
+  "The host where the DBGP-enabled debugger client listens for connections; default: 127.0.0.1.\n\n"
 
-HELP_OPT( "--debug-port, -p" )
-  "The port on which the DBGP-enabled debugger client listens for connections. Defaults to: 28028\n\n"
+HELP_OPT( "--debug-port, -p " )
+  "The port on which the DBGP-enabled debugger client listens for connections; defaults: 28028.\n\n"
 #endif /* ZORBA_WITH_DEBUGGER */
 
-HELP_OPT( "--default-collation" )
+HELP_OPT( "--default-collation " )
   "Add the given collation and set the value of the default collation in the static context to the given collation.\n\n"
 
 HELP_OPT( "--disable-http-resolution" )
@@ -139,8 +139,8 @@
 HELP_OPT( "--execute-plan" )
   "Do not compile the query; instead load the execution plan from the file specified by the -f -q options (or by any file specified without any other argument), and execute the loaded plan.\n\n"
 
-HELP_OPT( "--external-variable, -e" )
-  "Provide the value for a variable given a file (name=file) or a value (name:=value)\n\n"
+HELP_OPT( "--external-variable, -e {=file|:=}" )
+  "Set the value for an externsl variable.\n\n"
 
 // f //
 
@@ -149,7 +149,7 @@
 
 // h //
 
-HELP_OPT( "--help, -h" )
+HELP_OPT( "--help" )
   "Print this help message.\n\n"
 
 // i //
@@ -163,8 +163,8 @@
 HELP_OPT( "--inline-udf" )
   "Inline user-defined functions.\n\n"
 
-HELP_OPT( "--iterator-tree " )
-  "Print the iterator tree in in one of DOT, JSON, or XML formats.\n\n"
+HELP_OPT( "--iterator-tree {dot|json|xml}>" )
+  "Print the iterator tree in the given format.\n\n"
 
 // j //
 
@@ -176,7 +176,7 @@
 HELP_OPT( "--lib-module, -l" )
   "Query compiler option to treat the query as a library module. If this is set --compile-only option is also s

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-03 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-03 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-03 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-03 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-01 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-01 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-01 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba.

Commit message:
Fixed help message.

Requested reviews:
  Paul J. Lucas (paul-lucas)

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935

Fixed help message.
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208935
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'bin/zorbacmd_args.cpp'
--- bin/zorbacmd_args.cpp	2014-02-28 00:31:04 +
+++ bin/zorbacmd_args.cpp	2014-03-01 15:35:13 +
@@ -808,7 +808,7 @@
   if ( !error )
 error = check_args();
   if ( error ) {
-cout << "Error: " << error << "\nUse -h for help." << endl;
+cout << "Error: " << error << "\nUse --help for help." << endl;
 exit( 1 );
   }
   return argv - argv_orig;

=== modified file 'test/iterplans/apitest_args.cpp'
--- test/iterplans/apitest_args.cpp	2014-02-21 14:58:32 +
+++ test/iterplans/apitest_args.cpp	2014-03-01 15:35:13 +
@@ -548,7 +548,7 @@
   if ( !error )
 error = check_args();
   if ( error ) {
-cout << "Error: " << error << "\nUse -h for help." << endl;
+cout << "Error: " << error << "\nUse --help for help." << endl;
 exit( 1 );
   }
   return argv - argv_orig;

-- 
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-01 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208907
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208907
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-01 Thread Zorba Build Bot
Validation queue succeeded - proposal merged!
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208907
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-03-01 Thread Zorba Build Bot
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208907

Progress dashboard at http://jenkins.zorba.io:8180/view/ValidationQueue
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208907
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-02-28 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208907
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-02-28 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Description changed to:

Discovered a bug where strings like street addresses, e.g., "870 Market 
Street", will cause an exception to be thrown due to the mis-guessing about 
what kind of JSON token a string really is. Fixed.

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208907
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208907
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:~paul-lucas/zorba/pjl-misc into lp:zorba

2014-02-28 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been 
updated.

Commit Message changed to:

Discovered a bug where strings like street addresses, e.g., "870 Market 
Street", will cause an exception to be thrown due to the mis-guessing about 
what kind of JSON token a string really is. Fixed.

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208907
-- 
https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/208907
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


  1   2   3   4   5   6   7   8   9   10   >