lib: talloc: Call talloc_chunk_from_ptr() less often in __talloc_with_prefix()
authorAndrew Bartlett <abartlet@samba.org>
Wed, 29 Jun 2016 22:46:37 +0000 (15:46 -0700)
committerJeremy Allison <jra@samba.org>
Sun, 3 Jul 2016 12:26:17 +0000 (14:26 +0200)
Rename 'ptc' pointer to parent as it's re-used as
that name later in the function.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
lib/talloc/talloc.c

index 26cecee..007b7a4 100644 (file)
@@ -637,6 +637,7 @@ static inline void *__talloc_with_prefix(const void *context,
        struct talloc_chunk *tc = NULL;
        struct talloc_memlimit *limit = NULL;
        size_t total_len = TC_HDR_SIZE + size + prefix_len;
+       struct talloc_chunk *parent = NULL;
 
        if (unlikely(context == NULL)) {
                context = null_context;
@@ -650,14 +651,14 @@ static inline void *__talloc_with_prefix(const void *context,
                return NULL;
        }
 
-       if (context != NULL) {
-               struct talloc_chunk *ptc = talloc_chunk_from_ptr(context);
+       if (likely(context != NULL)) {
+               parent = talloc_chunk_from_ptr(context);
 
-               if (ptc->limit != NULL) {
-                       limit = ptc->limit;
+               if (parent->limit != NULL) {
+                       limit = parent->limit;
                }
 
-               tc = tc_alloc_pool(ptc, TC_HDR_SIZE+size, prefix_len);
+               tc = tc_alloc_pool(parent, TC_HDR_SIZE+size, prefix_len);
        }
 
        if (tc == NULL) {
@@ -689,9 +690,7 @@ static inline void *__talloc_with_prefix(const void *context,
        tc->name = NULL;
        tc->refs = NULL;
 
-       if (likely(context)) {
-               struct talloc_chunk *parent = talloc_chunk_from_ptr(context);
-
+       if (likely(context != NULL)) {
                if (parent->child) {
                        parent->child->parent = NULL;
                        tc->next = parent->child;