Skip to content

Allow legacy custom derive authors to disable warnings in downstream crates#38533

Merged
bors merged 1 commit into
rust-lang:masterfrom
jseyfried:legacy_custom_derive_deprecation
Dec 23, 2016
Merged

Allow legacy custom derive authors to disable warnings in downstream crates#38533
bors merged 1 commit into
rust-lang:masterfrom
jseyfried:legacy_custom_derive_deprecation

Conversation

@jseyfried

Copy link
Copy Markdown
Contributor

This PR allows legacy custom derive authors to use a pre-deprecated method registry.register_custom_derive() instead of registry.register_syntax_extension() to avoid downstream deprecation warnings.

r? @nrc

@jseyfried

Copy link
Copy Markdown
Contributor Author

cc @SergioBenitez

@nrc nrc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

r = me with some docs

Comment thread src/librustc_plugin/registry.rs Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you document here when an author should use this and what its effects are please?

@jseyfried
jseyfried force-pushed the legacy_custom_derive_deprecation branch from b0243d6 to fa3e910 Compare December 23, 2016 05:44
@jseyfried
jseyfried force-pushed the legacy_custom_derive_deprecation branch from fa3e910 to c12fc66 Compare December 23, 2016 05:49
@jseyfried

Copy link
Copy Markdown
Contributor Author

@bors r=nrc

@bors

bors commented Dec 23, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit c12fc66 has been approved by nrc

@bors

bors commented Dec 23, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit c12fc66 with merge 467a7f0...

bors added a commit that referenced this pull request Dec 23, 2016
Allow legacy custom derive authors to disable warnings in downstream crates

This PR allows legacy custom derive authors to use a pre-deprecated method `registry.register_custom_derive()` instead of `registry.register_syntax_extension()` to avoid downstream deprecation warnings.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants