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

2012-04-25 Thread Matthias Brantner
Matthias Brantner has proposed merging lp:~zorba-coders/zorba/bug-988412 into 
lp:zorba.

Requested reviews:
  Matthias Brantner (matthias-brantner)
  William Candillon (wcandillon)
Related bugs:
  Bug #988412 in Zorba: date:current-dateTime daylight saving 
  https://bugs.launchpad.net/zorba/+bug/988412

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-988412/+merge/103523

fix for bug #988412 (date:current-dateTime daylight saving)
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-988412/+merge/103523
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'ChangeLog'
--- ChangeLog	2012-04-24 12:39:38 +
+++ ChangeLog	2012-04-25 17:27:26 +
@@ -30,6 +30,7 @@
   * Fixed performance problem with the findNodeSources function of the no-copy rule
   * Fixed bug #872234 (prevent a rewritting to take place in case of sequential expr)
   * Fixed bug #906494 (default compile with D_FILE_OFFSET_BITS=64)
+  * Fixed bug #988412 (date:current-dateTime daylight saving)
   * Fixed bug #912586, #912593 and #912722 (assertion failures with lax validation)
   * Fixed bug #921458 (file:read-text-lines() blocking)
   * Fixed bug #981405 (do not hoist expr containing try-catch variables out of the

=== modified file 'modules/com/zorba-xquery/www/modules/datetime.xq.src/datetime.cpp'
--- modules/com/zorba-xquery/www/modules/datetime.xq.src/datetime.cpp	2012-04-24 12:39:38 +
+++ modules/com/zorba-xquery/www/modules/datetime.xq.src/datetime.cpp	2012-04-25 17:27:26 +
@@ -32,21 +32,23 @@
 long   timezone,
 unsigned short millitm)
 {
+  int lSummerTimeShift = 0;
 #if defined (WIN32)
   struct _timeb timebuffer;
   _ftime_s( timebuffer );
   localtime_s(gmtm, timebuffer.time); //thread safe localtime on Windows
+  lSummerTimeShift = -timebuffer.timezone * 60;
+  if (gmtm.tm_isdst != 0)
+lSummerTimeShift += 3600;
 #else
   struct timeb timebuffer;
   ftime( timebuffer );
   localtime_r(timebuffer.time, gmtm); //thread safe localtime on Linux
+  localtime_r(timebuffer.time, gmtm); //thread safe localtime on Linux
+  lSummerTimeShift = gmtm.tm_gmtoff;
 #endif
 
-  int lSummerTimeShift = 0;
-  if (gmtm.tm_isdst != 0)
-lSummerTimeShift = 3600;
-
-  timezone = -timebuffer.timezone * 60 + lSummerTimeShift; // in seconds
+  timezone = lSummerTimeShift; // in seconds
   millitm = timebuffer.millitm;
 }
 

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


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

2012-04-25 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/bug-988412 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-988412/+merge/103523
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-988412/+merge/103523
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


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

2012-04-25 Thread Matthias Brantner
Review: Approve


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

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


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

2012-04-25 Thread Zorba Build Bot
Validation queue job bug-988412-2012-04-25T17-54-01.991Z is finished. The final 
status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-988412/+merge/103523
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


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

2012-04-25 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/bug-988412 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-988412/+merge/103523
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-988412/+merge/103523
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