Merge branch 'port' of https://github.com/fgsfdsfgs/perfect_dark into port-debugger

Makefile.port

@@ -230,6 +230,10 @@ CXX_O_FILES := $(patsubst %.cpp, $(B_DIR)/%.o, $(CXX_FILES))
 
 O_FILES := $(C_O_FILES) $(CXX_O_FILES)
 
+ifneq (,$(findstring windows,$(TARGET_PLATFORM)))
+  O_FILES += $(B_DIR)/icon.res
+endif
+
 INCLUDES = \
   -I include \
   -I include/PR \
@@ -248,7 +252,7 @@ else
   LD := $(CC)
 endif
 
-$(C_O_FILES): OPT_LVL := $(GCC_OPT_LVL)
+$(O_FILES): OPT_LVL := $(GCC_OPT_LVL)
 
 COMMON_CFLAGS := $(C_DEFINES) -DAVOID_UB=1 $(INCLUDES) $(TARGET_CFLAGS) -g \
   -fno-inline-functions \
@@ -323,6 +327,9 @@ $(B_DIR)/%.o: %.cpp $(JSON_HEADERS)
 	@mkdir -p $(dir $@)
 	$(CXX) -c $(CXXFLAGS) $(OPT_LVL) -o $@ $<
 
+$(B_DIR)/%.res: dist/windows/%.rc
+	windres $< -O coff -o $@
+
 ################################################################################
 # Clean rules
 

README.md

@@ -137,5 +137,6 @@ It might be possible to build a 32-bit ARM executable, but this has not been tes
 * [Ship of Harkinian team](https://github.com/Kenix3/libultraship/tree/main/src/graphic/Fast3D), Emill and MaikelChan for the libultraship version of fast3d that this port uses;
 * lieff for [minimp3](https://github.com/lieff/minimp3);
 * Mouse Injector and 1964GEPD authors for some of the 60FPS- and mouselook-related fixes;
+* NicNamSam for the icon;
 * everyone who has submitted pull requests and issues to this repository and tested the port;
 * probably more I'm forgetting.

dist/windows/icon.rc

@@ -0,0 +1 @@
+id ICON icon.ico

port/include/romdata.h

@@ -22,6 +22,6 @@ u8 *romdataSegGetData(const char *segName);
 u8 *romdataSegGetDataEnd(const char *segName);
 u32 romdataSegGetSize(const char *segName);
 
-void gbcRomCheck(void);
+s32 romdataCheckGbcRom(void);
 
 #endif

port/src/main.c

@@ -109,7 +109,8 @@ int main(int argc, const char **argv)
 	audioInit();
 	romdataInit();
 	debuggerInit();
-	gbcRomCheck();
+
+	g_ValidGbcRomFound = romdataCheckGbcRom();
 
 	gameInit();
 

port/src/romdata.c

@@ -11,9 +11,6 @@
 #include "system.h"
 #include "preprocess.h"
 #include "platform.h"
-#ifndef PLATFORM_N64
-#include "bss.h"
-#endif
 
 /**
  * asset files and ROM segments can be replaced by optional external files,
@@ -392,43 +389,51 @@ s32 romdataInit(void)
 	return 0;
 }
 
-void gbcRomCheck(void)
+static inline bool romdataCheckGbcRomContents(const u8 *gbcRomFile, const u32 gbcRomSize)
 {
-	u32 gbcRomSize;
-	u8* gbcRomFile = fsFileLoad(GBC_ROM_NAME, &gbcRomSize);
-
-	if (!gbcRomFile) {
-		return;
-	}
-
 	if (gbcRomSize != GBC_ROM_SIZE) {
-		free(gbcRomFile);
-		return;
+		return false;
 	}
 
 	// ROM title
-
 	if (memcmp(gbcRomFile + 0x134, "PerfDark   VPDE", 15) != 0) {
-		free(gbcRomFile);
-		return;
+		return false;
 	}
 
 	// Licensee code
-
 	if (memcmp(gbcRomFile + 0x144, "4Y", 2) != 0) {
-		free(gbcRomFile);
-		return;
+		return false;
 	}
 
 	// Header and global checksums
-
 	if (gbcRomFile[0x14D] != 0xA1 || gbcRomFile[0x14E] != 0xAD || gbcRomFile[0x14F] != 0x0F) {
-		free(gbcRomFile);
-		return;
+		return false;
 	}
 
-	g_validGbcRomFound = true;
-	free(gbcRomFile);
+	return true;
+}
+
+s32 romdataCheckGbcRom(void)
+{
+	if (fsFileSize(GBC_ROM_NAME) < 0) {
+		// bail early if it doesn't exist to avoid generating error messages
+		return false;
+	}
+
+	u32 gbcRomSize = 0;
+	u8 *gbcRomFile = fsFileLoad(GBC_ROM_NAME, &gbcRomSize);
+	if (!gbcRomFile) {
+		return false;
+	}
+
+	const bool ret = romdataCheckGbcRomContents(gbcRomFile, gbcRomSize);
+	sysMemFree(gbcRomFile);
+
+	if (ret) {
+		sysLogPrintf(LOG_NOTE, "romdataCheckGbcRom: valid GBC rom found");
+	}
+
+	return ret;
 }
 
 s32 romdataFileGetSize(s32 fileNum)

src/game/pak.c

@@ -330,7 +330,7 @@ bool var80075d14 = true;
 #endif
 
 #ifndef PLATFORM_N64
-bool g_validGbcRomFound = false;
+bool g_ValidGbcRomFound = false;
 #endif
 
 u32 pakGetBlockSize(s8 device)
@@ -5997,7 +5997,7 @@ bool gbpakIsAnyPerfectDark(void)
 
 	return false;
 #else
-	return g_validGbcRomFound;
+	return g_ValidGbcRomFound;
 #endif
 }
 

src/include/bss.h

@@ -294,7 +294,7 @@ extern struct chrdata *g_MpBotChrPtrs[MAX_BOTS];
 extern s32 g_JpnMaxCacheItems;
 extern s32 var8009d370jf;
 #ifndef PLATFORM_N64
-extern bool g_validGbcRomFound;
+extern bool g_ValidGbcRomFound;
 #endif
 
 #endif