Skip to content
This repository has been archived by the owner on Feb 20, 2019. It is now read-only.

StaticOnly should be enabled by default #423

Open
jvican opened this issue May 9, 2016 · 2 comments
Open

StaticOnly should be enabled by default #423

jvican opened this issue May 9, 2016 · 2 comments

Comments

@jvican
Copy link
Member

jvican commented May 9, 2016

Currently, if we want to ensure that picklers/unpicklers are generated statically at compile-time, this flag has to be imported in the scope.

I would propose to turn it around and create a flag called AllowDynamic that would make the program compile in case it has to generate picklers/unpicklers at runtime. Otherwise, StaticOnly would be chosen by default.

I think this is a reasonable way to deal with this, since most of our users expect static picklers.

@jvican
Copy link
Member Author

jvican commented May 9, 2016

WDYT @jsuereth ?

@andreaTP
Copy link

andreaTP commented May 9, 2016

👍

@jvican jvican added the ready label May 9, 2016
@jvican jvican added this to the 0.11.0-M1 milestone May 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants