diff mbox

Fix Wundef warning for DEBUG in mktime

Message ID 20140626171842.GA12565@spoyarek.pnq.redhat.com
State Rejected
Headers

Commit Message

Siddhesh Poyarekar June 26, 2014, 5:18 p.m. UTC 
  Set DEBUG to 0 by default instead of undefined by default.  The
generated code is unchanged.

Alternatively we could just remove the debug code since we don't
normally do something like this for any of the other functions.  I
don't have a strong opinion about it either way.

Siddhesh

	* time/mktime.c: Define DEBUG to 0.

---
 time/mktime.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Roland McGrath June 26, 2014, 5:19 p.m. UTC | #1 
How close are we to mktime being harmonized with gnulib already?
If we're already there, then please coordinate this change with them.
diff mbox

Patch

diff --git a/time/mktime.c b/time/mktime.c
index 963e4b9..a4a2d58 100644
--- a/time/mktime.c
+++ b/time/mktime.c
@@ -17,9 +17,9 @@ 
    License along with the GNU C Library; if not, see
    <http://www.gnu.org/licenses/>.  */
 
-/* Define this to have a standalone program to test this implementation of
+/* Define this to 1 to have a standalone program to test this implementation of
    mktime.  */
-/* #define DEBUG 1 */
+#define DEBUG 0
 
 #ifndef _LIBC
 # include <config.h>