From 2a7d4cd21395d1cbab9d85b6251ada2f0d2d69e7 Mon Sep 17 00:00:00 2001 From: korenkonder Date: Sun, 16 Jul 2023 09:37:37 +0300 Subject: [PATCH] Renaming things in Equip Test --- src/CRE/item_table.cpp | 6 +- src/CRE/item_table.hpp | 4 +- src/CRE/rob/rob.cpp | 3 +- src/ReDIVA/data_test/equip_test.cpp | 154 +++++++++++++-------------- src/ReDIVA/data_test/motion_test.cpp | 30 +++--- src/ReDIVA/data_test/motion_test.hpp | 6 +- src/ReDIVA/pv_game/pv_game.cpp | 3 +- 7 files changed, 102 insertions(+), 104 deletions(-) diff --git a/src/CRE/item_table.cpp b/src/CRE/item_table.cpp index 17f408d5..2c3fa49d 100644 --- a/src/CRE/item_table.cpp +++ b/src/CRE/item_table.cpp @@ -96,13 +96,13 @@ void item_table_handler_array_init() { item_table_handler_array[i].set_path((chara_index)i); } -const item_cos_data* item_table_handler_array_get_item_cos_data_by_module_index( - chara_index chara_index, int32_t module_index) { +const item_cos_data* item_table_handler_array_get_item_cos_data( + chara_index chara_index, int32_t cos_id) { const item_table* table = item_table_handler_array_get_table(chara_index); if (!table) return 0; - auto elem = table->cos.find(module_index); + auto elem = table->cos.find(cos_id); if (elem != table->cos.end()) return &elem->second; else if (table->cos.size()) diff --git a/src/CRE/item_table.hpp b/src/CRE/item_table.hpp index 42d900dc..23f9fe36 100644 --- a/src/CRE/item_table.hpp +++ b/src/CRE/item_table.hpp @@ -121,8 +121,8 @@ struct item_table { }; extern void item_table_handler_array_init(); -extern const item_cos_data* item_table_handler_array_get_item_cos_data_by_module_index( - chara_index chara_index, int32_t module_index); +extern const item_cos_data* item_table_handler_array_get_item_cos_data( + chara_index chara_index, int32_t cos_id); extern const item_table_item* item_table_handler_array_get_item( chara_index chara_index, int32_t item_no); extern std::string item_table_handler_array_get_item_name( diff --git a/src/CRE/rob/rob.cpp b/src/CRE/rob/rob.cpp index fad7ce34..7ccb70b3 100644 --- a/src/CRE/rob/rob.cpp +++ b/src/CRE/rob/rob.cpp @@ -11303,8 +11303,7 @@ static void rob_chara_set_pv_data(rob_chara* rob_chr, int8_t chara_id, rob_chr->pv_data = pv_data; rob_chr->chara_init_data = chara_init_data_get(chara_index); if (!check_module_index_is_501(module_index)) { - const item_cos_data* cos - = item_table_handler_array_get_item_cos_data_by_module_index(chara_index, module_index); + const item_cos_data* cos = item_table_handler_array_get_item_cos_data(chara_index, module_index); if (cos) rob_chr->item_cos_data.set_chara_index_item_nos(chara_index, cos->arr); } diff --git a/src/ReDIVA/data_test/equip_test.cpp b/src/ReDIVA/data_test/equip_test.cpp index 60388b13..0b7d27f6 100644 --- a/src/ReDIVA/data_test/equip_test.cpp +++ b/src/ReDIVA/data_test/equip_test.cpp @@ -15,14 +15,14 @@ class DataTestEquipDw : public dw::Shell { public: struct Data { bool disp; - bool field_1; - bool field_2; + bool set_item_no; + bool reset_item_no; bool refresh; int32_t item_no; const item_cos_data* cos; chara_index chara_index; int32_t module_index; - bool field_18; + bool update_item; int32_t disp_parts; std::vector items[DATA_TEST_ITEM_EQUIP_MAX]; std::map field_278[DATA_TEST_ITEM_EQUIP_MAX]; @@ -59,9 +59,12 @@ public: int32_t GetItemNo(int32_t chara_id); int32_t GetModuleIndex(int32_t chara_id); bool GetRefresh(int32_t chara_id); + bool GetResetItemNo(int32_t chara_id); + bool GetSetItemNo(int32_t chara_id); int32_t GetSelectedCharaItemNo(int32_t chara_id, dw::Widget* data); bool IsEquipAllowed(int32_t equip); void ItemReset(); + void ResetCharaItemCos(int32_t chara_id); void SetCharaID(int32_t value); void SetCharaIndex(int32_t chara_id, ::chara_index chara_index); void SetCharaItemCos(int32_t chara_id, const item_cos_data* cos); @@ -72,6 +75,9 @@ public: void SetItemNo(int32_t chara_id, int32_t value); void SetModuleIndex(int32_t chara_id, int32_t module_index); void SetRefresh(int32_t chara_id, bool value); + void SetResetItemNo(int32_t chara_id, bool value); + void SetSetItemNo(int32_t chara_id, bool value); + void SetUpdateItem(int32_t chara_id, bool value); static void Disp1pCallback(dw::Widget* data); static void Disp2pCallback(dw::Widget* data); @@ -85,12 +91,6 @@ public: void sub_14025FD80(int32_t chara_id); - bool sub_1402606C0(int32_t chara_id); - bool sub_140260720(int32_t chara_id); - void sub_140260C40(int32_t chara_id); - void sub_140261370(int32_t chara_id, bool value); - void sub_1402613D0(int32_t chara_id, bool value); - void sub_1402613F0(int32_t chara_id, bool value); void sub_140261520(int32_t chara_id); void sub_140261580(int32_t chara_id); void sub_140261AF0(int32_t chara_id); @@ -124,10 +124,10 @@ static void data_test_equip_dw_set_chara_index(int32_t chara_id, chara_index cha static void data_test_equip_dw_set_chara_item_cos(int32_t chara_id, const item_cos_data* cos); static const item_cos_data* data_test_equip_dw_get_cos(int32_t chara_id); static int32_t data_test_equip_dw_get_item_no(int32_t chara_id); -static void data_test_equip_dw_set_module_index(int32_t chara_id, int32_t module_index); -static bool data_test_equip_dw_sub_1402606C0(int32_t chara_id); -static bool data_test_equip_dw_sub_140260720(int32_t chara_id); -static void data_test_equip_dw_sub_140260C40(int32_t chara_id); +static bool data_test_equip_dw_get_reset_item_no(int32_t chara_id); +static bool data_test_equip_dw_get_set_item_no(int32_t chara_id); +static void data_test_equip_dw_reset_chara_item_cos(int32_t chara_id); +static void data_test_equip_dw_set_module_index(int32_t chara_id, int32_t cos); static void data_test_equip_dw_sub_140261520(int32_t chara_id); static std::vector* data_test_item_array_get(::chara_index chara_index); @@ -140,8 +140,8 @@ static void data_test_item_equip_array_unload(); static int32_t data_test_item_get_item_equip(chara_index chara_index, int32_t item_no); -DtmEqVs::DtmEqVs() : chara_id(), chara_index(), curr_chara_index(), -module_index(), curr_module_index(), disp_parts() { +DtmEqVs::DtmEqVs() : chara_id(), chara_index(), +curr_chara_index(), module_index(), curr_module_index(), disp_parts() { state = -1; } @@ -163,7 +163,7 @@ bool DtmEqVs::Ctrl() { data_test_equip_dw_set_chara_item_cos(chara_id, item_cos_data->get_cos()); state = 3; case 3: - if (dtm_mot_array_sub_140291C10()) { + if (dtm_mot_array_get_loaded()) { CtrlDispParts(); CtrlChara(); @@ -173,12 +173,12 @@ bool DtmEqVs::Ctrl() { temp_cos = *rob_chr->item_cos_data.get_cos(); bool reload = false; - if (data_test_equip_dw_sub_1402606C0(chara_id)) { + if (data_test_equip_dw_get_set_item_no(chara_id)) { item_cos_data->set_chara_index_item(chara_index, item_no); reload = true; } - if (data_test_equip_dw_sub_140260720(chara_id)) { + if (data_test_equip_dw_get_reset_item_no(chara_id)) { item_cos_data->set_chara_index_item_zero(chara_index, item_no); reload = true; } @@ -194,7 +194,7 @@ bool DtmEqVs::Ctrl() { sub_1402CF8F0(); }*/ - data_test_equip_dw_sub_140260C40(chara_id); + data_test_equip_dw_reset_chara_item_cos(chara_id); if (reload) { if (rob_chr) @@ -262,7 +262,7 @@ void DtmEqVs::CtrlChara() { curr_chara_index = chara_index; curr_module_index = module_index; - const item_cos_data* cos = item_table_handler_array_get_item_cos_data_by_module_index(chara_index, module_index); + const item_cos_data* cos = item_table_handler_array_get_item_cos_data(chara_index, module_index); if (!check_module_index_is_501(curr_module_index) || !cos) { data_test_equip_dw_sub_140261520(chara_id); data_test_equip_dw_set_chara_item_cos(chara_id, cos); @@ -316,8 +316,8 @@ void equip_test_free() { } } -DataTestEquipDw::Data::Data() : disp(), field_1(), field_2(), refresh(), item_no(), -cos(), chara_index(), module_index(), field_18(), disp_parts(), current_items() { +DataTestEquipDw::Data::Data() : disp(), set_item_no(), reset_item_no(), refresh(), item_no(), +cos(), chara_index(), module_index(), update_item(), disp_parts(), current_items() { } @@ -458,7 +458,7 @@ void DataTestEquipDw::Hide() { dw::Shell::SetDisp(false); for (DataTestEquipDw::Data& i : data) - i.field_18 = false; + i.update_item = false; data_test_item_equip_array_unload(); } @@ -466,12 +466,12 @@ void DataTestEquipDw::Hide() { void DataTestEquipDw::DataReset(int32_t chara_id) { DataTestEquipDw::Data& data = this->data[chara_id]; data.disp = true; - data.field_1 = false; - data.field_2 = false; + data.set_item_no = false; + data.reset_item_no = false; data.refresh = false; data.item_no = 0; data.chara_index = CHARA_MIKU; - data.field_18 = false; + data.update_item = false; data.disp_parts = 0; for (int32_t i = 0; i < DATA_TEST_ITEM_EQUIP_MAX; i++) { @@ -496,6 +496,10 @@ const item_cos_data* DataTestEquipDw::GetCos(int32_t chara_id) { return data[chara_id].cos; } +int32_t DataTestEquipDw::GetCurrentCharaItem(int32_t chara_id, dw::Widget* data) { + return this->data[chara_id].current_items[data->callback_data.i32]; +} + int32_t DataTestEquipDw::GetDispParts(int32_t chara_id) { return data[chara_id].disp_parts; } @@ -512,6 +516,10 @@ bool DataTestEquipDw::GetRefresh(int32_t chara_id) { return data[chara_id].refresh; } +bool DataTestEquipDw::GetResetItemNo(int32_t chara_id) { + return data[chara_id].reset_item_no; +} + int32_t DataTestEquipDw::GetSelectedCharaItemNo(int32_t chara_id, dw::Widget* data) { dw::ListBox* list_box = dynamic_cast(data); if (list_box) @@ -519,6 +527,10 @@ int32_t DataTestEquipDw::GetSelectedCharaItemNo(int32_t chara_id, dw::Widget* da return 0; } +bool DataTestEquipDw::GetSetItemNo(int32_t chara_id) { + return data[chara_id].set_item_no; +} + bool DataTestEquipDw::IsEquipAllowed(int32_t equip) { if (equip >= 0 && equip < DATA_TEST_ITEM_EQUIP_MAX && (equip == 0 || equip == 1 || equip == 4 || equip == 6 || equip == 8 @@ -540,6 +552,13 @@ void DataTestEquipDw::ItemReset() { item.parts = 0; } +void DataTestEquipDw::ResetCharaItemCos(int32_t chara_id) { + data[chara_id].set_item_no = false; + data[chara_id].reset_item_no = false; + data[chara_id].item_no = 0; + data[chara_id].cos = 0; +} + void DataTestEquipDw::SetCharaID(int32_t value) { chara_id = value; } @@ -601,6 +620,18 @@ void DataTestEquipDw::SetRefresh(int32_t chara_id, bool value) { data[chara_id].refresh = value; } +void DataTestEquipDw::SetResetItemNo(int32_t chara_id, bool value) { + data[chara_id].reset_item_no = value; +} + +void DataTestEquipDw::SetSetItemNo(int32_t chara_id, bool value) { + data[chara_id].set_item_no = value; +} + +void DataTestEquipDw::SetUpdateItem(int32_t chara_id, bool value) { + data[chara_id].update_item = value; +} + void DataTestEquipDw::Disp1pCallback(dw::Widget* data) { DispCallback(0, data); } @@ -633,16 +664,16 @@ void DataTestEquipDw::ItemCallback(dw::Widget* data) { int32_t curr_item_no = data_test_equip_dw->GetCurrentCharaItem(chara_id, data); if (item_no) { - data_test_equip_dw->sub_140261370(chara_id, true); - data_test_equip_dw->sub_1402613D0(chara_id, false); - data_test_equip_dw->sub_1402613F0(chara_id, true); + data_test_equip_dw->SetSetItemNo(chara_id, true); + data_test_equip_dw->SetResetItemNo(chara_id, false); + data_test_equip_dw->SetUpdateItem(chara_id, true); data_test_equip_dw->SetItemNo(chara_id, item_no); data_test_equip_dw->SetCurrentItemNo(chara_id, data, item_no); } else if (curr_item_no) { - data_test_equip_dw->sub_140261370(chara_id, false); - data_test_equip_dw->sub_1402613D0(chara_id, true); - data_test_equip_dw->sub_1402613F0(chara_id, true); + data_test_equip_dw->SetSetItemNo(chara_id, false); + data_test_equip_dw->SetResetItemNo(chara_id, true); + data_test_equip_dw->SetUpdateItem(chara_id, true); data_test_equip_dw->SetItemNo(chara_id, curr_item_no); data_test_equip_dw->SetCurrentItemNo(chara_id, data, curr_item_no); } @@ -660,15 +691,15 @@ void DataTestEquipDw::PartsCallback(dw::Widget* data) { int32_t chara_id = data_test_equip_dw->GetCharaID(); ::chara_index chara_index = data_test_equip_dw->GetCharaIndex(chara_id); - int32_t module_index = data_test_equip_dw->GetModuleIndex(chara_id); + int32_t cos = data_test_equip_dw->GetModuleIndex(chara_id); - const item_cos_data* v9 = item_table_handler_array_get_item_cos_data_by_module_index(chara_index, module_index); + const item_cos_data* v9 = item_table_handler_array_get_item_cos_data(chara_index, cos); size_t selected_item = list_box->list->selected_item; switch (selected_item) { case 0: break; case 1: - v9 = item_table_handler_array_get_item_cos_data_by_module_index(chara_index, 499); + v9 = item_table_handler_array_get_item_cos_data(chara_index, 499); break; case 2: default: { @@ -732,37 +763,6 @@ void DataTestEquipDw::sub_14025FD80(int32_t chara_id) { } } -int32_t DataTestEquipDw::GetCurrentCharaItem(int32_t chara_id, dw::Widget* data) { - return this->data[chara_id].current_items[data->callback_data.i32]; -} - -bool DataTestEquipDw::sub_1402606C0(int32_t chara_id) { - return data[chara_id].field_1; -} - -bool DataTestEquipDw::sub_140260720(int32_t chara_id) { - return data[chara_id].field_2; -} - -void DataTestEquipDw::sub_140260C40(int32_t chara_id) { - data[chara_id].field_1 = false; - data[chara_id].field_2 = false; - data[chara_id].item_no = 0; - data[chara_id].cos = 0; -} - -void DataTestEquipDw::sub_140261370(int32_t chara_id, bool value) { - data[chara_id].field_1 = value; -} - -void DataTestEquipDw::sub_1402613D0(int32_t chara_id, bool value) { - data[chara_id].field_2 = value; -} - -void DataTestEquipDw::sub_1402613F0(int32_t chara_id, bool value) { - data[chara_id].field_18 = value; -} - void DataTestEquipDw::sub_140261520(int32_t chara_id) { if (this->chara_id == chara_id) { sub_140261C40(chara_id); @@ -878,26 +878,26 @@ static int32_t data_test_equip_dw_get_item_no(int32_t chara_id) { return 0; } -static void data_test_equip_dw_set_module_index(int32_t chara_id, int32_t module_index) { +static bool data_test_equip_dw_get_reset_item_no(int32_t chara_id) { if (data_test_equip_dw) - data_test_equip_dw->SetModuleIndex(chara_id, module_index); -} - -static bool data_test_equip_dw_sub_1402606C0(int32_t chara_id) { - if (data_test_equip_dw) - return data_test_equip_dw->sub_1402606C0(chara_id); + return data_test_equip_dw->GetResetItemNo(chara_id); return false; } -static bool data_test_equip_dw_sub_140260720(int32_t chara_id) { +static bool data_test_equip_dw_get_set_item_no(int32_t chara_id) { if (data_test_equip_dw) - return data_test_equip_dw->sub_140260720(chara_id); + return data_test_equip_dw->GetSetItemNo(chara_id); return false; } -static void data_test_equip_dw_sub_140260C40(int32_t chara_id) { +static void data_test_equip_dw_reset_chara_item_cos(int32_t chara_id) { if (data_test_equip_dw) - data_test_equip_dw->sub_140260C40(chara_id); + data_test_equip_dw->ResetCharaItemCos(chara_id); +} + +static void data_test_equip_dw_set_module_index(int32_t chara_id, int32_t cos) { + if (data_test_equip_dw) + data_test_equip_dw->SetModuleIndex(chara_id, cos); } static void data_test_equip_dw_sub_140261520(int32_t chara_id) { diff --git a/src/ReDIVA/data_test/motion_test.cpp b/src/ReDIVA/data_test/motion_test.cpp index bb082343..db82f06f 100644 --- a/src/ReDIVA/data_test/motion_test.cpp +++ b/src/ReDIVA/data_test/motion_test.cpp @@ -675,7 +675,7 @@ void DataTestMot::Disp() { void DataTestMot::sub_140286280() { dtm_mot_array[0].sub_1402922C0(true); dtm_mot_array[0].sub_1402922C0(true); - if (data.sync_frame && (!dtm_mot_array[0].sub_140291C10() || !dtm_mot_array[1].sub_140291C10())) { + if (data.sync_frame && (!dtm_mot_array[0].GetLoaded() || !dtm_mot_array[1].GetLoaded())) { dtm_mot_array[0].sub_1402922C0(false); dtm_mot_array[0].sub_1402922C0(false); } @@ -889,7 +889,7 @@ void DataTestMotA3d::Sync1pFrame() { } DtmMot::DtmMot() : rob_bone_data(), type(), rot_y(), pre_offset(), post_offset(), divide(), -loop(), change_motion(), use_opd(), field_D6(), reset_mot(), save_only_start_frame(), state(), +loop(), change_motion(), use_opd(), loaded(), reset_mot(), save_only_start_frame(), state(), frame(), delta_frame(), looped(), start_frame(), ab_frame(), ab_loop(), set_motion_index() { chara_id = -1; chara_index = CHARA_MAX; @@ -943,13 +943,7 @@ bool DtmMot::Init() { } pv_id = DtmMot::ConvertMotionSetNameToPVID(aft_mot_db->get_motion_set_name(motion_set_id)); - field_D6 = false; - return true; -} - -bool DtmMot::sub_140291C10() { - if (state) - return field_D6; + loaded = false; return true; } @@ -1025,7 +1019,7 @@ bool DtmMot::Ctrl() { state = 11; } break; case 7: { - field_D6 = false; + loaded = false; if (skin_param_manager_check_task_ready(chara_id) || pv_osage_manager_array_get_disp(chara_id) || osage_play_data_manager_check_task_ready()) @@ -1160,7 +1154,7 @@ bool DtmMot::Ctrl() { state = 11; } break; case 11: { - field_D6 = true; + loaded = true; if (!field_100) break; @@ -1432,6 +1426,12 @@ float_t DtmMot::GetFrameCount() { return 0.0f; } +bool DtmMot::GetLoaded() { + if (state) + return loaded; + return true; +} + float_t DtmMot::GetStep() { return step[3]; } @@ -1649,15 +1649,15 @@ void motion_test_free() { } } +bool dtm_mot_array_get_loaded() { + return dtm_mot_array[0].GetLoaded() && dtm_mot_array[1].GetLoaded(); +} + void dtm_mot_array_set_reset_mot() { dtm_mot_array[0].SetResetMot(); dtm_mot_array[1].SetResetMot(); } -bool dtm_mot_array_sub_140291C10() { - return dtm_mot_array[0].sub_140291C10() && dtm_mot_array[1].sub_140291C10(); -} - DataTestFaceMotDw::DataTestFaceMotDw(int32_t chara_id) { this->chara_id = chara_id; diff --git a/src/ReDIVA/data_test/motion_test.hpp b/src/ReDIVA/data_test/motion_test.hpp index e80ef29b..818261dc 100644 --- a/src/ReDIVA/data_test/motion_test.hpp +++ b/src/ReDIVA/data_test/motion_test.hpp @@ -129,7 +129,7 @@ public: bool change_motion; bool disp; bool use_opd; - bool field_D6; + bool loaded; bool reset_mot; bool partial_mot; bool save_only_start_frame; @@ -166,6 +166,7 @@ public: void GetABLoop(bool& ab_loop, float_t& a_frame, float_t& b_frame); float_t GetFrame(); float_t GetFrameCount(); + bool GetLoaded(); float_t GetStep(); void ResetABLoop(); void SetABFrame(bool a); @@ -190,7 +191,6 @@ public: static int32_t ConvertMotionSetNameToPVID(const char* set_name); - bool sub_140291C10(); void sub_1402922C0(bool value); }; @@ -200,5 +200,5 @@ extern DtmMot* dtm_mot_array; extern void motion_test_init(); extern void motion_test_free(); +extern bool dtm_mot_array_get_loaded(); extern void dtm_mot_array_set_reset_mot(); -extern bool dtm_mot_array_sub_140291C10(); diff --git a/src/ReDIVA/pv_game/pv_game.cpp b/src/ReDIVA/pv_game/pv_game.cpp index 13bd786a..4717d4be 100644 --- a/src/ReDIVA/pv_game/pv_game.cpp +++ b/src/ReDIVA/pv_game/pv_game.cpp @@ -6367,8 +6367,7 @@ static void sub_140105010(pv_game_chara* arr, size_t max_count, pv_game_item_mas for (int32_t i = 0; i < PV_PERFORMER_ITEM_MAX; i++) { int32_t item_no = pv->get_performer_item(performer, item); if (item_no == -2) { - const item_cos_data* cos = - item_table_handler_array_get_item_cos_data_by_module_index(chr->chara_index, chr->cos); + const item_cos_data* cos = item_table_handler_array_get_item_cos_data(chr->chara_index, chr->cos); if (!cos) continue;