Re: [PATCH] Re: Time zones (again)

2016-06-22 Thread Robert C. Helling
Hi, > Am 23.06.2016 um 04:17 schrieb Thiago Macieira : > > The assignment on that second line erases any settings from the previous line. > > If you want to parse *as* GMT, the only way to do it is using ISO format and > appending a "Z" to the target date. Something like > >timestamp = QDa

Re: [PATCH] Re: Time zones (again)

2016-06-22 Thread Thiago Macieira
On quarta-feira, 22 de junho de 2016 22:47:33 PDT Robert Helling wrote: > here is a patch which implements (I hope) what I promised. I did some quick > checks in particular with the planner (there it indeed fixes a bug reported > in the user forum which got me started in the first place thinking ab

Re: [PATCH] Re: Time zones (again)

2016-06-22 Thread Linus Torvalds
On Wed, Jun 22, 2016 at 4:20 PM, Dirk Hohndel wrote: > > OK, this has gone wrong so many times... Linus, Thiago, would you take a > look at this and give an ACK if you agree that this is reasonable to take? The patch looks good to me. The only case that isn't obvious is the use of startTime, but

Re: [PATCH] Re: Time zones (again)

2016-06-22 Thread Dirk Hohndel
On Wed, Jun 22, 2016 at 10:47:33PM +0200, Robert Helling wrote: > > > Am 21.06.2016 um 22:44 schrieb Linus Torvalds > > : > > > > On Tue, Jun 21, 2016 at 1:43 PM, Robert Helling wrote: > >> > >> I didn’t mean init of subsurface. I meant init of QDateTime. > > > > Yes, if that works, then that

Re: [PATCH] Re: Time zones (again)

2016-06-22 Thread Anton Lundin
On 22 June, 2016 - Robert Helling wrote: > > > Am 21.06.2016 um 22:44 schrieb Linus Torvalds > > : > > > > On Tue, Jun 21, 2016 at 1:43 PM, Robert Helling wrote: > >> > >> I didn’t mean init of subsurface. I meant init of QDateTime. > > > > Yes, if that works, then that sounds like a good id

[PATCH] Re: Time zones (again)

2016-06-22 Thread Robert Helling
> Am 21.06.2016 um 22:44 schrieb Linus Torvalds : > > On Tue, Jun 21, 2016 at 1:43 PM, Robert Helling wrote: >> >> I didn’t mean init of subsurface. I meant init of QDateTime. > > Yes, if that works, then that sounds like a good idea. > > Linus Hi, here is a patch which implemen