-
Notifications
You must be signed in to change notification settings - Fork 120
bugfix(filesystem): Prevent loading wrong INIZH.big from Data/INI directory to prevent INI CRC mismatch #1879
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
bugfix(filesystem): Prevent loading wrong INIZH.big from Data/INI directory to prevent INI CRC mismatch #1879
Conversation
…atches on EA App/Origin installs
|
Where is the Delete code? Can that be removed if it does not work anyway? |
Core/GameEngineDevice/Source/StdDevice/Common/StdBIGFileSystem.cpp
Outdated
Show resolved
Hide resolved
Core/GameEngineDevice/Source/Win32Device/Common/Win32BIGFileSystem.cpp
Outdated
Show resolved
Hide resolved
Core/GameEngineDevice/Source/Win32Device/Common/Win32BIGFileSystem.cpp
Outdated
Show resolved
Hide resolved
GeneralsMD/Code/GameEngine/Source/Common/GameEngine.cpp line 402, and we can remove it yes. I'm unsure if it works on other installs eg the original CD, but we don't need it in any case. Remove it? |
|
I would say if this new change avoids all trouble, then remove the hack from Kris and transfer his original comment to the new hack to clarify the intention. |
Why does it not delete it? Does it fail because it does not have Admin privileges? |
Yes, from the issue "this does not work on the EA app/origin version of the game because the folder is owned by the app and is not writable." |
xezon
left a comment
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.
Remove it?
Yes. Move the EA comment to the new place.
Core/GameEngineDevice/Source/StdDevice/Common/StdBIGFileSystem.cpp
Outdated
Show resolved
Hide resolved
Core/GameEngineDevice/Source/StdDevice/Common/StdBIGFileSystem.cpp
Outdated
Show resolved
Hide resolved
Done |
xezon
left a comment
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.
Looks good.
This change skips loading any BIG file matching
Data\INI\INIZH.bigduring archive initialization.Testing