From 73000498674ea2a72204c0e137c45bdfa7afd524 Mon Sep 17 00:00:00 2001 From: Bruce Hill Date: Sun, 4 Aug 2024 17:31:33 -0400 Subject: Switch to TABLE_INCREF --- builtins/table.c | 6 ------ builtins/table.h | 1 + compile.c | 2 +- 3 files changed, 2 insertions(+), 7 deletions(-) diff --git a/builtins/table.c b/builtins/table.c index e0c94593..f3f97ec3 100644 --- a/builtins/table.c +++ b/builtins/table.c @@ -107,12 +107,6 @@ static void maybe_copy_on_write(table_t *t, const TypeInfo *type) } } -public void Table$mark_copy_on_write(table_t *t) -{ - ARRAY_INCREF(t->entries); - if (t->bucket_info) t->bucket_info->data_refcount = TABLE_MAX_DATA_REFCOUNT; -} - // Return address of value or NULL public void *Table$get_raw(table_t t, const void *key, const TypeInfo *type) { diff --git a/builtins/table.h b/builtins/table.h index 82e12286..e136f460 100644 --- a/builtins/table.h +++ b/builtins/table.h @@ -41,6 +41,7 @@ void Table$remove(table_t *t, const void *key, const TypeInfo *type); void Table$clear(table_t *t); table_t Table$sorted(table_t t, const TypeInfo *type); void Table$mark_copy_on_write(table_t *t); +#define TABLE_INCREF(t) ({ ARRAY_INCREF((t).entries); if ((t).bucket_info) (t).bucket_info.data_refcount += ((t).bucket_info.data_refcount < TABLE_MAX_DATA_REFCOUNT); }) int32_t Table$compare(const table_t *x, const table_t *y, const TypeInfo *type); bool Table$equal(const table_t *x, const table_t *y, const TypeInfo *type); uint32_t Table$hash(const table_t *t, const TypeInfo *type); diff --git a/compile.c b/compile.c index 1c258777..9f91161c 100644 --- a/compile.c +++ b/compile.c @@ -2172,7 +2172,7 @@ CORD compile(env_t *env, ast_t *ast) if (ptr->pointed->tag == ArrayType) { return CORD_all("({ array_t *arr = ", compile(env, indexing->indexed), "; ARRAY_INCREF(*arr); *arr; })"); } else if (ptr->pointed->tag == TableType) { - return CORD_all("({ table_t *t = ", compile(env, indexing->indexed), "; Table$mark_copy_on_write(t); *t; })"); + return CORD_all("({ table_t *t = ", compile(env, indexing->indexed), "; TABLE_INCREF(*t); *t; })"); } else { return CORD_all("*(", compile(env, indexing->indexed), ")"); } -- cgit v1.2.3