minor fix for refcount/flags to be unsigned
authorKevin P. Fleming <kpfleming@digium.com>
Sat, 14 May 2005 23:32:31 +0000 (23:32 +0000)
committerKevin P. Fleming <kpfleming@digium.com>
Sat, 14 May 2005 23:32:31 +0000 (23:32 +0000)
explicitly test refcount as a logical value for builtin_expect (thanks to rizzo for the cluebat!)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@5651 65c4cc65-6c06-0410-ace0-fbb531ad65f3

include/asterisk/astobj.h

index ca18cf5..ac1b280 100755 (executable)
@@ -129,8 +129,8 @@ extern "C" {
  */
 #define ASTOBJ_COMPONENTS_NOLOCK_FULL(type,namelen,hashes) \
        char name[namelen]; \
-       int refcount; \
-       int objflags; \
+       unsigned int refcount; \
+       unsigned int objflags; \
        __ASTOBJ_HASH(type,hashes)
        
 /*! \brief Add ASTOBJ components to a struct (without locking support).
@@ -213,7 +213,7 @@ extern "C" {
        do { \
                int newcount = 0; \
                ASTOBJ_WRLOCK(object); \
-               if (__builtin_expect((object)->refcount, 1)) \
+               if (__builtin_expect((object)->refcount > 0, 1)) \
                        newcount = --((object)->refcount); \
                else \
                        ast_log(LOG_WARNING, "Unreferencing unreferenced (object)!\n"); \