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

Work around compile error in the serialization system. Fixes #1991. #2001

Merged
merged 2 commits into from Dec 22, 2017

Conversation

Projects
None yet
1 participant
@s-ludwig
Member

s-ludwig commented Dec 13, 2017

A warning is emitted because this workaround breaks safety inference.

Ping @timotheecour (just FYI)

s-ludwig added some commits Dec 13, 2017

Work around compile error for system properties in the serialization …
…system. Fixes #1991.

A warning is emitted because this workaround breaks safety inference.

@s-ludwig s-ludwig merged commit a49cff0 into master Dec 22, 2017

6 checks passed

codecov/patch 100% of diff hit (target 62.989%)
Details
codecov/project 63.033% (+0.044%) compared to 13ab841
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@s-ludwig s-ludwig deleted the issue_1991_system_property_serialization branch Dec 22, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment