Skip to content

gamenet default fallback on decoding - #149

Merged
heinrich5991 merged 2 commits into
heinrich5991:masterfrom
Patiga:snap-default-fallback
Aug 25, 2026
Merged

heinrich5991 merged 2 commits into
heinrich5991:masterfrom
Patiga:snap-default-fallback

Conversation

@Patiga

@Patiga Patiga commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator

Modify gamenet generation script to consider default values. kinda(?) solves #122

Note that this does not fix the case mentioned by @Jupeyy, as there is no default values for tunes in the spec.
However, this does in particular add default values to DdnetCharacter, which is arguably one of the most important snap objects.

The default value is only used, if a UnexpectedEnd error occurs.

@Patiga

Patiga commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator Author

I switched to f-strings as I'm more used to writing them. At the time this script was written they probably didn't exist yet. If I should use the older formatting style, I can do that ofc

@heinrich5991 heinrich5991 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Yea, looks sensible. Thanks. :)

@heinrich5991
heinrich5991 merged commit 1f6a78f into heinrich5991:master Aug 25, 2026
8 checks passed
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