Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upMissing feature gate for the default of `'static` #36216
Comments
nikomatsakis
added
A-typesystem
P-high
T-compiler
E-mentor
labels
Sep 2, 2016
This comment has been minimized.
This comment has been minimized.
|
Sure. I guess I'll need to look into how feature gates are implemented and then conditionally use the old/new behavior. |
This comment has been minimized.
This comment has been minimized.
|
I am up for mentoring this for whomever (hence the E-mentor tag). This was my description of how to add the change. I think what you need to do to fix this is as follows:
|
This comment has been minimized.
This comment has been minimized.
|
ok @llogiq above comment may help :) |
This comment has been minimized.
This comment has been minimized.
|
closed by #36332 |
nikomatsakis
closed this
Sep 13, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
nikomatsakis commentedSep 2, 2016
We neglected to add a feature gate for the new default to
'staticin statics and constants (added in #35915). Need to fix this pronto.cc @llogiq -- you up for that? (Sorry to ask you twice, just decided I wanted a way to track this so I don't forget.)