Possible problem with timemodule.c [1.5.2c1]

[Andy Dustman]

[I decided not to bug Guido directly with this...]

My compile is completely clean except for Modules/timemodule.c:

./timemodule.c: In function `time_strptime':
./timemodule.c:429: warning: assignment makes pointer from integer without
a cast

This is in time_strptime(), naturally. The code immediately before this
is:

#ifdef HAVE_STRPTIME
/* extern char *strptime(); /* Enable this if it's not declared in <time.h> */

On Linux, strptime() IS declared in <time.h>. However, I find nothing in
timemodule.c that would cause <time.h> to be included for Linux. configure
does find strptime and does cause HAVE_STRPTIME to be defined in config.h.

This is unlikely to be a "showstopper", but I thought I would point it
out. This may simply be a Linux (RedHat 5.2) problem. The more I look at
<time.h>, the more I lean towards thie conclusion. The prototype for
strptime() is not defined unless __USE_XOPEN is defined. The solution,
however, is not obvious.

-- 
andy dustman  | programmer/analyst |  comstar communications corporation
telephone: 770.485.6025 / 706.549.7689 | icq: 32922760 | pgp: 0xc72f3f1d