summaryrefslogtreecommitdiffstats
path: root/autoload.c
diff options
context:
space:
mode:
authorKaz Kylheku <kaz@kylheku.com>2023-05-02 20:59:49 -0700
committerKaz Kylheku <kaz@kylheku.com>2023-05-02 20:59:49 -0700
commita6b4df71dc35dcf9c923cbf3967155331bcc98fc (patch)
tree096c6400abe87d25793c4c8a1063fe78ce80f1f8 /autoload.c
parenta5dd1b3eb409116ed2249205ead2322782641595 (diff)
downloadtxr-a6b4df71dc35dcf9c923cbf3967155331bcc98fc.tar.gz
txr-a6b4df71dc35dcf9c923cbf3967155331bcc98fc.tar.bz2
txr-a6b4df71dc35dcf9c923cbf3967155331bcc98fc.zip
gc: use single allocation for prot_array.
* gc.c (prot_array): Add self pointer; arr member becomes flexible array. (prot_array_mark): We now check the handle itself for null, because the whole thing is freed. (prot_array_free): Function removed. (prot_array_ops): Wire cobj_destroy_free_op in place of prot_array_free. This fixes a memory leak because prot_array_free was not freeing the handle, only the array. (gc_prot_array_alloc): Fix to allocate everything in one swoop and store the self-pointer in the named member rather than arr[-1]. The self argument is not required; we drop it. The size argument cannot be anywhere near INT_PTR_MAX, because such an array wouldn't fit into virtual memory, so it is always safe to add a small value to the size. (prot_array_free): Obtain the self-pointer, and free the handle, replacing it with a null pointer. * gc.h (gc_prot_array_alloc): Declaration updated. * lib.c (ssort_vec): Don't pass self to gc_prot_array_alloc. * lib.h (container): New macro.
Diffstat (limited to 'autoload.c')
0 files changed, 0 insertions, 0 deletions