diff options
author | Tim Peters <tim.peters@gmail.com> | 2000-07-23 19:28:35 (GMT) |
---|---|---|
committer | Tim Peters <tim.peters@gmail.com> | 2000-07-23 19:28:35 (GMT) |
commit | 8315ea579046727bb234dc4692ca28c8514eacbe (patch) | |
tree | d1b2e26eb953b19ee6c753e4842f604becf76fc1 /Include/pyport.h | |
parent | e21107145aed5b9bfeeb2f2d5f76944288c1dc07 (diff) | |
download | cpython-8315ea579046727bb234dc4692ca28c8514eacbe.zip cpython-8315ea579046727bb234dc4692ca28c8514eacbe.tar.gz cpython-8315ea579046727bb234dc4692ca28c8514eacbe.tar.bz2 |
Included assert.h in Python.h -- it's absurd that this basic tool of
good C practice hasn't been available to everything all along.
Added Py_SAFE_DOWNCAST(VALUE, WIDE, NARROW) macro to pyport.h; this
just casts VALUE from type WIDE to type NARROW, but assert-fails if
Py_DEBUG is defined and info is lost due to casting.
Replaced a line in Fredrik's fix to marshal.c to use the new macro.
Diffstat (limited to 'Include/pyport.h')
-rw-r--r-- | Include/pyport.h | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/Include/pyport.h b/Include/pyport.h index 4faba7c..ed8a28e 100644 --- a/Include/pyport.h +++ b/Include/pyport.h @@ -27,6 +27,10 @@ RETSIGTYPE Meaning: Expands to void or int, depending on what the platform wants signal handlers to return. Note that only void is ANSI! Used in: Py_RETURN_FROM_SIGNAL_HANDLER + +Py_DEBUG +Meaning: Extra checks compiled in for debug mode. +Used in: Py_SAFE_DOWNCAST **************************************************************************/ @@ -74,6 +78,19 @@ extern "C" { #define Py_RETURN_FROM_SIGNAL_HANDLER(VALUE) \ Py_FORCE_EXPANSION(RETSIGTYPE) ## _PySIGRETURN(VALUE) +/* Py_SAFE_DOWNCAST(VALUE, WIDE, NARROW) + * Cast VALUE to type NARROW from type WIDE. In Py_DEBUG mode, this + * assert-fails if any information is lost. + * Caution: + * VALUE may be evaluated more than once. + */ +#ifdef Py_DEBUG +#define Py_SAFE_DOWNCAST(VALUE, WIDE, NARROW) \ + (assert((WIDE)(NARROW)(VALUE) == (VALUE)), (NARROW)(VALUE)) +#else +#define Py_SAFE_DOWNCAST(VALUE, WIDE, NARROW) (NARROW)(VALUE) +#endif + #ifdef __cplusplus } #endif |