From: Victor Stinner <victor.stinner@gmail.com>
Date: Thu, 20 Apr 2017 11:41:09 +0000 (+0200)
Subject: Only define get_zone() and get_gmtoff() if needed (#1193)
X-Git-Tag: v3.7.0a1~932
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=8f5cdfa9fc1bb6b4d9a33fc281987252f6398430;p=python

Only define get_zone() and get_gmtoff() if needed (#1193)

Only define the get_zone() and get_gmtoff() private functions in the
time module if these functions are needed to initialize the module.

The change fixes the following warnings on AIX:

Modules/timemodule.c:1175:1: warning: 'get_gmtoff' defined but not used [-Wunused-function]
Modules/timemodule.c:1164:1: warning: 'get_zone' defined but not used [-Wunused-function]
---

diff --git a/Modules/timemodule.c b/Modules/timemodule.c
index 25eb92dada..423c001e2b 100644
--- a/Modules/timemodule.c
+++ b/Modules/timemodule.c
@@ -1160,6 +1160,7 @@ PyDoc_STRVAR(get_clock_info_doc,
 \n\
 Get information of the specified clock.");
 
+#if !defined(HAVE_TZNAME) || defined(__GLIBC__) || defined(__CYGWIN__)
 static void
 get_zone(char *zone, int n, struct tm *p)
 {
@@ -1180,6 +1181,7 @@ get_gmtoff(time_t t, struct tm *p)
     return timegm(p) - t;
 #endif
 }
+#endif /* !defined(HAVE_TZNAME) || defined(__GLIBC__) || defined(__CYGWIN__) */
 
 static void
 PyInit_timezone(PyObject *m) {