]> git.lizzy.rs Git - rust.git/commit
auto merge of #18291 : japaric/rust/dstify, r=aturon
authorbors <bors@rust-lang.org>
Tue, 28 Oct 2014 19:56:56 +0000 (19:56 +0000)
committerbors <bors@rust-lang.org>
Tue, 28 Oct 2014 19:56:56 +0000 (19:56 +0000)
commit98bbccf2c7ea06e22ea8654a796121b9969a1760
tree0e09986b0530ff94be0589e05f27ab827a0a526b
parent3fa2b565378d292e49f17b20cb31b9bbd108b38b
parent94ddb51c9c0a53103dc0c2a4c0260e71f62e6ef8
auto merge of #18291 : japaric/rust/dstify, r=aturon

This PR changes the signature of several methods from `foo(self, ...)` to `foo(&self, ...)`/`foo(&mut self, ...)`, but there is no breakage of the usage of these methods due to the autoref nature of `method.call()`s. This PR also removes the lifetime parameter from some traits (`Trait<'a>` -> `Trait`). These changes break any use of the extension traits for generic programming, but those traits are not meant to be used for generic programming in the first place. In the whole rust distribution there was only one misuse of a extension trait as a bound, which got corrected (the bound was unnecessary and got removed) as part of this PR.

I've kept the commits as small and self-contained as possible for reviewing sake, but I can squash them when the review is over.

See this [table] to get an idea of what's left to be done. I've already DSTified [`Show`][show] and I'm working on `Hash`, but bootstrapping those changes seem to require a more recent snapshot (#18259 does the trick)

r? @aturon
cc #16918

[show]: https://github.com/japaric/rust/commits/show
[table]: https://docs.google.com/spreadsheets/d/1MZ_iSNuzsoqeS-mtLXnj9m0hBYaH5jI8k9G_Ud8FT5g/edit?usp=sharing