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

Adding Japanese game support #379

Merged
merged 10 commits into from
Apr 21, 2023
Merged

Adding Japanese game support #379

merged 10 commits into from
Apr 21, 2023

Conversation

UTDZac
Copy link
Collaborator

@UTDZac UTDZac commented Apr 21, 2023

This adds compatibility support to allow the Tracker to play Japanese FireRed ROMs, similar to how it can play other non-English games. This is only now possible because of the support for Japanese character rendering in Bizhawk 2.9. As for mGBA, it technically displays these text characters fine, but the formatting is still messy because there is no way to properly determine the length of a UTF16 string in this context.

The translations were imported from this Github repo.

I had to do some hacking in GameSettings to allow for Japanese ROM addresses, mostly cause EWRAM and IWRAM were different. I want to say that 99% of the addresses are correct, but it's possible some of them may not work.

image

@UTDZac UTDZac requested a review from Fellshadow April 21, 2023 18:22
@UTDZac
Copy link
Collaborator Author

UTDZac commented Apr 21, 2023

Forgot to note, the changes to Battle.lua were mostly to account for nil addresses or undefined ability addresses.

Copy link
Collaborator

@Fellshadow Fellshadow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems fine enough to me, hopefully addresses work fine 👀

Comment on lines +414 to +417
[0x00630000] = { -- Japanese
versionName = "Pokemon FireRed J",
versionIndex = 7,
},
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could be worth mentioning the v1.0 here? (either directly in versionName or as a comment, just so the note is there for anyone else who looks at this)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Part of why i left this out is my lack of research to know what rom i was using and what roms are possible. I think the rom im using is 1.0. and I think its the only one that exist. but im not entirely sure. I dont have a different version that i found that i could test.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hm yeah, that's fair i suppose

@UTDZac UTDZac merged commit ebd657a into dev Apr 21, 2023
@UTDZac UTDZac deleted the utdzac/japanese-support branch April 21, 2023 19:00
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

Successfully merging this pull request may close these issues.

2 participants