[PATCH] Re: Time zones (again)

Anton Lundin glance at acc.umu.se
Wed Jun 22 14:02:42 PDT 2016


On 22 June, 2016 - Robert Helling wrote:

> 
> > Am 21.06.2016 um 22:44 schrieb Linus Torvalds <torvalds at linux-foundation.org>:
> > 
> > On Tue, Jun 21, 2016 at 1:43 PM, Robert Helling <helling at atdotde.de> 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 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 about time zones again) but maybe I did not cover everything. More testing might be advised. But looking at the code leaves me quite confident about it.
> 
> Best
> Robert

> From 24d2a7e53c387166a0301066931016dd114229e3 Mon Sep 17 00:00:00 2001
> From: "Robert C. Helling" <helling at atdotde.de>
> Date: Wed, 22 Jun 2016 22:46:22 +0200
> Subject: [PATCH] Unify handling of QDateTime time zone information

...

> diff --git a/qt-models/diveplannermodel.cpp b/qt-models/diveplannermodel.cpp
> index 16a2e40..613b260 100644
> --- a/qt-models/diveplannermodel.cpp
> +++ b/qt-models/diveplannermodel.cpp
> @@ -4,6 +4,7 @@
>  #include "qt-models/cylindermodel.h"
>  #include "core/planner.h"
>  #include "qt-models/models.h"
> +#include <QDebug>

Nit-pickings but its a stray include.


//Anton


-- 
Anton Lundin	+46702-161604


More information about the subsurface mailing list