Skip to content

Added WKS structs and fields#1841

Open
Knightmore wants to merge 2 commits into
aers:mainfrom
Knightmore:WKS
Open

Added WKS structs and fields#1841
Knightmore wants to merge 2 commits into
aers:mainfrom
Knightmore:WKS

Conversation

@Knightmore
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the requested changes Changes requested through comments label May 20, 2026
Copy link
Copy Markdown
Collaborator

@wolfcomp wolfcomp left a comment

Choose a reason for hiding this comment

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

Compiler optimization hides the real locations if you aren't careful

Comment thread FFXIVClientStructs/FFXIV/Client/Game/WKS/WKSManager.cs Outdated
Comment thread FFXIVClientStructs/FFXIV/Client/Game/WKS/WKSManager.cs Outdated
@Knightmore
Copy link
Copy Markdown
Contributor Author

Compiler optimization hides the real locations if you aren't careful

Thanks! Any way to not step into that trap again?
As I was just hitting my head against sub_140ECB970 where it writes those fields and a couple more that I haven't mapped in this PR yet to make sense of this overlap.

@github-actions github-actions Bot removed the requested changes Changes requested through comments label May 20, 2026
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