Skip to content

Commit

Permalink
Remove dead code, no more size pointers in load queue
Browse files Browse the repository at this point in the history
  • Loading branch information
Niko committed Feb 20, 2025
1 parent 4e3a2e4 commit fc9a353
Show file tree
Hide file tree
Showing 11 changed files with 23 additions and 30 deletions.
9 changes: 4 additions & 5 deletions decompile/General/LOAD/LOAD_17_DramFile.c
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

void DECOMP_LOAD_DramFileCallback();

void* DECOMP_LOAD_DramFile(void* bigfilePtr, int subfileIndex, int* ptrDestination, int* size, int callbackOrFlags)
void* DECOMP_LOAD_DramFile(void* bigfilePtr, int subfileIndex, int* ptrDestination, int callbackOrFlags)
{
struct LoadQueueSlot lqs;

Expand All @@ -12,8 +12,7 @@ void* DECOMP_LOAD_DramFile(void* bigfilePtr, int subfileIndex, int* ptrDestinati
lqs.flags = 0;
lqs.type = LT_DRAM;
lqs.subfileIndex = subfileIndex;
lqs.ptrDestination = DECOMP_LOAD_ReadFile(bigfilePtr, LT_DRAM, subfileIndex, 0, size, 0);
lqs.size = *size;
lqs.ptrDestination = DECOMP_LOAD_ReadFile(bigfilePtr, LT_DRAM, subfileIndex, 0, 0);
lqs.callback.funcPtr = NULL;

DECOMP_LOAD_DramFileCallback(&lqs);
Expand All @@ -27,7 +26,7 @@ void* DECOMP_LOAD_DramFile(void* bigfilePtr, int subfileIndex, int* ptrDestinati

if(callbackOrFlags == -2)
{
void* ptrDest = DECOMP_LOAD_ReadFile(bigfilePtr, LT_DRAM, subfileIndex, 0, size, DECOMP_LOAD_DramFileCallback);
void* ptrDest = DECOMP_LOAD_ReadFile(bigfilePtr, LT_DRAM, subfileIndex, 0, DECOMP_LOAD_DramFileCallback);

*ptrDestination = (int)ptrDest;
data.currSlot.ptrDestination = ptrDest;
Expand All @@ -37,5 +36,5 @@ void* DECOMP_LOAD_DramFile(void* bigfilePtr, int subfileIndex, int* ptrDestinati

// valid callback

return DECOMP_LOAD_ReadFile(bigfilePtr, LT_DRAM, subfileIndex, ptrDestination, size, DECOMP_LOAD_DramFileCallback);
return DECOMP_LOAD_ReadFile(bigfilePtr, LT_DRAM, subfileIndex, ptrDestination, DECOMP_LOAD_DramFileCallback);
}
9 changes: 4 additions & 5 deletions decompile/General/LOAD/LOAD_19_VramFile.c
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#include <common.h>

void* DECOMP_LOAD_VramFile(void* bigfilePtr, int subfileIndex, int* ptrDestination, int* size, int callbackOrFlags)
void* DECOMP_LOAD_VramFile(void* bigfilePtr, int subfileIndex, int* ptrDestination, int callbackOrFlags)
{
struct LoadQueueSlot lqs;

Expand All @@ -10,8 +10,7 @@ void* DECOMP_LOAD_VramFile(void* bigfilePtr, int subfileIndex, int* ptrDestinati
lqs.flags = 0;
lqs.type = LT_VRAM;
lqs.subfileIndex = subfileIndex;
lqs.ptrDestination = DECOMP_LOAD_ReadFile(bigfilePtr, LT_VRAM, subfileIndex, 0, size, 0);
lqs.size = *size;
lqs.ptrDestination = DECOMP_LOAD_ReadFile(bigfilePtr, LT_VRAM, subfileIndex, 0, 0);
lqs.callback.funcPtr = NULL;

DECOMP_LOAD_VramFileCallback(&lqs);
Expand All @@ -29,7 +28,7 @@ void* DECOMP_LOAD_VramFile(void* bigfilePtr, int subfileIndex, int* ptrDestinati

if(callbackOrFlags == -2)
{
void* ptrDest = DECOMP_LOAD_ReadFile(bigfilePtr, LT_VRAM, subfileIndex, 0, size, &DECOMP_LOAD_VramFileCallback);
void* ptrDest = DECOMP_LOAD_ReadFile(bigfilePtr, LT_VRAM, subfileIndex, 0, &DECOMP_LOAD_VramFileCallback);

*ptrDestination = (int)ptrDest;
data.currSlot.ptrDestination = ptrDest;
Expand All @@ -39,5 +38,5 @@ void* DECOMP_LOAD_VramFile(void* bigfilePtr, int subfileIndex, int* ptrDestinati

// valid callback

return DECOMP_LOAD_ReadFile(bigfilePtr, LT_VRAM, subfileIndex, ptrDestination, size, &DECOMP_LOAD_VramFileCallback);
return DECOMP_LOAD_ReadFile(bigfilePtr, LT_VRAM, subfileIndex, ptrDestination, &DECOMP_LOAD_VramFileCallback);
}
10 changes: 4 additions & 6 deletions decompile/General/LOAD/LOAD_21_ReadFile.c
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#include <common.h>

void* DECOMP_LOAD_ReadFile(struct BigHeader* bigfile, u_int loadType, int subfileIndex, void *ptrDst, int *size, void * callback)
void* DECOMP_LOAD_ReadFile(struct BigHeader* bigfile, u_int loadType, int subfileIndex, void *ptrDst, void * callback)
{
// param1 is the Pointer to CD position of BIGFILE

Expand All @@ -14,8 +14,6 @@ void* DECOMP_LOAD_ReadFile(struct BigHeader* bigfile, u_int loadType, int subfil
struct BigEntry* entry = BIG_GETENTRY(bigfile);
int eSize = entry[subfileIndex].size;
int eOffs = entry[subfileIndex].offset;

*size = eSize;

#ifndef USE_PCDRV
CdIntToPos(bigfile->cdpos + eOffs, &cdLoc);
Expand All @@ -30,11 +28,11 @@ void* DECOMP_LOAD_ReadFile(struct BigHeader* bigfile, u_int loadType, int subfil
lqs->flags |= 1;

// make sure RAM has room for sector alignment
ptrDst = (void *)DECOMP_MEMPACK_AllocMem((*size + 0x7ffU) & 0xfffff800); // "FILE"
ptrDst = (void *)DECOMP_MEMPACK_AllocMem((eSize + 0x7ffU) & 0xfffff800); // "FILE"

// undo sector-align alloc,
// allocate just "needed" bytes
DECOMP_MEMPACK_ReallocMem(*size);
DECOMP_MEMPACK_ReallocMem(eSize);

if (loadType == LT_VRAM)
DECOMP_MEMPACK_ReallocMem(0);
Expand Down Expand Up @@ -69,7 +67,7 @@ void* DECOMP_LOAD_ReadFile(struct BigHeader* bigfile, u_int loadType, int subfil
while (1)
{
uVar5 = CdControl(CdlSetloc, &cdLoc, &paramOutput[0]);
uVar5 &= CdRead(*size + 0x7ffU >> 0xb, ptrDst, 0x80);
uVar5 &= CdRead(eSize + 0x7ffU >> 0xb, ptrDst, 0x80);

#ifndef REBUILD_PC
// if no errors
Expand Down
2 changes: 1 addition & 1 deletion decompile/General/LOAD/LOAD_32_LangFile.c
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ void DECOMP_LOAD_LangFile(int bigfilePtr, int lang)
lngFile =
DECOMP_LOAD_ReadFile(
(struct BigHeader*)bigfilePtr, 1, BI_LANGUAGEFILE + lang,
(void*)sdata->lngFile, &size, NULL);
(void*)sdata->lngFile, NULL);

// This is not ReadFileAsync, this is ReadFile,
// so the program halts until completion of read
Expand Down
1 change: 0 additions & 1 deletion decompile/General/LOAD/LOAD_34_AppendQueue.c
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ void DECOMP_LOAD_AppendQueue(int bigfile, int type, int fileIndex, void* destina
lqs->type = type;
lqs->subfileIndex = fileIndex;
lqs->ptrDestination = destinationPtr;
lqs->size = 0;
lqs->callback.funcPtr = callback;

sdata->queueLength++;
Expand Down
6 changes: 2 additions & 4 deletions decompile/General/LOAD/LOAD_36_NextQueuedFile.c
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ void DECOMP_LOAD_NextQueuedFile()
LT_RAW,
curr->subfileIndex,
curr->ptrDestination,
&curr->size,
DECOMP_LOAD_CDRequestCallback);
}

Expand All @@ -49,7 +48,6 @@ void DECOMP_LOAD_NextQueuedFile()
(void*)curr->ptrBigfileCdPos,
(int)curr->subfileIndex,
(int*)curr->ptrDestination,
(int*)&curr->size,
(int)curr->callback.funcPtr);
}

Expand All @@ -60,7 +58,6 @@ void DECOMP_LOAD_NextQueuedFile()
(void*)curr->ptrBigfileCdPos,
(int)curr->subfileIndex,
(int*)curr->ptrDestination,
(int*)&curr->size,
(int)curr->callback.funcPtr);
}

Expand All @@ -79,7 +76,8 @@ void DECOMP_LOAD_NextQueuedFile()
{
struct LoadQueueSlot* curr = &data.currSlot;

// Use callback if present
// UNUSED, there's no callback after completion
// of any VRAM file because its never the last in the queue
if(curr->callback.funcPtr != 0)
{
(*curr->callback.funcPtr)(curr);
Expand Down
2 changes: 1 addition & 1 deletion decompile/General/LOAD/LOAD_44_TenStages.c
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ int DECOMP_LOAD_TenStages(struct GameTracker* gGT, int loadingStage, struct BigH

#ifndef REBUILD_PC
// Load Intro TIM for Copyright Page from VRAM file
DECOMP_LOAD_VramFile(bigfile, 0x1fe, 0, &vramSize, 0xffffffff);
DECOMP_LOAD_VramFile(bigfile, 0x1fe, 0, 0xffffffff);
DECOMP_MainInit_VRAMDisplay();
#endif

Expand Down
4 changes: 2 additions & 2 deletions decompile/General/MAIN/MainMain.c
Original file line number Diff line number Diff line change
Expand Up @@ -845,7 +845,7 @@ void StateZero()
#ifndef FastBoot
#ifndef REBUILD_PC
// Load Intro TIM for "SCEA Presents" from VRAM file
DECOMP_LOAD_VramFile(sdata->ptrBigfile1, 0x1fd, 0, &vramSize, 0xffffffff);
DECOMP_LOAD_VramFile(sdata->ptrBigfile1, 0x1fd, 0, 0xffffffff);
DECOMP_MainInit_VRAMDisplay();
#endif
#endif
Expand Down Expand Up @@ -882,7 +882,7 @@ void StateZero()
// the area between 2 screen buffers and top right corner in vram
// sdata->ptrBigfile1 is the Pointer to "cd position of bigfile"
// Add a bookmark before loading (param_3 is 0 in the call)
DECOMP_LOAD_VramFile(sdata->ptrBigfile1, 0x102, 0, &vramSize, 0xffffffff);
DECOMP_LOAD_VramFile(sdata->ptrBigfile1, 0x102, 0, 0xffffffff);

sdata->mainGameState = 3;

Expand Down
2 changes: 1 addition & 1 deletion decompile/General/zGlobal_DATA.c
Original file line number Diff line number Diff line change
Expand Up @@ -2728,7 +2728,7 @@ struct Data data =
.type = 0,
.subfileIndex = -1,
.ptrDestination = 0,
.size = 0,
.size_UNUSED = 0,
.callback.flags = 0
},

Expand Down
6 changes: 3 additions & 3 deletions include/decomp_functions.h
Original file line number Diff line number Diff line change
Expand Up @@ -281,8 +281,9 @@ void DECOMP_LOAD_Callback_Overlay_231(void);
void DECOMP_LOAD_Callback_Overlay_232(void);
void DECOMP_LOAD_Callback_Overlay_233(void);
void DECOMP_LOAD_ReadFileASyncCallback(CdlIntrResult result, uint8_t* unk);
void* DECOMP_LOAD_ReadFile(struct BigHeader* bigfile, u_int loadType, int subfileIndex, void* destination, int *size, void * callback);
void* DECOMP_LOAD_VramFile(void* bigfilePtr, int subfileIndex, int* ptrDestination, int* size, int callbackOrFlags);
void* DECOMP_LOAD_ReadFile(struct BigHeader* bigfile, u_int loadType, int subfileIndex, void* destination, /*int *size,*/ void * callback);
void* DECOMP_LOAD_VramFile(void* bigfilePtr, int subfileIndex, int* ptrDestination, /*int* size,*/ int callbackOrFlags);
void* DECOMP_LOAD_DramFile(void* bigfilePtr, int subfileIndex, int* ptrDestination, /*int* size,*/ int callbackOrFlags);
void* DECOMP_LOAD_ReadDirectory(char* filename);
void* DECOMP_LOAD_XnfFile(char* filename, void* ptrDestination, int* size);
int DECOMP_LOAD_TenStages(struct GameTracker* gGT, int loadingStage, struct BigHeader* bigfile);
Expand Down Expand Up @@ -828,7 +829,6 @@ void DECOMP_LOAD_GlobalModelPtrs_MPK(void);
void DECOMP_LOAD_OvrEndRace(unsigned int param_1);
void DECOMP_LOAD_OvrLOD(unsigned int param_1);
void DECOMP_LOAD_OvrThreads(unsigned int param_1);
void* DECOMP_LOAD_DramFile(void* bigfilePtr, int subfileIndex, int* ptrDestination, int* size, int callbackOrFlags);
void DECOMP_LOAD_DriverMPK(unsigned int param_1, int levelLOD, void (*param_3)(struct LoadQueueSlot*));
void DECOMP_LibraryOfModels_Clear(struct GameTracker* gGT);
void DECOMP_DecalGlobal_Store(struct GameTracker* gGT, struct LevTexLookup* LTL);
Expand Down
2 changes: 1 addition & 1 deletion include/namespace_Load.h
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ struct LoadQueueSlot
void* ptrDestination;

// 0x10
unsigned int size;
unsigned int size_UNUSED;

// 0x14
union
Expand Down

0 comments on commit fc9a353

Please sign in to comment.