Skip to content

options: Allow configuring the generation of constructors. #606

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

Closed
wants to merge 1 commit into from

Conversation

emilio
Copy link
Contributor

@emilio emilio commented Apr 3, 2017

Fixes #556

Sorry for the delay fixing this btw.

@highfive
Copy link

highfive commented Apr 3, 2017

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!

@emilio
Copy link
Contributor Author

emilio commented Apr 3, 2017

r? @fitzgen or @upsuper

@emilio
Copy link
Contributor Author

emilio commented Apr 3, 2017

Self-stamping this since it's trivial and collides with similar changes for destructors.

@bors-servo r+

@bors-servo
Copy link

📌 Commit 1217d2d has been approved by emilio

@bors-servo
Copy link

⌛ Testing commit 1217d2d with merge ed8577a...

bors-servo pushed a commit that referenced this pull request Apr 3, 2017
options: Allow configuring the generation of constructors.

Fixes #556

Sorry for the delay fixing this btw.
@emilio emilio mentioned this pull request Apr 3, 2017
@emilio
Copy link
Contributor Author

emilio commented Apr 3, 2017

Actually, added a tiny fix over #608 (need to generate the functions), so let's get that reviewed instead.

@emilio emilio closed this Apr 3, 2017
@emilio emilio deleted the gen-constructors branch April 3, 2017 11:46
bors-servo pushed a commit that referenced this pull request Apr 3, 2017
Destructor codegen

Based on #542, and on top of #606, with a bunch more tests and fixes.
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.

No constructor generated when specifying --generate types,methods,functions,vars
4 participants