← Back to team overview

zorba-coders team mailing list archive

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

 

>   - the type of the timezone argument should be a long and the value indicate
> seconds (see also bug #1103115)
> This is a big change, it involves the way the TimeZone is managed internally,
> this is not the scope of this bug, this will require much more time and I
> think the changes for this bug need to be released.
> The changes on DateTimeStamp where merged from bug #924756, so, perhaps this
> new behavior can be required on bug #1103115 or on a new bug.
I leave the decision to Chris whether we want to extend the already broken API or do it once the bug is fixed.

> 
> - Why do you remove the catch clause in src/store/naive/simple_pul.cpp?
> Because the next clause does exactly the same.
OK. I hope Markos doesn't use this clause regularly.

-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1132937/+merge/155640
Your team Zorba Coders is subscribed to branch lp:zorba.


References