diff options
author | jan.nijtmans <jan.nijtmans@noemail.net> | 2012-12-13 11:00:13 (GMT) |
---|---|---|
committer | jan.nijtmans <jan.nijtmans@noemail.net> | 2012-12-13 11:00:13 (GMT) |
commit | 33bc3f551d75032142dc91b452f89a751f5c26e9 (patch) | |
tree | e9f1de6715632fe894e2c0886b56197f12dedf13 /generic | |
parent | 6b9edcab8fd8e7e48dc9b4eb4d51c95368de8156 (diff) | |
parent | 17298e8dd0217a61629194c735959152f70c93c3 (diff) | |
download | tcl-33bc3f551d75032142dc91b452f89a751f5c26e9.zip tcl-33bc3f551d75032142dc91b452f89a751f5c26e9.tar.gz tcl-33bc3f551d75032142dc91b452f89a751f5c26e9.tar.bz2 |
Fix Tcl_DecrRefCount macro, not to refer to its objPtr parameter twice.
FossilOrigin-Name: 4658460021749bf2e225d54f7f8103c03c147a0c
Diffstat (limited to 'generic')
-rw-r--r-- | generic/tcl.h | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/generic/tcl.h b/generic/tcl.h index e921ec5..b253db7 100644 --- a/generic/tcl.h +++ b/generic/tcl.h @@ -798,10 +798,7 @@ typedef struct Tcl_Obj { * whether an object is shared (i.e. has reference count > 1). Note: clients * should use Tcl_DecrRefCount() when they are finished using an object, and * should never call TclFreeObj() directly. TclFreeObj() is only defined and - * made public in tcl.h to support Tcl_DecrRefCount's macro definition. Note - * also that Tcl_DecrRefCount() refers to the parameter "obj" twice. This - * means that you should avoid calling it with an expression that is expensive - * to compute or has side effects. + * made public in tcl.h to support Tcl_DecrRefCount's macro definition. */ void Tcl_IncrRefCount _ANSI_ARGS_((Tcl_Obj *objPtr)); @@ -2315,7 +2312,12 @@ EXTERN void Tcl_GetMemoryInfo _ANSI_ARGS_((Tcl_DString *dsPtr)); * http://c2.com/cgi/wiki?TrivialDoWhileLoop */ # define Tcl_DecrRefCount(objPtr) \ - do { if (--(objPtr)->refCount <= 0) TclFreeObj(objPtr); } while(0) + do { \ + Tcl_Obj *obj = (objPtr); \ + if ((obj)->refCount-- < 2) { \ + TclFreeObj(obj); \ + } \ + } while(0) # define Tcl_IsShared(objPtr) \ ((objPtr)->refCount > 1) #endif |