astobj2: Declare private variable data_size for AO2_DEBUG only.
authorCorey Farrell <git@cfware.com>
Sat, 29 Oct 2016 15:19:53 +0000 (11:19 -0400)
committerCorey Farrell <git@cfware.com>
Sat, 29 Oct 2016 15:19:53 +0000 (11:19 -0400)
Every ao2 object contains storage for a private variable data_size,
though the value is never read if AO2_DEBUG is disabled.  This change
makes the variable conditional, reducing memory usage.

ASTERISK-26524 #close

Change-Id: If859929e507676ebc58b0f84247a4231e11da07f

main/astobj2.c

index 114f260..c8e48bb 100644 (file)
@@ -52,8 +52,10 @@ struct __priv_data {
        ao2_destructor_fn destructor_fn;
        /*! This field is used for astobj2 and ao2_weakproxy objects to reference each other */
        void *weakptr;
+#if defined(AO2_DEBUG)
        /*! User data size for stats */
        size_t data_size;
+#endif
        /*! The ao2 object option flags */
        uint32_t options;
        /*! magic number.  This is used to verify that a pointer passed in is a
@@ -669,11 +671,11 @@ static void *internal_ao2_alloc(size_t data_size, ao2_destructor_fn destructor_f
        /* Initialize common ao2 values. */
        obj->priv_data.ref_counter = 1;
        obj->priv_data.destructor_fn = destructor_fn;   /* can be NULL */
-       obj->priv_data.data_size = data_size;
        obj->priv_data.options = options;
        obj->priv_data.magic = AO2_MAGIC;
 
 #ifdef AO2_DEBUG
+       obj->priv_data.data_size = data_size;
        ast_atomic_fetchadd_int(&ao2.total_objects, 1);
        ast_atomic_fetchadd_int(&ao2.total_mem, data_size);
        ast_atomic_fetchadd_int(&ao2.total_refs, 1);