Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Serialization: Host serialization cleanup #328

Closed
wants to merge 2 commits into from
Closed

Serialization: Host serialization cleanup #328

wants to merge 2 commits into from

Commits on Aug 6, 2019

  1. Make serialization methods more concise 

    Reviewed the style of the (de)serialization methods. Removed redundant
or rather cryptic instructions. Leveraged comprehensions, dried a few
bits of code. Made the code more concise and expressive. Unified names
a bit.
    @Glutexo
    Glutexo committed Aug 6, 2019
    Configuration menu
    Copy the full SHA
    356e4d7 View commit details
    Browse the repository at this point in the history

  2. Trust the deserialized values 

    Change the (de)serialization logic, so the serialization expects that
the object is valid. All default values and validation happens now upon
deserialization. That concentrates the logic into a single place.
    @Glutexo
    Glutexo committed Aug 6, 2019
    Configuration menu
    Copy the full SHA
    5c099fe View commit details
    Browse the repository at this point in the history