Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

dyanikoglu
Copy link
Contributor

Not sure why those calls were not included in serialize functions.

@MothDoctor MothDoctor self-assigned this Aug 27, 2024
@MothDoctor MothDoctor added the improvement Polishing existing functionality label Nov 17, 2024
@rweber89
Copy link

@dyanikoglu this change doesnt include a version for the manual serialization, so not all changes in newer version will be backwards compatible.

@MothDoctor
Copy link
Contributor

Indeed, this might break loading already created saves in released games.
Is there a specific reason why you proposed these changes? I understand that current code feels incomplete, but works fine with properties marked as SaveGame?

@dyanikoglu
Copy link
Contributor Author

In my custom save system, I'm serializing those structs manually and writing into saved data. If it's likely break compatibility with existing projects, the PR can be closed.

@MothDoctor MothDoctor force-pushed the 5.x branch 2 times, most recently from 402b773 to 1a4a2d9 Compare March 1, 2025 19:50
@MothDoctor MothDoctor force-pushed the 5.x branch 3 times, most recently from 4b00fe8 to daa3fef Compare May 15, 2025 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Polishing existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants