JParticle debug (#3090)

1 files changed, 70 insertions, 32 deletions

diff --git a/src/JSystem/JParticle/JPAResourceLoader.cpp b/src/JSystem/JParticle/JPAResourceLoader.cpp
index e065ee72b3..d0f3c8ebd4 100644
--- a/src/JSystem/JParticle/JPAResourceLoader.cpp
+++ b/src/JSystem/JParticle/JPAResourceLoader.cpp
@@ -12,12 +12,35 @@
 #include "JSystem/JParticle/JPAResource.h"
 #include "JSystem/JParticle/JPAResourceManager.h"
 
+static void dummy1() {
+    JUT_WARN(0, "JPA : wrong version file %s\n");
+    OS_REPORT("JPA : no jpa file\n");
+}
+
 JPAResourceLoader::JPAResourceLoader(u8 const* data, JPAResourceManager* mgr) {
     if (*(u32*)(data + 4) == '2-10') {
         load_jpc(data, mgr);
+    } else {
+        JUT_WARN(48, "JPA : wrong version file\n");
     }
 }
 
+static void dummy2() {
+    OS_REPORT("tex_num && \"a resource have to use one or more texture\"");
+    OS_REPORT("p_res != 0");
+    OS_REPORT("(p_res->ppFld != 0) || (p_res->fldNum == 0)");
+    OS_REPORT("(p_res->ppKey != 0) || (p_res->keyNum == 0)");
+    OS_REPORT("(p_res->texIdxTbl != 0) || (p_res->texNum == 0)");
+    OS_REPORT("p_res->ppFld[fld_no] != 0");
+    OS_REPORT("p_res->ppKey[key_no] != 0");
+    OS_REPORT("p_res->pDyn != 0");
+    OS_REPORT("p_res->pBsp != 0");
+    OS_REPORT("p_res->pEsp != 0");
+    OS_REPORT("p_res->pCsp != 0");
+    OS_REPORT("p_res->pEts != 0");
+    OS_REPORT("JPA : wrong type block in jpc file %x\n");
+}
+
 struct JPAResourceHeader {
     /* 0x0 */ u16 mUsrIdx;
     /* 0x2 */ u16 mBlockNum;
@@ -26,74 +49,89 @@ struct JPAResourceHeader {
     /* 0x6 */ u8 mTDB1Num;
 };
 
-void JPAResourceLoader::load_jpc(u8 const* data, JPAResourceManager* mgr) {
-    JKRHeap* heap = mgr->mpHeap;
-    mgr->mResMax = *(u16*)(data + 8);
-    mgr->mTexMax = *(u16*)(data + 0xA);
-    mgr->mpResArr = new (heap, 0) JPAResource*[mgr->mResMax];
-    mgr->mpTexArr = new (heap, 0) JPATexture*[mgr->mTexMax];
+void JPAResourceLoader::load_jpc(u8 const* data, JPAResourceManager* p_res_mgr) {
+    JKRHeap* heap = p_res_mgr->mpHeap;
+    p_res_mgr->resMaxNum = *(u16*)(data + 8);
+    p_res_mgr->texMaxNum = *(u16*)(data + 0xA);
+    p_res_mgr->pResAry = new (heap, 0) JPAResource*[p_res_mgr->resMaxNum];
+    p_res_mgr->pTexAry = new (heap, 0) JPATexture*[p_res_mgr->texMaxNum];
+    JUT_ASSERT(199, (p_res_mgr->pResAry != NULL) && (p_res_mgr->pTexAry != 0));
 
     u32 offset = 0x10;
     for (int i = 0; i < *(u16*)(data + 8); i++) {
         JPAResourceHeader* header = (JPAResourceHeader*)(data + offset);
-        JPAResource* res = new (heap, 0) JPAResource();
-        res->mFieldBlockNum = header->mFieldBlockNum;
-        res->mpFieldBlocks = res->mFieldBlockNum != 0 ?
-            new (heap, 0) JPAFieldBlock*[res->mFieldBlockNum] : NULL;
-        res->mKeyBlockNum = header->mKeyBlockNum;
-        res->mpKeyBlocks = res->mKeyBlockNum != 0 ?
-            new (heap, 0) JPAKeyBlock*[res->mKeyBlockNum] : NULL;
-        res->mTDB1Num = header->mTDB1Num;
-        res->mpTDB1 = NULL;
-        res->mUsrIdx = header->mUsrIdx;
+        JPAResource* p_res = new (heap, 0) JPAResource();
+        JUT_ASSERT(211, p_res != NULL);
+        p_res->fldNum = header->mFieldBlockNum;
+        p_res->ppFld = p_res->fldNum != 0 ?
+            new (heap, 0) JPAFieldBlock*[p_res->fldNum] : NULL;
+        JUT_ASSERT(216, (p_res->ppFld != NULL) || (p_res->fldNum == 0));
+        p_res->keyNum = header->mKeyBlockNum;
+        p_res->ppKey = p_res->keyNum != 0 ?
+            new (heap, 0) JPAKeyBlock*[p_res->keyNum] : NULL;
+        JUT_ASSERT(221, (p_res->ppKey != NULL) || (p_res->keyNum == 0));
+        p_res->texNum = header->mTDB1Num;
+        p_res->mpTDB1 = NULL;
+        p_res->mUsrIdx = header->mUsrIdx;
 
         offset += 8;
-        u32 field_idx = 0;
-        u32 key_idx = 0;
+        u32 fld_no = 0;
+        u32 key_no = 0;
 
         for (int j = 0; j < header->mBlockNum; j++) {
             u32 magic = *(u32*)(data + offset);
             u32 size = *(u32*)(data + offset + 4);
             switch (magic) {
             case 'FLD1':
-                res->mpFieldBlocks[field_idx] = new (heap, 0) JPAFieldBlock(data + offset, heap);
-                field_idx++;
+                p_res->ppFld[fld_no] = new (heap, 0) JPAFieldBlock(data + offset, heap);
+                JUT_ASSERT(244, p_res->ppFld[fld_no] != NULL);
+                fld_no++;
                 break;
             case 'KFA1':
-                res->mpKeyBlocks[key_idx] = new (heap, 0) JPAKeyBlock(data + offset);
-                key_idx++;
+                p_res->ppKey[key_no] = new (heap, 0) JPAKeyBlock(data + offset);
+                JUT_ASSERT(249, p_res->ppKey[key_no] != NULL);
+                key_no++;
                 break;
             case 'BEM1':
-                res->mpDynamicsBlock = new (heap, 0) JPADynamicsBlock(data + offset);
+                p_res->pDyn = new (heap, 0) JPADynamicsBlock(data + offset);
+                JUT_ASSERT(254, p_res->pDyn != NULL);
                 break;
             case 'BSP1':
-                res->mpBaseShape = new (heap, 0) JPABaseShape(data + offset, heap);
+                p_res->pBsp = new (heap, 0) JPABaseShape(data + offset, heap);
+                JUT_ASSERT(258, p_res->pBsp != NULL);
                 break;
             case 'ESP1':
-                res->mpExtraShape = new (heap, 0) JPAExtraShape(data + offset);
+                p_res->pEsp = new (heap, 0) JPAExtraShape(data + offset);
+                JUT_ASSERT(262, p_res->pEsp != NULL);
                 break;
             case 'SSP1':
-                res->mpChildShape = new (heap, 0) JPAChildShape(data + offset);
+                p_res->pCsp = new (heap, 0) JPAChildShape(data + offset);
+                JUT_ASSERT(266, p_res->pCsp != NULL);
                 break;
             case 'ETX1':
-                res->mpExTexShape = new (heap, 0) JPAExTexShape(data + offset);
+                p_res->pEts = new (heap, 0) JPAExTexShape(data + offset);
+                JUT_ASSERT(270, p_res->pEts != NULL);
                 break;
             case 'TDB1':
-                res->mpTDB1 = (const u16*)(data + offset + 8);
+                p_res->mpTDB1 = (const u16*)(data + offset + 8);
+                break;
+            default:
+                JUT_WARN(275, "JPA : wrong type block in jpc file %d %x\n", header->mBlockNum, offset);
                 break;
             }
             offset += size;
         }
 
-        res->init(heap);
-        mgr->registRes(res);
+        p_res->init(heap);
+        p_res_mgr->registRes(p_res);
     }
 
     offset = *(u32*)(data + 0xC);
     for (int i = 0; i < *(u16*)(data + 0xA); i++) {
         u32 size = *(u32*)(data + offset + 4);
-        JPATexture* tex = new (heap, 0) JPATexture(data + offset);
-        mgr->registTex(tex);
+        JPATexture* p_tex = new (heap, 0) JPATexture(data + offset);
+        JUT_ASSERT(298, p_tex != NULL);
+        p_res_mgr->registTex(p_tex);
         offset += size;
     }
 }