diff options
| author | Tim Peters <tim.peters@gmail.com> | 2001-08-29 21:37:10 (GMT) |
|---|---|---|
| committer | Tim Peters <tim.peters@gmail.com> | 2001-08-29 21:37:10 (GMT) |
| commit | 79248aa1e4a8f6510b1f0ef95dc9592d51e16d6c (patch) | |
| tree | 8bdd40a12e1a9c279a7cd0741333a4a1e00c2bf6 /Python/thread_nt.h | |
| parent | 1936745668f44dd121b50fef31a70efc07cc4ebf (diff) | |
| download | cpython-79248aa1e4a8f6510b1f0ef95dc9592d51e16d6c.zip cpython-79248aa1e4a8f6510b1f0ef95dc9592d51e16d6c.tar.gz cpython-79248aa1e4a8f6510b1f0ef95dc9592d51e16d6c.tar.bz2 | |
SF bug [#456252] Python should never stomp on [u]intptr_t.
pyport.h: typedef a new Py_intptr_t type.
DELICATE ASSUMPTION: That HAVE_UINTPTR_T implies intptr_t is
available as well as uintptr_t. If that turns out not to be
true, things must get uglier (C99 wants both, so I think it's
an assumption we're *likely* to get away with).
thread_nt.h, PyThread_start_new_thread: MS _beginthread is documented
as returning unsigned long; no idea why uintptr_t was being used.
Others: Always use Py_[u]intptr_t, never [u]intptr_t directly.
Diffstat (limited to 'Python/thread_nt.h')
| -rw-r--r-- | Python/thread_nt.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Python/thread_nt.h b/Python/thread_nt.h index f32304f..1d27627 100644 --- a/Python/thread_nt.h +++ b/Python/thread_nt.h @@ -152,7 +152,7 @@ static void PyThread__init_thread(void) */ int PyThread_start_new_thread(void (*func)(void *), void *arg) { - uintptr_t rv; + unsigned long rv; int success = 0; dprintf(("%ld: PyThread_start_new_thread called\n", PyThread_get_thread_ident())); @@ -161,7 +161,7 @@ int PyThread_start_new_thread(void (*func)(void *), void *arg) rv = _beginthread(func, 0, arg); /* use default stack size */ - if (rv != -1) { + if (rv != (unsigned long)-1) { success = 1; dprintf(("%ld: PyThread_start_new_thread succeeded: %p\n", PyThread_get_thread_ident(), rv)); } |