Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

0.9.2 breaks save file of Pokemon White 2 Japanese version #1158

Open
cloudfish opened this issue Jul 17, 2021 · 7 comments
Open

0.9.2 breaks save file of Pokemon White 2 Japanese version #1158

cloudfish opened this issue Jul 17, 2021 · 7 comments

Comments

@cloudfish
Copy link

Steps to reproduce this:

  1. Save game in Japanese White2 in 0.9.1.
  2. Load game in Japanese White2 in 0.9.2. Will cause a bunch of "undefined ARM9 instruction" errors and a blank screen.
  3. Restart the game. It cannot read the save file and will prompt to start a new game. The save file can still be read by either Japanese or English version of the game in 0.9.1, though.

I guess it has something to do with the new save file handling mechanism?
pnw2jpn.sav.zip
output0.9.1.txt
output0.9.2.txt

@RSDuck
Copy link
Member

RSDuck commented Jul 17, 2021

I cannot reproduce. The savefile loads fine for me in both 0.9.1, 0.9.2 and latest source.

@cloudfish
Copy link
Author

cloudfish commented Jul 17, 2021

It seems that only ROMs of "non-standard" size will not work in 0.9.2. I uploaded my version of the ROM to Google drive. Could you try to reproduce with this ROM? (It is 303MB as opposed to 512MB, and melonDS already warns about this in the log I uploaded)
redacted

A save file is actually not needed to reproduce this. I am able to start a new game (without the save file) using this ROM in 0.9.1, but in 0.9.2, after the initial kana/kanji selection, the game shows a blue screen complaining that the save function is not working.

P.S. While I am able to workaround this using a 512MB Japanese ROM, I used Japanese ROM only to minimize the example; I am actually using a fan-made Chinese translation for which no full-sized version is available.

@Arisotura
Copy link
Member

after investigation it appears that your ROM is a patched version for flashcarts and stuff, that has the IR-comm code disabled

however, since 0.9.2, melonDS uses the gamecode to detect that it should emulate an IR cart, and enforces that

(the same ROM only works in 0.9.1 and older because the implementation was a hack, and was more lax)

a 'solution' would be altering the gamecode so it doesn't start with IRxx. we're debating what we can do on our side for these cases. (from an accuracy standpoint, such a patched ROM would not work on its original cart, but it works on flashcarts and such, so this is problematic)

@RSDuck
Copy link
Member

RSDuck commented May 11, 2023

fixed by cd4ef57 ca7fb4f pasted wrong link long time ago

@RSDuck
Copy link
Member

RSDuck commented Jan 28, 2024

this issue isn't actually fixed. It seems to be distinct from the one with the bad DSi header. The rom I have which reproduces it seems to be valid (atleast in the places I looked) and still produces the error when not handling it as a DSi cartridge. It works in desmume though.

@RSDuck RSDuck reopened this Jan 28, 2024
@RSDuck
Copy link
Member

RSDuck commented Jan 28, 2024

I forgot to mention, the game works without problem when not treating it as an IR rom.

JesseTG/melonds-ds#150 seems to be yet another variation. I don't know whether it has the same root or it is a different one.

@GlazedBelmont
Copy link

GlazedBelmont commented Feb 19, 2024

on my end, I fixed it by repatching my rom. (for the issue linked right above)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants