Re: [PATCH] Add timegm POSIX call

[Corinna Vinschen <vinschen at redhat dot com>]

On Aug 15 09:52, Andreas Köpke wrote:
> > On Aug 13 18:17, Andrew Russell via newlib wrote:
> > > >From e182faa79c35984b667029ef7b6e4a8ce7329897 Mon Sep 17 00:00:00 2001
> > > 
> > > From: Andrew Russell <ahrussell@google.com>
> > > Date: Fri, 10 Aug 2018 12:14:18 -0700
> > > Subject: [PATCH 1/4] Start of mktime.c copy to timegm.c
> > > 
> > > I am proposing to add the timegm POSIX call to
> > > Newlib. Part of this refactors some of the code in libc/time/local.h and
> > > libc/time/mktime.c, per this discussion:
> > > 
> > > https://sourceware.org/ml/newlib/2018/msg00186.html
> > 
> > I'm looking for comments from other (non-Cygwin) devs here.
> > 
> > 
> > Thanks,
> > Corinna
> 
> I did not test it yet but we need this function. cygwin has it, but uses a 
> completely different codebase (localtime.cc). 
> 
> It is always needed if your default timezone is not GMT and you need to 
> exchange time information with e.g. sync mechanisms  like NTP or GPS. You 
> usually get broken down time information referenced to GMT and need to re-
> assemble using GMT as reference. 
> 
> Using setenv/getenv to temporarily change the timezone does not work: there is 
> a memory leak in the implementation. Well, at least it looks like one. 

Patch?


Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

Attachment: signature.asc
Description: PGP signature