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

Make IDL attribute macros a bit more concise. #7900

Closed
wants to merge 1 commit into from

Conversation

@eefriedman
Copy link
Contributor

eefriedman commented Oct 7, 2015

Instead of making the getter and setter separate macros, just combine
them into one, which is almost always what we want to do.

Review on Reviewable

@bors-servo
Copy link
Contributor

bors-servo commented Oct 7, 2015

The latest upstream changes (presumably #7898) made this pull request unmergeable. Please resolve the merge conflicts.

Instead of making the getter and setter separate macros, just combine
them into one, which is almost always what we want to do.
@eefriedman eefriedman force-pushed the eefriedman:attr-cleanup branch from 3047041 to 907a0d1 Oct 8, 2015
@eefriedman
Copy link
Contributor Author

eefriedman commented Oct 8, 2015

Rebased.

@jdm jdm removed the S-needs-rebase label Oct 8, 2015
@bors-servo
Copy link
Contributor

bors-servo commented Oct 10, 2015

The latest upstream changes (presumably #7960) made this pull request unmergeable. Please resolve the merge conflicts.

@eefriedman
Copy link
Contributor Author

eefriedman commented Nov 1, 2015

At this point, I'd have to redo this from scratch... and I'm not sure it's worthwhile. Closing.

@eefriedman eefriedman closed this Nov 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.