Skip to content

Conversation

@wilzbach
Copy link
Contributor

@wilzbach wilzbach commented Jan 9, 2018

primitiveTypeInfo is a relict from the old doFormat days.
It's not used anywhere anymore.

See also: #4978

Separated from: #6014

@wilzbach wilzbach requested a review from andralex as a code owner January 9, 2018 19:37
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @wilzbach!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

@wilzbach wilzbach changed the title Remove unused primitiveTypeInfo and deprecate unused Mangle Remove unused std.format.primitiveTypeInfo and deprecate unused std.format.Mangle Jan 9, 2018
Copy link
Member

@quickfur quickfur left a comment

Choose a reason for hiding this comment

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

LGTM.

It's doubtful whether any user code would depend on this, but the deprecation should serve as fair warning.

@wilzbach
Copy link
Contributor Author

wilzbach commented Jan 9, 2018

It's doubtful whether any user code would depend on this, but the deprecation should serve as fair warning.

I don't think anyone is using this either, but technically it's an undocumented, public symbol, so 🤷‍♂️

@dlang-bot dlang-bot merged commit 671b4b6 into dlang:master Jan 9, 2018
@wilzbach wilzbach deleted the deprecate-mangle branch January 10, 2018 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants