kr_ps2: Compatibility Enhancement: Improved Configuration Handling

[AKuHAK]

Pull Request checklist

Note: these are not necessarily requirements

• I reformatted the code with clang-format
• I checked to make sure my submission worked
• I am the author of submission or have permission from the original author
• Requires update of the PS2SDK or other dependencies
• Others (please specify below)

Pull Request description

kr_ps2:
This pull request addresses the compatibility issue between OPL and ee_core, resolving a situation where configurations were being loaded by passing parameters, resulting in convoluted processes and difficulties in understanding the sequence of operations. The previous method was prone to exceeding parameter limits and causing confusion. The proposed changes replace the previous approach with a more efficient and organized solution.

Furthermore, the IGR exit path has been increased to 256 and logic changed so now spaces allowed in path.

PS2LOGO, PADEMU, IGR, cheats were not tested, looking for testers.

fixes #676

[uyjulian]

Code lgtm. I've been planning to do this for a while myself

[AKuHAK]

@10522 just to confirm. You tested this elf and

1. you checked first that IGR without spaces does work
2. you checked second that IGR with spaces doesn't work

Cause from your report in #676 it isnt clear if you did first step.

[10522]

mc0:/BOOT/BOOT.ELF > IGR > wLE_kHn
mc0:/BOOT/BOOT 2.ELF > IGR > wLE_kHn
mass:/BOOT/BOOT.ELF > IGR > PS2 Browser
mass:/BOOT 2.ELF > IGR > PS2 Browser

@AKuHAK, PS2LOGO works well. I have not tested PADEMU and PS2RD. Also, I could not test the long IGR path, since in the OPL settings you cannot write more than 32 characters.

[AKuHAK]

mass:/BOOT/BOOT.ELF > IGR > PS2 Browser
mass:/BOOT 2.ELF > IGR > PS2 Browser

This is different problem not related to opl. Here is an issue. Thanks for confirming that the spaces works fine.

[AKuHAK]

in the OPL settings you cannot write more than 32 characters.

WIP

[rickgaiser]

It's hard to check if everything is ok. But in general this looks like a good improvement. So LGTM.

Merge?

[AKuHAK]

Yes, it can be merged, partially tested and is ok.