Fix "overflow error: int too big to convert error"#11
Closed
twise2 wants to merge 1 commit intoSiegeEngineers:mainfrom
Closed
Fix "overflow error: int too big to convert error"#11twise2 wants to merge 1 commit intoSiegeEngineers:mainfrom
twise2 wants to merge 1 commit intoSiegeEngineers:mainfrom
Conversation
Tech tree errors when trying to write an int 8
Author
Member
|
Uhhhh… Well the error message is correct. You simply can't have more than 255 unit connections in the tech tree section of the dat file. That field may be a I think the tech tree section of the dat file is not used by AoE2 DE anyway? Not sure. In any case, your "fix" sadly only breaks the file format 🥲 |
Author
|
hmm, you appear to be correct as usual. If that's the case it may be worth pulling the file entirely. It's throwing me off a bit still because the length before I am appending appears to be length appears to be 290 before the overflow is caused. |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.


Tech tree errors when trying to write an int 8