diff options
| author | Jonathan Bradley <jcb@pikum.xyz> | 2025-10-30 16:18:07 -0400 |
|---|---|---|
| committer | Jonathan Bradley <jcb@pikum.xyz> | 2025-10-30 16:18:07 -0400 |
| commit | 3f788084e9a7efec681f6770516942084fc35235 (patch) | |
| tree | ea9be12ac92959b855d1ae7e2025fe137471f41a /src | |
| parent | 59467da83d60e7bcd92cdb60f08f9d331f6ee537 (diff) | |
pke: refactor FontRender to pk_bkt_arr
Diffstat (limited to 'src')
| -rw-r--r-- | src/font.cpp | 151 | ||||
| -rw-r--r-- | src/font.hpp | 28 | ||||
| -rw-r--r-- | src/plugins.cpp | 6 | ||||
| -rw-r--r-- | src/serialization-font.cpp | 2 | ||||
| -rw-r--r-- | src/serialization-static-ui.cpp | 10 | ||||
| -rw-r--r-- | src/serialization.cpp | 9 | ||||
| -rw-r--r-- | src/static-ui.cpp | 18 | ||||
| -rw-r--r-- | src/static-ui.hpp | 4 |
8 files changed, 104 insertions, 124 deletions
diff --git a/src/font.cpp b/src/font.cpp index 9ca42c5..496c052 100644 --- a/src/font.cpp +++ b/src/font.cpp @@ -29,7 +29,6 @@ const char *PKE_PROJECT_FONT_SPACING_UNDERLINE_Y = "spacing_underline_y: "; const char *PKE_PROJECT_FONT_SPACING_UNDERLINE_THICKNESS = "spacing_underline_thickness: "; TypeSafeInt_B(FontTypeIndex); -TypeSafeInt_B(FontRenderIndex); TypeSafeInt_B(FONT_GLYPH_CHAR_FLAG); TypeSafeInt_B(FONT_RENDER_SURFACE_AREA_TYPE_FLAG); @@ -37,6 +36,7 @@ struct FontTypeData { FontType *arr_ft = nullptr; uint64_t unused_fts = 0xFFFFFFFFFFFFFFFF; FontTypeIndex n_ft{0}; + pk_membucket *bkt; } ftd; struct FontInstanceBufferItem { @@ -322,12 +322,13 @@ bool FontType_Inner_CalcTransforms(const FontType *ft, FontRender *fr, FontInsta void FontType_Init() { FontTypeIndex fti; + ftd.bkt = pk_mem_bucket_create(__FILE__, PK_MEM_DEFAULT_BUCKET_SIZE, PK_MEMBUCKET_FLAG_NONE); ftd.n_ft = FontTypeIndex{8}; - ftd.arr_ft = pk_new_arr<FontType>(8); + ftd.arr_ft = pk_new_arr<FontType>(8, ftd.bkt); for (FontTypeIndex_T i = 0; i < 8; ++i) { FontType *ft = &ftd.arr_ft[i]; ft->glyphs = nullptr; - ft->renders = nullptr; + new (&ft->renders) pk_bkt_arr_t<FontRender>{pk_bkt_arr_handle_MAX, ftd.bkt, ftd.bkt}; } union pke_asset_details ak_img_details { .texture = { @@ -359,7 +360,9 @@ void FontType_Teardown() { if ((ftd.unused_fts & (1llu << i)) != 0) continue; FontType_Unload(FontTypeIndex{(FontTypeIndex_T)i}); } - if (ftd.arr_ft != nullptr) pk_delete_arr<FontType>(ftd.arr_ft, (FontTypeIndex_T)ftd.n_ft); + if (ftd.arr_ft != nullptr) pk_delete_arr<FontType>(ftd.arr_ft, (FontTypeIndex_T)ftd.n_ft, ftd.bkt); + pk_mem_bucket_destroy(ftd.bkt); + ftd.bkt = nullptr; } // TODO - Memory Pressure @@ -369,10 +372,8 @@ void FontType_Teardown() { void FontType_Tick(double delta) { (void)delta; VkResult vkResult; - FontInstanceBufferItem *fibis = nullptr; size_t index; FontType *ft; - FontRender *fr; for (FontTypeIndex_T i = 0; i < (FontTypeIndex_T)ftd.n_ft; ++i) { if ((ftd.unused_fts & (1llu << i)) != 0) continue; @@ -380,24 +381,17 @@ void FontType_Tick(double delta) { index = 0; ft = &ftd.arr_ft[i]; - for (FontRenderIndex_T k = 0; k < (FontRenderIndex_T)ft->n_render; ++k) { - if ((ft->unused_frs & (1llu << k)) != 0) continue; - fr = &ft->renders[k]; + pk_bkt_arr_t<FontRender>::FN_Iter iter_fn; + iter_fn.func = [&ft, &index](FontRender *fr) { if (fr->isMarkedForRemoval == true) { - if (fr->text.reserved > 0) { - // 2025-04-16 - JCB - // not passing a specific bucket because pk_cstr doesn't store it. - // FontType_AddStringRender requires a `pk_cstr &&`, so we are the proper owner. - // It's up to us to free it. - pk_delete_arr<char>(fr->text.val, fr->text.reserved); - } - ft->unused_frs |= (1llu << k); - new (fr) FontRender{}; + if (fr->text.reserved > 0) { pk_delete_arr<char>(fr->text.val, fr->text.reserved); } + pk_bkt_arr_free_handle(&ft->renders, fr->fr_handle); ft->gr.should_update_instance_buffer = true; - continue; + return; } index += fr->n_glyphs; - } + }; + pk_bkt_arr_iterate(&ft->renders, iter_fn.invoke, &iter_fn); if (ft->isMarkedForRemoval == true) { FontType_Unload(FontTypeIndex{i}); @@ -408,22 +402,27 @@ void FontType_Tick(double delta) { continue; } + ft->bindings.instance_counter = index; if (index == 0) { - ft->bindings.instance_counter = index; continue; } ft->gr.should_update_instance_buffer = false; - fibis = pk_new_arr<FontInstanceBufferItem>(index); - ft->bindings.instance_counter = index; + pk_arr_t<FontInstanceBufferItem> fibis{}; + pk_arr_resize(&fibis, index); index = 0; - for (FontRenderIndex_T k = 0; k < (FontRenderIndex_T)ft->n_render; ++k) { - if ((ft->unused_frs & (1llu << k)) != 0) continue; - fr = &ft->renders[k]; + + iter_fn.func = [&ft, &index, &fibis](FontRender *fr) { + if (fr->isMarkedForRemoval == true) { + return; + } + pk_arr_resize(&fibis, index + fr->n_glyphs); if (FontType_Inner_CalcTransforms(ft, fr, &fibis[index])) { index += fr->n_glyphs; } - } + }; + pk_bkt_arr_iterate(&ft->renders, iter_fn.invoke, &iter_fn); + ft->bindings.instance_counter = index; // check recreate buffer if (ft->bindings.instance_buffer_max_count < index) { @@ -436,7 +435,7 @@ void FontType_Tick(double delta) { } pkvk_buffer_create_data create_data{}; - create_data.buffer_byte_length[0] = sizeof(FontInstanceBufferItem) * ft->bindings.instance_counter; + create_data.buffer_byte_length[0] = fibis.stride * fibis.next; create_data.n_buffers = 1; create_data.index_instance = 0; create_data.index_index = -1; @@ -450,7 +449,7 @@ void FontType_Tick(double delta) { } PKVK_TmpBufferDetails tmpBufferDetails{}; - PKVK_BeginBuffer(graphicsFamilyIndex, sizeof(FontInstanceBufferItem) * ft->bindings.instance_counter, tmpBufferDetails); + PKVK_BeginBuffer(graphicsFamilyIndex, fibis.stride * fibis.next, tmpBufferDetails); assert(tmpBufferDetails.buffer != VK_NULL_HANDLE); { VkCommandBufferBeginInfo vkCommandBufferBeginInfo; @@ -462,12 +461,12 @@ void FontType_Tick(double delta) { vkResult = vkBeginCommandBuffer(tmpBufferDetails.cmdBuffer, &vkCommandBufferBeginInfo); assert(vkResult == VK_SUCCESS); - memcpy(tmpBufferDetails.deviceData, fibis, sizeof(FontInstanceBufferItem) * ft->bindings.instance_counter); + memcpy(tmpBufferDetails.deviceData, fibis.data, fibis.stride * fibis.next); VkBufferCopy vk_buffer_copy{}; vk_buffer_copy.srcOffset = 0; vk_buffer_copy.dstOffset = 0; - vk_buffer_copy.size = sizeof(FontInstanceBufferItem) * ft->bindings.instance_counter; + vk_buffer_copy.size = fibis.stride * fibis.next; vkCmdCopyBuffer(tmpBufferDetails.cmdBuffer, tmpBufferDetails.buffer, ft->bindings.bd_instance.buffer, 1, &vk_buffer_copy); vkResult = vkEndCommandBuffer(tmpBufferDetails.cmdBuffer); @@ -489,7 +488,7 @@ void FontType_Tick(double delta) { assert(vkResult == VK_SUCCESS); } PKVK_EndBuffer(tmpBufferDetails); - pk_delete_arr<FontInstanceBufferItem>(fibis, ft->bindings.instance_counter); + pk_arr_reset(&fibis); } } @@ -653,7 +652,6 @@ FontType* FontType_GetFonts(uint64_t &idx_unused) { FontTypeIndex FontType_RegisterFont(pk_cstr title, AssetHandle fontTextureHandle, AssetHandle glyphsHandle, FontTypeMSDFSettings *msdf_settings, FontTypeSpacing *spacing) { VkResult vkResult; - constexpr VkDeviceSize startingGlyphCount = 4; const Asset *fontTexture = nullptr; const Asset *glyphs = nullptr; FontTypeIndex idx{0}; @@ -680,11 +678,11 @@ FontTypeIndex FontType_RegisterFont(pk_cstr title, AssetHandle fontTextureHandle throw "[font.cpp] out of font type slots"; } ftd.n_ft = FontTypeIndex{8}; - FontType *arr = pk_new_arr<FontType>((FontTypeIndex_T)ftd.n_ft); + FontType *arr = pk_new_arr<FontType>((FontTypeIndex_T)ftd.n_ft, ftd.bkt); if (idx > FontTypeIndex{0}) { memcpy(arr, ftd.arr_ft, sizeof(FontType) * (FontTypeIndex_T)idx); } - if (ftd.arr_ft != nullptr && ftd.arr_ft != CAFE_BABE(FontType)) pk_delete_arr<FontType>(ftd.arr_ft, (FontTypeIndex_T)idx); + if (ftd.arr_ft != nullptr && ftd.arr_ft != CAFE_BABE(FontType)) pk_delete_arr<FontType>(ftd.arr_ft, (FontTypeIndex_T)idx, ftd.bkt); ftd.arr_ft = arr; } @@ -695,9 +693,7 @@ FontTypeIndex FontType_RegisterFont(pk_cstr title, AssetHandle fontTextureHandle ft->title = title; ft->fontTextureAssetHandle = fontTextureHandle; ft->glyphDetailsAssetHandle = glyphsHandle; - ft->renders = pk_new_arr<FontRender>(startingGlyphCount); - ft->n_render = FontRenderIndex{startingGlyphCount}; - ft->unused_frs = 0xFFFFFFFFFFFFFFFF; + new (&ft->renders) pk_bkt_arr_t<FontRender>{pk_bkt_arr_handle_MAX, ftd.bkt, ftd.bkt}; ft->msdf_settings = *msdf_settings; ft->spacing = *spacing; @@ -832,15 +828,15 @@ void FontType_Unload(FontTypeIndex idx) { FontType *ft = &ftd.arr_ft[(FontTypeIndex_T)idx]; - // TODO specific bucket - if (ft->renders != nullptr) { - for (FontRenderIndex_T i = 0; i < (FontRenderIndex_T)ft->n_render; ++i) { - if ((ft->unused_frs & (1llu << i)) != 0) continue; - if (ft->renders[i].glyph_indices != nullptr) { - pk_delete_arr<uint32_t>(ft->renders[i].glyph_indices, ft->renders[i].n_glyphs); + if (ft->renders.head_r.b > 0 || ft->renders.head_r.i > 0) { + pk_bkt_arr_t<FontRender>::FN_Iter iter_fn{}; + iter_fn.func = [](FontRender *fr) { + if (fr->glyph_indices != nullptr) { + pk_delete_arr<uint32_t>(fr->glyph_indices, fr->n_glyphs, ftd.bkt); } - } - pk_delete_arr<FontRender>(ft->renders, (FontTypeIndex_T)ft->n_render); + }; + pk_bkt_arr_iterate(&ft->renders, iter_fn.invoke, &iter_fn); + pk_bkt_arr_teardown(&ft->renders); } if (ft->gr.vkDescriptorSet != VK_NULL_HANDLE && ft->gr.vkDescriptorPool != VK_NULL_HANDLE) { @@ -962,44 +958,27 @@ void FontType_cstr_to_unicode(const FontType &ft, pk_arr_t<uint32_t> &arr, const // However, the new buffer is exactly the length it needs to be and no greater. // Consider using a heuristic to allocate a buffer larger than needed. // At the time of writing, the only way to do this is to un-register text. -FontRenderHandle FontType_AddStringRender(FontTypeIndex idx_ft, const pk_cstr &&str, FontRenderSettings *settings, Entity_Base *parent, pk_uuid uuid) { +FontTypeRender FontType_AddStringRender(FontTypeIndex idx_ft, const pk_cstr &&str, FontRenderSettings *settings, Entity_Base *parent, pk_uuid uuid) { assert(settings != nullptr); FontType *ft = &ftd.arr_ft[(FontTypeIndex_T)idx_ft]; FontRender *fr; - FontRenderIndex_T fri; + FontTypeRender ret{}; uint32_t i, count; - FontRenderIndex idx_fr = FontRenderIndex{0}; - for (fri = 0; fri < (FontRenderIndex_T)ft->n_render; ++fri) { - if ((ft->unused_frs & (1llu << fri)) != 0) { - ft->unused_frs &= ~(1llu << fri); - idx_fr = FontRenderIndex{fri}; - break; - } - } + ret.index_ft = idx_ft; + ret.handle_fr = (FontRenderHandle)pk_bkt_arr_new_handle(&ft->renders); + fr = &ft->renders[ret.handle_fr]; - if (idx_fr >= ft->n_render) { - if (idx_fr >= FontRenderIndex{PKE_FONT_MAX_FONT_RENDERS}) { - throw "[font.cpp] out of FontRender slots"; - } - FontRenderIndex_T old_count{static_cast<FontRenderIndex_T>(ft->n_render)}; - ft->n_render += FontRenderIndex{8}; - FontRender *arr = pk_new_arr<FontRender>((FontRenderIndex_T)ft->n_render); - memcpy(arr, ft->renders, sizeof(FontRender) * old_count); - pk_delete_arr<FontRender>(ft->renders, old_count); - ft->renders = arr; - } - fr = &ft->renders[(FontRenderIndex_T)idx_fr]; new (fr) FontRender{}; fr->uuid = uuid; ECS_CreateEntity(fr, parent); - fr->fr_handle.index_ft = idx_ft; - fr->fr_handle.index_fr = idx_fr; + fr->index_ft = idx_ft; + fr->fr_handle = ret.handle_fr; fr->settings = *settings; fr->text = str; if (window == NULL) { - return fr->fr_handle; + return ret; } // insert new characters into tmp buffer @@ -1013,7 +992,7 @@ FontRenderHandle FontType_AddStringRender(FontTypeIndex idx_ft, const pk_cstr && // TODO specific bucket count = glyph_indices.next; fr->n_glyphs = count; - fr->glyph_indices = pk_new_arr<uint32_t>(count); + fr->glyph_indices = pk_new_arr<uint32_t>(count, ftd.bkt); for (i = 0; i < count; ++i) { fr->glyph_indices[i] = glyph_indices[i]; } @@ -1021,35 +1000,35 @@ FontRenderHandle FontType_AddStringRender(FontTypeIndex idx_ft, const pk_cstr && ft->gr.should_update_instance_buffer = true; - return fr->fr_handle; + return ret; } -FontRender *FontType_GetFontRender(FontRenderHandle frh) { +FontRender *FontType_GetFontRender(FontTypeRender frh) { if ((ftd.unused_fts & (1llu << (FontTypeIndex_T)frh.index_ft)) != 0) { return nullptr; } FontType *ft = &ftd.arr_ft[static_cast<FontTypeIndex_T>(frh.index_ft)]; - if ((ft->unused_frs & (1llu << (FontRenderIndex_T)frh.index_fr)) != 0) { + if (pk_bkt_arr_handle_validate(&ft->renders, frh.handle_fr) != PK_BKT_ARR_HANDLE_VALIDATION_VALID) { return nullptr; } - return &ft->renders[static_cast<FontRenderIndex_T>(frh.index_fr)]; + return &ft->renders[frh.handle_fr]; } -void FontType_UpdateStringRender(FontRenderHandle frh, FontRenderSettings *settings) { +void FontType_UpdateStringRender(FontTypeRender frh, FontRenderSettings *settings) { assert(settings != nullptr); assert((ftd.unused_fts & (1llu << (FontTypeIndex_T)frh.index_ft)) == 0); FontType *ft = &ftd.arr_ft[(FontTypeIndex_T)frh.index_ft]; - assert((ft->unused_frs & (1llu << (FontTypeIndex_T)frh.index_fr)) == 0); + assert(pk_bkt_arr_handle_validate(&ft->renders, frh.handle_fr) == PK_BKT_ARR_HANDLE_VALIDATION_VALID); ft->gr.should_update_instance_buffer = true; - ft->renders[(FontRenderIndex_T)frh.index_fr].settings = *settings; + ft->renders[frh.handle_fr].settings = *settings; } -void FontType_UpdateStringRenderText(FontRenderHandle frh, pk_cstr &&cstr) { +void FontType_UpdateStringRenderText(FontTypeRender frh, pk_cstr &&cstr) { uint32_t i, count; assert((ftd.unused_fts & (1llu << (FontTypeIndex_T)frh.index_ft)) == 0); FontType &ft = ftd.arr_ft[static_cast<FontTypeIndex_T>(frh.index_ft)]; - assert((ft.unused_frs & (1llu << (FontTypeIndex_T)frh.index_fr)) == 0); - FontRender &fr = ft.renders[static_cast<FontRenderIndex_T>(frh.index_fr)]; + assert(pk_bkt_arr_handle_validate(&ft.renders, frh.handle_fr) == PK_BKT_ARR_HANDLE_VALIDATION_VALID); + FontRender &fr = ft.renders[frh.handle_fr]; pk_cstr old_str = fr.text; if (window == NULL) { return; @@ -1063,23 +1042,23 @@ void FontType_UpdateStringRenderText(FontRenderHandle frh, pk_cstr &&cstr) { // TODO specific bucket count = glyph_indices.next; fr.n_glyphs = count; - fr.glyph_indices = pk_new_arr<uint32_t>(count); + fr.glyph_indices = pk_new_arr<uint32_t>(count, ftd.bkt); for (i = 0; i < count; ++i) { fr.glyph_indices[i] = glyph_indices[i]; } glyph_indices.data = nullptr; - if (fr.text.reserved > 0 && fr.text.val != NULL) pk_delete_arr<char>(fr.text.val, fr.text.reserved); + if (fr.text.reserved > 0 && fr.text.val != NULL) pk_delete_arr<char>(fr.text.val, fr.text.reserved, ftd.bkt); fr.text = cstr; ft.gr.should_update_instance_buffer = true; } -void FontType_RemoveStringRender(FontRenderHandle handle) { +void FontType_RemoveStringRender(FontTypeRender handle) { FontRender *fr; FontType *ft = &ftd.arr_ft[(FontTypeIndex_T)handle.index_ft]; // hack, but works ft->gr.should_update_instance_buffer = true; - fr = &ft->renders[(FontRenderIndex_T)handle.index_fr]; + fr = &ft->renders[handle.handle_fr]; ECS_MarkForRemoval(fr); } diff --git a/src/font.hpp b/src/font.hpp index dd1b682..cdbc309 100644 --- a/src/font.hpp +++ b/src/font.hpp @@ -8,10 +8,11 @@ #include "vendor-glm-include.hpp" #define PKE_FONT_MAX_FONT_TYPES 64 -#define PKE_FONT_MAX_FONT_RENDERS 64 TypeSafeInt_H(FontTypeIndex, uint16_t, 0xFFFF); -TypeSafeInt_H(FontRenderIndex, uint16_t, 0xFFFF); +struct FontRenderHandle : pk_bkt_arr_handle {}; + +constexpr FontRenderHandle FontRenderHandle_MAX = FontRenderHandle{ pk_bkt_arr_handle_MAX_constexpr }; TypeSafeInt_H(FONT_GLYPH_CHAR_FLAG, uint8_t, 0xFF); TypeSafeInt_H(FONT_RENDER_FLAG, uint8_t, 0xFF); @@ -40,14 +41,14 @@ const FONT_RENDER_SURFACE_AREA_TYPE_FLAG FONT_RENDER_SURFACE_AREA_TYPE_FLAGS_CEN const FONT_RENDER_SURFACE_AREA_TYPE_FLAG FONT_RENDER_SURFACE_AREA_TYPE_FLAGS_CENTER_BOTH = FONT_RENDER_SURFACE_AREA_TYPE_FLAG((1u << 0) | (1u << 1)); -struct FontRenderHandle { +struct FontTypeRender { FontTypeIndex index_ft; - FontRenderIndex index_fr; + FontRenderHandle handle_fr; }; -constexpr struct FontRenderHandle FontRenderHandle_MAX = { +constexpr FontTypeRender FontTypeRender_MAX = { .index_ft = FontTypeIndex_MAX, - .index_fr = FontRenderIndex_MAX, + .handle_fr = { pk_bkt_arr_handle_MAX }, }; struct FontGlyphChar { @@ -71,6 +72,7 @@ struct FontRenderSettings { }; struct FontRender : public Entity_Base { uint32_t *glyph_indices = nullptr; + FontTypeIndex index_ft = FontTypeIndex{0}; FontRenderHandle fr_handle = FontRenderHandle_MAX; uint32_t n_glyphs = 0; uint32_t buffer_start_index = 0; @@ -95,11 +97,9 @@ struct FontType : public Entity_Base { AssetHandle fontTextureAssetHandle; AssetHandle glyphDetailsAssetHandle; FontGlyphChar *glyphs; - FontRender *renders = nullptr; glm::vec2 atlas_size; uint32_t n_glyphs; - FontRenderIndex n_render = FontRenderIndex{0}; - uint64_t unused_frs; + pk_bkt_arr_t<FontRender> renders; FontTypeIndex index_ft = FontTypeIndex{0}; struct FontTypeMSDFSettings msdf_settings; struct FontTypeSpacing spacing; @@ -134,10 +134,10 @@ FontType* FontType_Get(FontTypeIndex idx); FontType* FontType_GetByTitle(const pk_cstr title); FontType* FontType_GetFonts(uint64_t &idx_unused); FontTypeIndex FontType_RegisterFont(pk_cstr title, AssetHandle fontTextureHandle, AssetHandle glyphsHandle, FontTypeMSDFSettings *msdf_settings, FontTypeSpacing *spacing); -FontRenderHandle FontType_AddStringRender(FontTypeIndex idx_ft, const pk_cstr &&str, FontRenderSettings *settings, Entity_Base *parent = nullptr, pk_uuid uuid = pk_uuid_zed); -FontRender *FontType_GetFontRender(FontRenderHandle frh); -void FontType_UpdateStringRender(FontRenderHandle frh, FontRenderSettings *settings); -void FontType_UpdateStringRenderText(FontRenderHandle frh, pk_cstr &&cstr); -void FontType_RemoveStringRender(FontRenderHandle frh); +FontTypeRender FontType_AddStringRender(FontTypeIndex idx_ft, const pk_cstr &&str, FontRenderSettings *settings, Entity_Base *parent = nullptr, pk_uuid uuid = pk_uuid_zed); +FontRender *FontType_GetFontRender(FontTypeRender frh); +void FontType_UpdateStringRender(FontTypeRender frh, FontRenderSettings *settings); +void FontType_UpdateStringRenderText(FontTypeRender frh, pk_cstr &&cstr); +void FontType_RemoveStringRender(FontTypeRender frh); #endif /* PKE_FONT_TYPE_HPP */ diff --git a/src/plugins.cpp b/src/plugins.cpp index 838237b..a702e39 100644 --- a/src/plugins.cpp +++ b/src/plugins.cpp @@ -13,9 +13,9 @@ #include <dlfcn.h> #endif -pk_arr_t<PKEPluginInterface> LoadedPkePlugins; -pk_arr_t<PkeCallback> pkePluginCallbacks; -pk_arr_t<CallbackSignature> sortedSignatures; +pk_arr_t<PKEPluginInterface> LoadedPkePlugins{}; +pk_arr_t<PkeCallback> pkePluginCallbacks{}; +pk_arr_t<CallbackSignature> sortedSignatures{}; void PkePlugin_Load(const char *path) { if (path == nullptr || path == CAFE_BABE(const char)) { diff --git a/src/serialization-font.cpp b/src/serialization-font.cpp index f2fb24b..328aa94 100644 --- a/src/serialization-font.cpp +++ b/src/serialization-font.cpp @@ -41,7 +41,7 @@ pk_handle pke_serialize_font_render(srlztn_serialize_helper *h, FontRender *fr) } { kve.key = SRLZTN_UI_FONT_RENDER_FONT_TYPE_TITLE; - FontType *ft = FontType_Get(fr->fr_handle.index_ft); + FontType *ft = FontType_Get(fr->index_ft); len = snprintf(nullptr, 0, "%s", ft->title.val); s = pk_new_arr<char>(len+1, h->bkt); sprintf(s, "%s", ft->title.val); diff --git a/src/serialization-static-ui.cpp b/src/serialization-static-ui.cpp index ae948ab..373bc16 100644 --- a/src/serialization-static-ui.cpp +++ b/src/serialization-static-ui.cpp @@ -25,7 +25,7 @@ pk_handle pke_serialize_ui_box_internal(srlztn_serialize_helper *h, pke_ui_box_t { kve.key = SRLZTN_UI_BOX_DATA_TEXT_FONT_RENDER_UUID; s = pk_new_arr<char>(37, h->bkt); - FontRender *fr = FontType_GetFontRender(data->font_render_handle); + FontRender *fr = FontType_GetFontRender(data->font_type_render); sprintf(s, pk_uuid_printf_format, pk_uuid_printf_var(fr->uuid)); kve.val = s; kve.end = SRLZTN_KVE_END; @@ -58,7 +58,7 @@ pk_handle pke_serialize_ui_box_internal(srlztn_serialize_helper *h, pke_ui_box_t { kve.key = SRLZTN_UI_BOX_DATA_BUTTON_TEXT_FONT_RENDER_UUID; s = pk_new_arr<char>(37, h->bkt); - FontRender *fr = FontType_GetFontRender(data->font_render_handle); + FontRender *fr = FontType_GetFontRender(data->font_type_render); sprintf(s, pk_uuid_printf_format, pk_uuid_printf_var(fr->uuid)); kve.val = s; kve.end = SRLZTN_KVE_END; @@ -139,7 +139,7 @@ void pke_deserialize_ui_box_internal(srlztn_deserialize_helper *h, pke_kve_conta FontRender *fr = static_cast<FontRender *>(ECS_GetEntityByUUID(font_render_uuid)); - data->font_render_handle = fr->fr_handle; + data->font_type_render = {fr->index_ft, {fr->fr_handle}}; } void pke_deserialize_ui_box_internal(srlztn_deserialize_helper *h, pke_kve_container *kvec, pke_ui_box_type_data::pke_ui_box_type_data_button_text *data) { @@ -150,7 +150,7 @@ void pke_deserialize_ui_box_internal(srlztn_deserialize_helper *h, pke_kve_conta pk_uuid font_render_uuid = pk_uuid_zed; // TODO - data->font_render_handle = FontRenderHandle_MAX; + data->font_type_render = FontTypeRender_MAX; data->pke_event_handle = PkeEventHandle_MAX; data->ev_id = pk_ev_id_T{0}; @@ -163,7 +163,7 @@ void pke_deserialize_ui_box_internal(srlztn_deserialize_helper *h, pke_kve_conta FontRender *fr = static_cast<FontRender *>(ECS_GetEntityByUUID(font_render_uuid)); - data->font_render_handle = fr->fr_handle; + data->font_type_render = {fr->index_ft, {fr->fr_handle}}; } void pke_deserialize_ui_box_internal(srlztn_deserialize_helper *h, pke_kve_container *kvec, pke_ui_box_type_data::pke_ui_box_type_data_button_image *data) { diff --git a/src/serialization.cpp b/src/serialization.cpp index 63415e0..a09a332 100644 --- a/src/serialization.cpp +++ b/src/serialization.cpp @@ -85,10 +85,11 @@ void pke_serialize_scene(srlztn_serialize_helper *h) { for (FontTypeIndex_T b = 0; b < PKE_FONT_MAX_FONT_TYPES; ++b) { if ((idx_unused & (1llu << b)) != 0) continue; FontType *ft = &fonts[b]; - for (FontRenderIndex_T i = 0; i < (FontRenderIndex_T)ft->n_render; ++i) { - if ((ft->unused_frs & (1llu << i)) != 0) continue; - pke_serialize_font_render(h, &ft->renders[i]); - } + pk_bkt_arr_t<FontRender>::FN_Iter iter_fn{}; + iter_fn.func = [&h](FontRender *fr) { + pke_serialize_font_render(h, fr); + }; + pk_bkt_arr_iterate(&ft->renders, iter_fn.invoke, &iter_fn); } pke_ui_box_count_T box_count; diff --git a/src/static-ui.cpp b/src/static-ui.cpp index 505f4bc..8addb7e 100644 --- a/src/static-ui.cpp +++ b/src/static-ui.cpp @@ -396,7 +396,7 @@ bool pke_ui_calc_px(pk_arr_t<pke_ui_box_instance_buffer_item> &buffer, pk_arr_t< FontRenderSettings frs; switch (box->type) { case PKE_UI_BOX_TYPE_TEXT: - frs = FontType_GetFontRender(box->type_data->text.font_render_handle)->settings; + frs = FontType_GetFontRender(box->type_data->text.font_type_render)->settings; frs.surface_area_pos = box->internal.px_corner; frs.surface_area_size = box->internal.px_size; if (PK_HAS_FLAG(box->flags, PKE_UI_BOX_FLAG_VISIBILITY_INVISIBLE)) { @@ -404,10 +404,10 @@ bool pke_ui_calc_px(pk_arr_t<pke_ui_box_instance_buffer_item> &buffer, pk_arr_t< } else { frs.flags = FONT_RENDER_FLAG(FONT_RENDER_FLAG_T(frs.flags) & (~FONT_RENDER_FLAG_T(FONT_RENDER_FLAG_VISIBILITY_INVISIBLE))); } - FontType_UpdateStringRender(box->type_data->text.font_render_handle, &frs); + FontType_UpdateStringRender(box->type_data->text.font_type_render, &frs); break; case PKE_UI_BOX_TYPE_BUTTON_TEXT: - frs = FontType_GetFontRender(box->type_data->button_text.font_render_handle)->settings; + frs = FontType_GetFontRender(box->type_data->button_text.font_type_render)->settings; frs.surface_area_pos = box->internal.px_corner; frs.surface_area_size = box->internal.px_size; if (PK_HAS_FLAG(box->flags, PKE_UI_BOX_FLAG_VISIBILITY_INVISIBLE)) { @@ -415,7 +415,7 @@ bool pke_ui_calc_px(pk_arr_t<pke_ui_box_instance_buffer_item> &buffer, pk_arr_t< } else { frs.flags = FONT_RENDER_FLAG(FONT_RENDER_FLAG_T(frs.flags) & (~FONT_RENDER_FLAG_T(FONT_RENDER_FLAG_VISIBILITY_INVISIBLE))); } - FontType_UpdateStringRender(box->type_data->button_text.font_render_handle, &frs); + FontType_UpdateStringRender(box->type_data->button_text.font_type_render, &frs); break; case PKE_UI_BOX_TYPE_BUTTON_IMAGE: break; @@ -749,15 +749,15 @@ void pke_ui_teardown_box_recursive(pke_ui_box *box, uint8_t depth) { // If correctly parented, this is unnecessary, but serialization does // not construct text boxes in this way. // Handle to avoid memory leaks. - fr = FontType_GetFontRender(box->type_data->text.font_render_handle); + fr = FontType_GetFontRender(box->type_data->text.font_type_render); if (fr != nullptr && fr->parentHandle != box->handle) { - FontType_RemoveStringRender(box->type_data->text.font_render_handle); + FontType_RemoveStringRender(box->type_data->text.font_type_render); } break; case PKE_UI_BOX_TYPE_BUTTON_TEXT: - fr = FontType_GetFontRender(box->type_data->button_text.font_render_handle); + fr = FontType_GetFontRender(box->type_data->button_text.font_type_render); if (fr != nullptr && fr->parentHandle != box->handle) { - FontType_RemoveStringRender(box->type_data->button_text.font_render_handle); + FontType_RemoveStringRender(box->type_data->button_text.font_type_render); } break; case PKE_UI_BOX_TYPE_BUTTON_IMAGE: @@ -887,7 +887,7 @@ void pke_ui_internal_new_typed_box(pke_ui_box *box, const PKE_UI_BOX_TYPE type) break; case PKE_UI_BOX_TYPE_BUTTON_TEXT: tp_data = pk_new<pke_ui_box_type_data>(pke_ui_master.bkt); - tp_data->button_text.font_render_handle = FontRenderHandle_MAX; + tp_data->button_text.font_type_render = FontTypeRender_MAX; pressed_ev = ECS_CreateEv(box, pk_uuid_max); tp_data->button_text.pke_event_handle = pressed_ev->pke_event_handle; tp_data->button_text.ev_id = pressed_ev->ev_id; diff --git a/src/static-ui.hpp b/src/static-ui.hpp index e4c86af..17301a1 100644 --- a/src/static-ui.hpp +++ b/src/static-ui.hpp @@ -141,10 +141,10 @@ struct pke_ui_box : public Entity_Base { // separate these if they become silly union pke_ui_box_type_data { struct pke_ui_box_type_data_text { - FontRenderHandle font_render_handle; + FontTypeRender font_type_render; } text; struct pke_ui_box_type_data_button_text { - FontRenderHandle font_render_handle; + FontTypeRender font_type_render; PkeEventHandle pke_event_handle; pk_ev_id_T ev_id; } button_text; |
