From: Sergey Kaplun via Tarantool-patches <tarantool-patches@dev.tarantool.org> To: Maxim Kokryashkin <m.kokryashkin@tarantool.org>, Sergey Bronnikov <sergeyb@tarantool.org> Cc: tarantool-patches@dev.tarantool.org Subject: [Tarantool-patches] [PATCH luajit] Drop unused function wrapper. Date: Thu, 19 Sep 2024 14:19:47 +0300 [thread overview] Message-ID: <20240919111947.12653-1-skaplun@tarantool.org> (raw) In-Reply-To: <20240902125421.16727-1-skaplun@tarantool.org> From: Mike Pall <mike> (cherry picked from commit 87ae18af97fd4de790bb6c476b212e047689cc93) This patch is a follow-up to the previous commit. Since the rehashing of the finalizer table is gone, `lj_tab_rehash()` becomes unused and can be dropped. Sergey Kaplun: * added the description for the patch Part of tarantool/tarantool#10199 Follows up tarantool/tarantool#10290 --- src/lj_tab.c | 7 ------- src/lj_tab.h | 3 --- 2 files changed, 10 deletions(-) diff --git a/src/lj_tab.c b/src/lj_tab.c index 1d6a4b7f..c99c6521 100644 --- a/src/lj_tab.c +++ b/src/lj_tab.c @@ -388,13 +388,6 @@ static void rehashtab(lua_State *L, GCtab *t, cTValue *ek) lj_tab_resize(L, t, asize, hsize2hbits(total)); } -#if LJ_HASFFI -void lj_tab_rehash(lua_State *L, GCtab *t) -{ - rehashtab(L, t, niltv(L)); -} -#endif - void lj_tab_reasize(lua_State *L, GCtab *t, uint32_t nasize) { lj_tab_resize(L, t, nasize+1, t->hmask > 0 ? lj_fls(t->hmask)+1 : 0); diff --git a/src/lj_tab.h b/src/lj_tab.h index 71e34945..8575e7bd 100644 --- a/src/lj_tab.h +++ b/src/lj_tab.h @@ -41,9 +41,6 @@ LJ_FUNC GCtab * LJ_FASTCALL lj_tab_new1(lua_State *L, uint32_t ahsize); LJ_FUNCA GCtab * LJ_FASTCALL lj_tab_dup(lua_State *L, const GCtab *kt); LJ_FUNC void LJ_FASTCALL lj_tab_clear(GCtab *t); LJ_FUNC void LJ_FASTCALL lj_tab_free(global_State *g, GCtab *t); -#if LJ_HASFFI -LJ_FUNC void lj_tab_rehash(lua_State *L, GCtab *t); -#endif LJ_FUNC void lj_tab_resize(lua_State *L, GCtab *t, uint32_t asize, uint32_t hbits); LJ_FUNCA void lj_tab_reasize(lua_State *L, GCtab *t, uint32_t nasize); -- 2.46.0
next prev parent reply other threads:[~2024-09-19 11:20 UTC|newest] Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top 2024-09-02 12:54 [Tarantool-patches] [PATCH luajit] FFI: Drop finalizer table rehash after GC cycle Sergey Kaplun via Tarantool-patches 2024-09-06 14:42 ` Sergey Bronnikov via Tarantool-patches 2024-09-07 5:15 ` Sergey Kaplun via Tarantool-patches 2024-09-09 13:37 ` Sergey Bronnikov via Tarantool-patches 2024-09-09 14:15 ` Sergey Kaplun via Tarantool-patches 2024-09-19 9:02 ` Maxim Kokryashkin via Tarantool-patches 2024-09-19 11:19 ` Sergey Kaplun via Tarantool-patches [this message] 2024-09-19 11:30 ` [Tarantool-patches] [PATCH luajit] Drop unused function wrapper Maxim Kokryashkin via Tarantool-patches 2024-09-19 18:53 ` Sergey Bronnikov via Tarantool-patches
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20240919111947.12653-1-skaplun@tarantool.org \ --to=tarantool-patches@dev.tarantool.org \ --cc=m.kokryashkin@tarantool.org \ --cc=sergeyb@tarantool.org \ --cc=skaplun@tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH luajit] Drop unused function wrapper.' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox