[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-03 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156766 -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156766 Your team Zorba Coders is

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-03 Thread Nicolae Brinza
Nicolae Brinza has proposed merging lp:~nbrinza/zorba/bugs2 into lp:zorba. Commit message: overflows in date/time casts now return FODT0001 and in durations return FODT0002. Requested reviews: Nicolae Brinza (nbrinza) Chris Hillery (ceejatec) Related bugs: Bug #898792 in Zorba: xs:QName

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-03 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 Your team Zorba Coders is

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-03 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 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

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-03 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Commit Message changed to: overflows in date/time casts now return FODT0001 and in durations return FODT0002. For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 --

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-03 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 Your team Zorba Coders is

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-05 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 Your team Zorba Coders is

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-05 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 Your team Zorba Coders is

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-06 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/156818 Your team Zorba Coders is

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-07 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Commit Message changed to: The jsoniq parser now accepts as a valid string literal. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/157559 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-07 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/157559 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/157559

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-07 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/157559 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 :

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-08 Thread Nicolae Brinza
Nicolae Brinza has proposed merging lp:~nbrinza/zorba/bugs2 into lp:zorba. Commit message: More datetime/duration fixes. Requested reviews: Nicolae Brinza (nbrinza) Chris Hillery (ceejatec) Related bugs: Bug #898792 in Zorba: xs:QName fails for non-literal input https

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-08 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/157752 -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/157752 Your team Zorba Coders is

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-08 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/157752 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

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-08 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/157752 -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/157752 Your team Zorba Coders is

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-08 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/157752 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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bump_FOTS_snapshot_Apr_09 into lp:zorba

2013-04-09 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bump_FOTS_snapshot_Apr_09/+merge/157896 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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-09 Thread Nicolae Brinza
I'm not sure either. It might be that the scanner .cpp file in the trunk had been generated with a different version of flex so cmake decided to rebuild it. -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/157559 Your team Zorba Coders is subscribed to branch lp:zorba. --

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-09 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/157752 -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/157752 Your team Zorba Coders is

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-09 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/157752 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

[Zorba-coders] [Merge] lp:~nbrinza/zorba/error-messages into lp:zorba

2013-04-10 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/error-messages into lp:zorba has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/148300 -- https://code.launchpad.net/~nbrinza/zorba/error-messages/+merge/148300

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-http_parse-fragment into lp:zorba

2013-04-10 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-http_parse-fragment/+merge/157991 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-http_parse-fragment into lp:zorba

2013-04-10 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/bug-http_parse-fragment into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-http_parse-fragment/+merge/157991 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1123164 into lp:zorba

2013-04-11 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1123164/+merge/158254 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 :

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-1085408 into lp:zorba

2013-04-16 Thread Nicolae Brinza
The proposal to merge lp:~paul-lucas/zorba/bug-1085408 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~paul-lucas/zorba/bug-1085408/+merge/159063 -- https://code.launchpad.net/~paul-lucas/zorba/bug-1085408/+merge/159063

Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-1085408 into lp:zorba

2013-04-16 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~paul-lucas/zorba/bug-1085408/+merge/159063 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-16 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159267 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159267

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-16 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159267 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159267

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-16 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Commit Message changed to: Merging of xquery and jsoniq grammar files step 1 For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159274 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-16 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159274 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159274

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-16 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159274 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159274

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-16 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159274 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159274

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-17 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159274 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159274

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-17 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Commit Message changed to: Merged the xquery and jsoniq grammar files into one For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159477 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-17 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159477 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159477

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-17 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159477 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-17 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159477 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159477

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-18 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159736 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159736

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-18 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159736 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-18 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-18 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-18 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Commit Message changed to: The new JSONiq grammar accept C++ style comments For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-18 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-18 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1123165 into lp:zorba

2013-04-19 Thread Nicolae Brinza
Ghislain, could you also please remove the fix that you submitted for scanner.l problem? Your proposal works, but on linux the GCC gives off a warning for that code. I have a proposal that fixes the issue in another branch, which Matthias will approve and merge soon. --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-19 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-19 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-19 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-19 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1123165 into lp:zorba

2013-04-19 Thread Nicolae Brinza
I too agree that it should be moved to the translator. That's where all the similar code currently is. Other than this and the scanner issue, it looks good for me. -- https://code.launchpad.net/~zorba-coders/zorba/bug-1123165/+merge/159822 Your team Zorba Coders is subscribed to branch

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1123165 into lp:zorba

2013-04-19 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1123165/+merge/159822 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-19 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-19 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-04-19 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/159739 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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1123165 into lp:zorba

2013-04-23 Thread Nicolae Brinza
We do not have to consider $f an object lookup in this case, right? I have the more general feeling that there might be something wrong in type inference regarding for loops on empty sequences. Since there is a partial application ?, it definitely cannot be an object lookup. --

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-23 Thread Nicolae Brinza
Nicolae Brinza has proposed merging lp:~nbrinza/zorba/bugs2 into lp:zorba. Commit message: Added W3C bug reference in disputed bugs Requested reviews: Nicolae Brinza (nbrinza) Sorin Marian Nasoi (sorin.marian.nasoi) Related bugs: Bug #898792 in Zorba: xs:QName fails for non-literal input

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-23 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/160440 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

Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-23 Thread Nicolae Brinza
I've added DISPUTED to the tests. -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/160440 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 :

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-23 Thread Nicolae Brinza
Nicolae Brinza has proposed merging lp:~nbrinza/zorba/bugs2 into lp:zorba. Commit message: Added W3C bug reference in disputed bugs. Requested reviews: Nicolae Brinza (nbrinza) Sorin Marian Nasoi (sorin.marian.nasoi) Related bugs: Bug #898792 in Zorba: xs:QName fails for non-literal input

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-23 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Commit Message changed to: Added W3C bug reference in disputed bugs. For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/160447 --

[Zorba-coders] [Merge] lp:~nbrinza/zorba/bugs2 into lp:zorba

2013-04-23 Thread Nicolae Brinza
The proposal to merge lp:~nbrinza/zorba/bugs2 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/160447 -- https://code.launchpad.net/~nbrinza/zorba/bugs2/+merge/160447 Your team Zorba Coders is

Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-1167704 into lp:zorba

2013-04-23 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~paul-lucas/zorba/bug-1167704/+merge/160437 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 :

Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-1167704 into lp:zorba

2013-04-23 Thread Nicolae Brinza
Looks good. -- https://code.launchpad.net/~paul-lucas/zorba/bug-1167704/+merge/160437 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

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-1167704 into lp:zorba

2013-04-23 Thread Nicolae Brinza
The proposal to merge lp:~paul-lucas/zorba/bug-1167704 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~paul-lucas/zorba/bug-1167704/+merge/160437 -- https://code.launchpad.net/~paul-lucas/zorba/bug-1167704/+merge/160437

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-01 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/161892 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/161892

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-01 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Commit Message changed to: Test merge to check for RQ regressions. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/161912 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-03 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162362 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162362

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-03 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Commit Message changed to: Added common language option and warnings. Re-enabled path expressions in JSONiq grammar. For more details, see:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-03 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
Nicolae Brinza has proposed merging lp:~zorba-coders/zorba/bug-1176038 into lp:zorba. Commit message: Fixed a bug in the parser that was revealed by bison 2.7 + #include clean-up Requested reviews: Nicolae Brinza (nbrinza) Paul J. Lucas (paul-lucas) Related bugs: Bug #1176038 in Zorba

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1176038/+merge/162953 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/bug-1176038 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1176038/+merge/162953 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/bug-1176038 into lp:zorba has been updated. Commit Message changed to: Fixed a bug in the parser that was revealed by bison 2.7 + #include clean-up For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1176038/+merge/162953 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/bug-1176038 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1176038/+merge/162953 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
I get absolutely no warnings whatsoever in the scanner, and neither does the remote queue, e.g. http://zorbatest.lambda.nu:8080/remotequeue/bug-1176038-2013-05-08T11-41-52.814Z/ctest.txt . If you get them with your compiler, can't you fix them? --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
I have the 2.7 bison and the 2.5.35 which are the same as you have and yet I do not get the warnings. I do have a different gcc -- 4.7, but I'm not really sure how to downgrade it to 4.2 which is what you have -- gcc version 4.2.1 (Based on Apple Inc. build 5658) (LLVM build 2336.11.00). And

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
The 2.5.35 version is that of flex. Your flex is 2.5.35 as well, but again it seems to be a customized Apple branch -- flex 2.5.35 Apple(flex-31). -- https://code.launchpad.net/~zorba-coders/zorba/bug-1176038/+merge/162953 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
Nicolae Brinza has proposed merging lp:~zorba-coders/zorba/bug-1176038 into lp:zorba. Commit message: Fixed a bug in the parser that was revealed by bison 2.7 + #include clean-up Requested reviews: Paul J. Lucas (paul-lucas) Nicolae Brinza (nbrinza) Related bugs: Bug #1176038 in Zorba

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/bug-1176038 into lp:zorba has been updated. Commit Message changed to: Fixed a bug in the parser that was revealed by bison 2.7 + #include clean-up For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1176038/+merge/162996 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1176038 into lp:zorba

2013-05-08 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1176038/+merge/162996 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-13 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-13 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-13 Thread Nicolae Brinza
Matthias, your suggestions for two improved warnings do not change the messages: you just switched the order of the words. The warnings are built around the template: feature not supported by the common language grammar{: $1} So it is not easy to make this swap. I could probably do it,

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-13 Thread Nicolae Brinza
Why would you need a qualified value if the option itself is already qualified? I guess because the option name and the option value may not live in the same namespace. Why not? It's not as if you do schema validation on the options and their values. The option needs to be qualified to

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-13 Thread Nicolae Brinza
(by the way, character entity references do not exist in XML. There are character references (#...;) and entity references (...;), but not both at the same time if I am correct :-) ) Indeed you're right. I trusted Wikipedia which uses the term character entity references and says ``The XML

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-13 Thread Nicolae Brinza
Fixed all the raised issues. -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/null-type-fix into lp:zorba

2013-05-15 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/null-type-fix/+merge/163883 Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/new-jsoniq. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/null-type-fix into lp:~zorba-coders/zorba/new-jsoniq

2013-05-15 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/null-type-fix/+merge/163902 Your team Zorba Coders is subscribed to branch lp:~zorba-coders/zorba/new-jsoniq. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
I've added a mention in the ChangeLog -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Approved = Rejected For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 Your

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Rejected = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
Done -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 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

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq into lp:zorba

2013-05-15 Thread Nicolae Brinza
Yeah, I just fixed that unit test as well... -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq/+merge/162375 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 :

<    1   2   3   4   5   >