Reviewing merge request #160: Use local time for output / logs / JSON

Implements #388

Commits that would be merged:

Version 1
  • Version 1
  • 992f725
  • Sergey 'Jin' Bostandzhyan
over 1 year ago
  • 86681f5
  • Sergey 'Jin' Bostandzhyan
over 1 year ago
  • bcf063e
  • Sergey 'Jin' Bostandzhyan
over 1 year ago
  • cd85db1
  • Sergey 'Jin' Bostandzhyan
over 1 year ago
  • c35070b
  • Sergey 'Jin' Bostandzhyan
over 1 year ago
  • 91a69e8
  • Sergey 'Jin' Bostandzhyan
over 1 year ago
  • 8b2d52f
  • Sergey 'Jin' Bostandzhyan
over 1 year ago
  • 8833177
  • Sergey 'Jin' Bostandzhyan
over 1 year ago
Showing 54889af-992f725

Comments

+   static long int g_GMT_Offset = 0;
+ #include "core/datetools.h"

I'd move that into the Date class

+   time_offset = t.tm_gmtoff * (-1);

–(t.tm_gmtoff) should achieve the same without multiplication :–)

+   time_offset = t.tm_gmtoff * (-1);

okay, forget what I said about the multiplication, g++ is of course smart enough to do the right thing here :–)

→ State changed from Open to Closed

merged

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository