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

2012-11-16 Thread Paul J. Lucas
Review: Approve -- https://code.launchpad.net/~paul-lucas/zorba/bug-866973/+merge/134763 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-coder

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

2012-11-16 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/bug-866973 into lp:zorba. Commit message: Added timezone to InvalidTimezoneException. Requested reviews: Nicolae Brinza (nbrinza) Sorin Marian Nasoi (sorin.marian.nasoi) Paul J. Lucas (paul-lucas) Related bugs: Bug #866973 in Zorba:

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

2012-11-16 Thread Paul J. Lucas
I just checked: ZorbaException already has a global operator<<(ostream&,ZorbaException const&) function defined for it. So why do you need to do anything at all? -- https://code.launchpad.net/~zorba-coders/zorba/zorba-error_printer_in_api/+merge/134405 Your team Zorba Coders is subscribed to bra

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

2012-11-16 Thread Paul J. Lucas
> Anyway, Paul, can you help me on one thing please? Can you tell me how I can > use this ErrorPrinter for warnings too? An XQueryWarning is just a typedef for XQueryException, so it should just work as-is. That aside: generally, I don't like to decide for the user how errors should be printed.

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

2012-11-16 Thread David Graf
Cool, now I can fix what someone else did wrong :-(. Anyway, Paul, can you help me on one thing please? Can you tell me how I can use this ErrorPrinter for warnings too? > It's a stupid API. There's no reason to have a class with a single static > member function. If you're going to make it p

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

2012-11-16 Thread Nicolae Brinza
Why is the ouptut of cli.xq sent to stderr instead of stdout ? -- https://code.launchpad.net/~zorba-coders/zorba/fots_driver/+merge/133988 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net