-
Notifications
You must be signed in to change notification settings - Fork 614
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
Document Save.cutsceneIndex
and adjacent data / code
#2286
Open
Feacur
wants to merge
35
commits into
zeldaret:main
Choose a base branch
from
Feacur:doc/cutscene_index
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+269
−233
Open
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
53cd39d
introduce `enum CutsceneIndex` as a first step
Feacur 050c736
crudely rename `enum CutsceneIndex` entries
Feacur 4049e33
run `check_format.py ...` separetely
Feacur 2ae3bff
apply a draft PR suggestion
Feacur 8cbdf80
rename `CS_INDEX_*` with PR suggestions
Feacur 2ec57c9
improve `CS_INDEX_*` naming, rem `ENTRANCE` suffix
Feacur b8953fd
apply Jenkins fixes
Feacur 32a2dd0
apply a PR suggestion
Feacur 2bd2ca2
remove redundant comments, fix the header comment
Feacur 6eaaa87
rename the flag for freeing `nextCutsceneIndex`
Feacur d9bbd67
rename the flag for automatic `sceneLayer` calc
Feacur eb75a6d
use explicit scene layer names instead of numbers
Feacur 02a977f
ok, now the last two `CS_INDEX_*` are less unknown
Feacur bee188d
Merge branch 'main' into doc/cutscene_index
Feacur 211bf74
apply Jenkins fixes
Feacur 450e24e
Merge branch 'main' into doc/cutscene_index
Feacur 3530507
Merge branch 'main' into doc/cutscene_index
Feacur 4b3e0dc
run `fix_bss` locally
Feacur 1d84bca
revert all local to the branch bss shifts
Feacur 133a74f
ugh, use defines instead of an enum
Feacur 0cbe8a4
still, add a BSS pragma for `z_demo.c`
Feacur 279759f
Merge branch 'main' into doc/cutscene_index
Feacur 5552e92
apply review changes
Feacur f728d55
factor out a cutscene layer formula
Feacur 106cb10
give scene layers names throughout code
Feacur 50acfc2
apply Jenkins fixes
Feacur 6efea0a
tweak `CS_INDEX` descriptions
Feacur e9adab6
apply PR review suggestions
Feacur 67f0eef
rename "auto" -> "default"
Feacur cbaad70
apply PR review suggestions
Feacur 0b5bc3d
Merge branch 'main' into doc/cutscene_index
Feacur 2edc035
Merge branch 'main' into doc/cutscene_index
Feacur 015380c
apply Jenkins fixes
Feacur 787bd84
Apply suggestions from code review
Feacur aacefa3
Merge branch 'main' into doc/cutscene_index
Feacur File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still don't think the name makes much sense, but I also don't have a strong idea for what to call this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will once again advocate for CS_INDEX_D (and CS_INDEX_F instead for 0xFFFF), since these also indicate a scripted cutscene despite not corresponding to any scene layer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair, although now that I think about it I kind of just wish the whole value was written out in full, like...
CS_INDEX_0
->CS_INDEX_FFF0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At this point I won't mind. Either way, you should know better than me, and I already expressed my standpoint. I mean, no subtext, I just trust your decision