Skip to content

Fix access violation crash when using Ultimate ASI Loader#1

Open
charlesthobe wants to merge 4 commits into
BuilderDemo7:masterfrom
charlesthobe:pr
Open

Fix access violation crash when using Ultimate ASI Loader#1
charlesthobe wants to merge 4 commits into
BuilderDemo7:masterfrom
charlesthobe:pr

Conversation

@charlesthobe

@charlesthobe charlesthobe commented May 15, 2026

Copy link
Copy Markdown

This makes the game not crash when using newer versions of Ultimate ASI Loader

Rationale for using newer versions of Ultimate ASI Loader; newer versions allow loading modded files without touching original game files as explained here

@charlesthobe charlesthobe force-pushed the pr branch 2 times, most recently from 284e726 to 1d135f2 Compare May 25, 2026 00:12
@charlesthobe charlesthobe marked this pull request as draft May 25, 2026 00:15
@charlesthobe charlesthobe marked this pull request as ready for review May 25, 2026 01:14
@charlesthobe charlesthobe force-pushed the pr branch 3 times, most recently from f8a5d55 to 113d114 Compare May 25, 2026 05:07
@charlesthobe

charlesthobe commented May 25, 2026

Copy link
Copy Markdown
Author

I need feedback on whether to resolve datatypes explicitly
e.g

WriteAt(0x641420, SettingsMgr->fInstances_Spawn_Radius);

vs

WriteAt<float>(0x641420, SettingsMgr->fInstances_Spawn_Radius);

in the first example it figures out that the data is float based on the second argument in the function call. In the latter example the type explicitly stated.

The code is functionaly the same, no bugs were fixed
This also fixes access violation crash when using newer versions of Ultimate ASI Loader
@charlesthobe charlesthobe marked this pull request as ready for review May 25, 2026 22:57
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.

1 participant