-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Allow the Self
type to be used in impls.
#522
Conversation
And an implementation to show how easy it is - less than 100 lines of code changed (including the test) and no changes outside of libsyntax. |
Nice! I've often wanted this. |
+1 simple and useful for macros |
+1, always seemed like an oversight that you couldn't do this. |
It'd certainly be convenient and I've kind of expected we'll add it sooner or later. As far as the implementation goes, I would have expected to handle it in astconv, but I guess doing it in libsyntax is ok too and avoids needing to modify |
I realised today that if we want to accommodate |
On Mon, Dec 15, 2014 at 05:27:54PM -0800, Nick Cameron wrote:
I expect this to fall out of the refactoring for AT we've been |
This RFC has been accepted, r=nikomatsakis Small, easy to implement, and widely desired. This will NOT block 1.0. |
Closes #310
See also some discussion in discuss and discussion in the Rust issue tracker.