Skip to content

jakx: Commit existing work from other PRs#4112

Merged
xTVaser merged 38 commits intoopen-goal:masterfrom
xTVaser:v/jakx-work
Jan 1, 2026
Merged

jakx: Commit existing work from other PRs#4112
xTVaser merged 38 commits intoopen-goal:masterfrom
xTVaser:v/jakx-work

Conversation

@xTVaser
Copy link
Copy Markdown
Member

@xTVaser xTVaser commented Dec 31, 2025

This attempts to get into master whatever work was done in this PR / it's earlier PR #3965

I don't want this work to be lost / floating around in massive PRs.

However the changes are:

  • switch to ntsc_v1 instead of PAL as the development target, as we have done for all other games
  • remove most of the copied-from-jak2/3 changes as they need to be confirmed during the decompilation process not just assumed
  • avoids committing any changes to game/kernel/common as it was not clear to me if these were changes made in jak x's kernel that were not properly broken out into it's own functions. We don't want to accidentally introduce bugs into jak1-3's kernel code.
    • in other words, if the change in the kernel only happens in jak x...it should likely be specific to jak x's kernel, not common.

VodBox and others added 30 commits August 11, 2024 23:03
it's unclear if these are changes that happened in jak x's kernel and therefore should have been properly split out into override functions, or if they are improvements on the original
@xTVaser xTVaser merged commit d3cc739 into open-goal:master Jan 1, 2026
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants