Skip to content
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

Always RenameInOutputMod instead of #[rust_name] #784

Merged
merged 4 commits into from
Feb 11, 2022

Conversation

adetaylor
Copy link
Collaborator

Previously we generated slightly simpler output by using
#[rust_name] attributes on the cxx::bridge mod. With this change,
we never do that and instead always perform any such renames
with a use statement in the output set of mods.

This simplifies our code, while making output more verbose;
it is a prerequisite for a subsequent step towards solving #486.

Previously we generated slightly simpler output by using
`#[rust_name]` attributes on the cxx::bridge mod. With this change,
we never do that and instead always perform any such renames
with a `use` statement in the output set of mods.

This simplifies our code, while making output more verbose;
it is a prerequisite for a subsequent step towards solving #486.
@adetaylor adetaylor force-pushed the never-use-rust-name-attribute branch from 6ab36e0 to d080037 Compare February 10, 2022 22:31
Because they don't take &self so that cxx doesn't consider
them methods on the C++ side.
@adetaylor adetaylor force-pushed the never-use-rust-name-attribute branch from d080037 to a4f9b51 Compare February 10, 2022 22:47
@adetaylor adetaylor merged commit 4966584 into main Feb 11, 2022
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.

1 participant