BLI_memarena: remove use of BLI_linklist
authorCampbell Barton <ideasman42@gmail.com>
Tue, 12 Feb 2019 05:21:27 +0000 (16:21 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Tue, 12 Feb 2019 05:28:10 +0000 (16:28 +1100)
Preprare for using memarena w/ makesdna, BLI_linklist depends on
BLI_mempool. Since linked list use in here is simple, do it inline.

source/blender/blenlib/intern/BLI_memarena.c

index 12b4b53..7126008 100644 (file)
 
 #include "BLI_utildefines.h"
 #include "BLI_memarena.h"
-#include "BLI_linklist.h"
 #include "BLI_strict_flags.h"
 
 #ifdef WITH_MEM_VALGRIND
 #  include "valgrind/memcheck.h"
 #endif
 
+struct MemBuf {
+       struct MemBuf *next;
+       uchar data[0];
+};
+
 struct MemArena {
        unsigned char *curbuf;
        const char *name;
-       LinkNode *bufs;
+       struct MemBuf *bufs;
 
        size_t bufsize, cursize;
        size_t align;
@@ -54,6 +58,15 @@ struct MemArena {
        bool use_calloc;
 };
 
+static void memarena_buf_free_all(struct MemBuf *mb)
+{
+       while (mb != NULL) {
+               struct MemBuf *mb_next = mb->next;
+               MEM_freeN(mb);
+               mb = mb_next;
+       }
+}
+
 MemArena *BLI_memarena_new(const size_t bufsize, const char *name)
 {
        MemArena *ma = MEM_callocN(sizeof(*ma), "memarena");
@@ -86,8 +99,7 @@ void BLI_memarena_use_align(struct MemArena *ma, const size_t align)
 
 void BLI_memarena_free(MemArena *ma)
 {
-       BLI_linklist_freeN(ma->bufs);
-
+       memarena_buf_free_all(ma->bufs);
 #ifdef WITH_MEM_VALGRIND
        VALGRIND_DESTROY_MEMPOOL(ma);
 #endif
@@ -124,8 +136,11 @@ void *BLI_memarena_alloc(MemArena *ma, size_t size)
                        ma->cursize = ma->bufsize;
                }
 
-               ma->curbuf = (ma->use_calloc ? MEM_callocN : MEM_mallocN)(ma->cursize, ma->name);
-               BLI_linklist_prepend(&ma->bufs, ma->curbuf);
+               struct MemBuf *mb = (ma->use_calloc ? MEM_callocN : MEM_mallocN)(sizeof(*mb) + ma->cursize, ma->name);
+               ma->curbuf = mb->data;
+               mb->next = ma->bufs;
+               ma->bufs = mb;
+
                memarena_curbuf_align(ma);
        }
 
@@ -164,12 +179,12 @@ void BLI_memarena_clear(MemArena *ma)
                size_t curbuf_used;
 
                if (ma->bufs->next) {
-                       BLI_linklist_freeN(ma->bufs->next);
+                       memarena_buf_free_all(ma->bufs->next);
                        ma->bufs->next = NULL;
                }
 
                curbuf_prev = ma->curbuf;
-               ma->curbuf = ma->bufs->link;
+               ma->curbuf = ma->bufs->data;
                memarena_curbuf_align(ma);
 
                /* restore to original size */