[Python-checkins] cpython (2.7): Issue #18672: Fixed format specifiers for Py_ssize_t in debugging output in

Thu Sep 5 17:05:43 CEST 2013

http://hg.python.org/cpython/rev/603b4d593758
changeset: 85550:603b4d593758
branch: 2.7
parent: 85543:3163a01ac074
user: Serhiy Storchaka <storchaka at gmail.com>
date: Thu Sep 05 18:02:57 2013 +0300
summary:
 Issue #18672: Fixed format specifiers for Py_ssize_t in debugging output in
the _sre moduel.
files:
 Misc/NEWS | 3 +++
 Modules/_sre.c | 36 ++++++++++++++++++++++--------------
 2 files changed, 25 insertions(+), 14 deletions(-)

diff --git a/Misc/NEWS b/Misc/NEWS
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -32,6 +32,9 @@
 Library
 -------
 
+- Issue #18672: Fixed format specifiers for Py_ssize_t in debugging output in
+ the _sre moduel.
+
 - Issue #18830: inspect.getclasstree() no more produces duplicated entries even
 when input list contains duplicates.
 
diff --git a/Modules/_sre.c b/Modules/_sre.c
--- a/Modules/_sre.c
+++ b/Modules/_sre.c
@@ -272,7 +272,7 @@
 if (cursize < minsize) {
 void* stack;
 cursize = minsize+minsize/4+1024;
- TRACE(("allocate/grow stack %d\n", cursize));
+ TRACE(("allocate/grow stack %" PY_FORMAT_SIZE_T "d\n", cursize));
 stack = PyMem_REALLOC(state->data_stack, cursize);
 if (!stack) {
 data_stack_dealloc(state);
@@ -592,12 +592,13 @@
 if (!i)
 break;
 }
- TRACE(("|%p|%p|COUNT %d\n", pattern, ptr,
+ TRACE(("|%p|%p|COUNT %" PY_FORMAT_SIZE_T "d\n", pattern, ptr,
 (SRE_CHAR*) state->ptr - ptr));
 return (SRE_CHAR*) state->ptr - ptr;
 }
 
- TRACE(("|%p|%p|COUNT %d\n", pattern, ptr, ptr - (SRE_CHAR*) state->ptr));
+ TRACE(("|%p|%p|COUNT %" PY_FORMAT_SIZE_T "d\n", pattern, ptr,
+ ptr - (SRE_CHAR*) state->ptr));
 return ptr - (SRE_CHAR*) state->ptr;
 }
 
@@ -684,7 +685,8 @@
 #define DATA_STACK_ALLOC(state, type, ptr) \
 do { \
 alloc_pos = state->data_stack_base; \
- TRACE(("allocating %s in %d (%d)\n", \
+ TRACE(("allocating %s in %" PY_FORMAT_SIZE_T "d " \
+ "(%" PY_FORMAT_SIZE_T "d)\n", \
 SFY(type), alloc_pos, sizeof(type))); \
 if (sizeof(type) > state->data_stack_size - alloc_pos) { \
 int j = data_stack_grow(state, sizeof(type)); \
@@ -698,13 +700,14 @@
 
 #define DATA_STACK_LOOKUP_AT(state, type, ptr, pos) \
 do { \
- TRACE(("looking up %s at %d\n", SFY(type), pos)); \
+ TRACE(("looking up %s at %" PY_FORMAT_SIZE_T "d\n", SFY(type), pos)); \
 ptr = (type*)(state->data_stack+pos); \
 } while (0)
 
 #define DATA_STACK_PUSH(state, data, size) \
 do { \
- TRACE(("copy data in %p to %d (%d)\n", \
+ TRACE(("copy data in %p to %" PY_FORMAT_SIZE_T "d " \
+ "(%" PY_FORMAT_SIZE_T "d)\n", \
 data, state->data_stack_base, size)); \
 if (size > state->data_stack_size - state->data_stack_base) { \
 int j = data_stack_grow(state, size); \
@@ -718,7 +721,8 @@
 
 #define DATA_STACK_POP(state, data, size, discard) \
 do { \
- TRACE(("copy data to %p from %d (%d)\n", \
+ TRACE(("copy data to %p from %" PY_FORMAT_SIZE_T "d " \
+ "(%" PY_FORMAT_SIZE_T "d)\n", \
 data, state->data_stack_base-size, size)); \
 memcpy(data, state->data_stack+state->data_stack_base-size, size); \
 if (discard) \
@@ -727,7 +731,8 @@
 
 #define DATA_STACK_POP_DISCARD(state, size) \
 do { \
- TRACE(("discard data from %d (%d)\n", \
+ TRACE(("discard data from %" PY_FORMAT_SIZE_T "d " \
+ "(%" PY_FORMAT_SIZE_T "d)\n", \
 state->data_stack_base-size, size)); \
 state->data_stack_base -= size; \
 } while(0)
@@ -831,8 +836,9 @@
 /* optimization info block */
 /* <INFO> <1=skip> <2=flags> <3=min> ... */
 if (ctx->pattern[3] && (end - ctx->ptr) < ctx->pattern[3]) {
- TRACE(("reject (got %d chars, need %d)\n",
- (end - ctx->ptr), ctx->pattern[3]));
+ TRACE(("reject (got %" PY_FORMAT_SIZE_T "d chars, "
+ "need %" PY_FORMAT_SIZE_T "d)\n",
+ (end - ctx->ptr), (Py_ssize_t) ctx->pattern[3]));
 RETURN_FAILURE;
 }
 ctx->pattern += ctx->pattern[1] + 1;
@@ -1207,7 +1213,7 @@
 
 ctx->count = ctx->u.rep->count+1;
 
- TRACE(("|%p|%p|MAX_UNTIL %d\n", ctx->pattern,
+ TRACE(("|%p|%p|MAX_UNTIL %" PY_FORMAT_SIZE_T "d\n", ctx->pattern,
 ctx->ptr, ctx->count));
 
 if (ctx->count < (Py_ssize_t) ctx->u.rep->pattern[1]) {
@@ -1270,7 +1276,7 @@
 
 ctx->count = ctx->u.rep->count+1;
 
- TRACE(("|%p|%p|MIN_UNTIL %d %p\n", ctx->pattern,
+ TRACE(("|%p|%p|MIN_UNTIL %" PY_FORMAT_SIZE_T "d %p\n", ctx->pattern,
 ctx->ptr, ctx->count, ctx->u.rep->pattern));
 
 if (ctx->count < (Py_ssize_t) ctx->u.rep->pattern[1]) {
@@ -1483,7 +1489,8 @@
 TRACE(("|%p|%p|JUMP_ASSERT_NOT\n", ctx->pattern, ctx->ptr));
 goto jump_assert_not;
 case JUMP_NONE:
- TRACE(("|%p|%p|RETURN %d\n", ctx->pattern, ctx->ptr, ret));
+ TRACE(("|%p|%p|RETURN %" PY_FORMAT_SIZE_T "d\n", ctx->pattern,
+ ctx->ptr, ret));
 break;
 }
 
@@ -1532,7 +1539,8 @@
 pattern += 1 + pattern[1];
 }
 
- TRACE(("prefix = %p %d %d\n", prefix, prefix_len, prefix_skip));
+ TRACE(("prefix = %p %" PY_FORMAT_SIZE_T "d %" PY_FORMAT_SIZE_T "d\n",
+ prefix, prefix_len, prefix_skip));
 TRACE(("charset = %p\n", charset));
 
 #if defined(USE_FAST_SEARCH)
-- 
Repository URL: http://hg.python.org/cpython
