This is the mail archive of the
newlib@sourceware.org
mailing list for the newlib project.
Re: [PATCH] libc/time/gmtime_r.c, libc/time/lcltime_r.c,, libc/time/local.h, libc/time/mktm_r.c: move localtime related functionality, from _mktm_r() to new _mklocaltm_r() to break dependency of gmtime() on, timezones
- From: Freddie Chopin <freddie_chopin at op dot pl>
- To: newlib at sourceware dot org
- Date: Thu, 11 Sep 2014 22:18:20 +0200
- Subject: Re: [PATCH] libc/time/gmtime_r.c, libc/time/lcltime_r.c,, libc/time/local.h, libc/time/mktm_r.c: move localtime related functionality, from _mktm_r() to new _mklocaltm_r() to break dependency of gmtime() on, timezones
- Authentication-results: sourceware.org; auth=none
- References: <54077334 dot 6050804 at op dot pl> <540A36F0 dot 6090903 at op dot pl> <540F7654 dot 7070403 at op dot pl> <1330566861 dot 45960793 dot 1410465086152 dot JavaMail dot zimbra at redhat dot com>
W dniu 2014-09-11 21:51, Jeff Johnston pisze:
Hi Freddie,
Corinna who was looking at this previously is on vacation so I am reviewing it.
Good to know, thanks for your review!
The only comments I have are:
1. month_lengths needs to be renamed to be __month_lengths because it is now externalized.
2. you might as well be consistent with other files in LIB_SOURCES list of Makefile.am with regards to tabs vs spaces.
Both issues fixed. Updated patch and updated changelog attached.
Regards,
FCh
Attachment:
ChangeLog.txt
Description: Text document
Attachment:
0001-libc-time-Remove-dependency-of-gmtime-on-time-zone-r.patch
Description: Text document
- Follow-Ups:
- Re: [PATCH] libc/time/gmtime_r.c, libc/time/lcltime_r.c,, libc/time/local.h, libc/time/mktm_r.c: move localtime related functionality, from _mktm_r() to new _mklocaltm_r() to break dependency of gmtime() on, timezones
- Re: [PATCH] libc/time/gmtime_r.c, libc/time/lcltime_r.c,, libc/time/local.h, libc/time/mktm_r.c: move localtime related functionality, from _mktm_r() to new _mklocaltm_r() to break dependency of gmtime() on, timezones
- References:
- [PATCH] libc/time/gmtime_r.c, libc/time/lcltime_r.c,, libc/time/local.h, libc/time/mktm_r.c: move localtime related functionality, from _mktm_r() to new _mklocaltm_r() to break dependency of gmtime() on, timezones
- Re: [PATCH] libc/time/gmtime_r.c, libc/time/lcltime_r.c,, libc/time/local.h, libc/time/mktm_r.c: move localtime related functionality, from _mktm_r() to new _mklocaltm_r() to break dependency of gmtime() on, timezones
- Re: [PATCH] libc/time/gmtime_r.c, libc/time/lcltime_r.c,, libc/time/local.h, libc/time/mktm_r.c: move localtime related functionality, from _mktm_r() to new _mklocaltm_r() to break dependency of gmtime() on, timezones
- Re: [PATCH] libc/time/gmtime_r.c, libc/time/lcltime_r.c,, libc/time/local.h, libc/time/mktm_r.c: move localtime related functionality, from _mktm_r() to new _mklocaltm_r() to break dependency of gmtime() on, timezones