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

Support field attributes on vertex and constant structures #1373

Merged
merged 2 commits into from
Sep 27, 2017

Conversation

mzmonsour
Copy link
Contributor

Adds support for attributes on struct fields, such as doc comments. This doesn't work on stable because of a macro parser bug. Wait until rust-lang/rust#42913 is in a stable release (1.20 probably) before merging this.

@kvark
Copy link
Member

kvark commented Sep 26, 2017

@mzmonsour is this ready to ship?
Note that you need to switch appveyor config to use 1.20.
Also, this PR should probably be aimed at pre-ll and/or v0.16 as opposed to master

@mzmonsour mzmonsour changed the base branch from master to pre-ll September 27, 2017 00:19
@mzmonsour
Copy link
Contributor Author

The PR should be ready to merge. I tried adjusting the appveyor config, although I may have messed something up in the process of targeting pre-ll, since appveyor is complaining about the PR. Let me know if there's something else I should fix.

@kvark
Copy link
Member

kvark commented Sep 27, 2017

bors r+

bors bot added a commit that referenced this pull request Sep 27, 2017
1373: Support field attributes on vertex and constant structures r=kvark a=mzmonsour

Adds support for attributes on struct fields, such as doc comments. This doesn't work on stable because of a macro parser bug. Wait until rust-lang/rust#42913 is in a stable release (1.20 probably) before merging this.
@bors
Copy link
Contributor

bors bot commented Sep 27, 2017

Build failed

@kvark kvark merged commit 8e188b7 into gfx-rs:pre-ll Sep 27, 2017
alexheretic pushed a commit to alexheretic/gfx that referenced this pull request Oct 2, 2017
* Apply field attributes for gfx_defines! structures
* Appveyor should target Rust 1.20
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.

2 participants