]> git.lizzy.rs Git - rust.git/commit
auto merge of #15999 : Kimundi/rust/fix_folder, r=nikomatsakis
authorbors <bors@rust-lang.org>
Thu, 31 Jul 2014 16:41:36 +0000 (16:41 +0000)
committerbors <bors@rust-lang.org>
Thu, 31 Jul 2014 16:41:36 +0000 (16:41 +0000)
commit8c00357f9d60ef67479a85e8df83aa59fe690623
treec58173ece36ca6b00e70cd4a355504c13c960968
parent9826e801bed408c2497a06c535a5691e1161c85f
parentda6070dbef81c1028ce3e87fd0102b2abbbc181e
auto merge of #15999 : Kimundi/rust/fix_folder, r=nikomatsakis

Note: This PR is motivated by an attempt to write an custom syntax extension that tried to use `syntax::fold`, and that could only do so by fixing bugs in it and copying out private functions.

---

Refactored `syntax::fold`

Prior to this, the code there had a few issues:

- Default implementations inconsistenly either had the prefix `noop_` or
  not.
- Some default methods where implemented in terms of a public noop function
  for user code to call, others where implemented directly on the trait
  and did not allow users of the trait to reuse the code.
- Some of the default implementations where private, and thus not reusable
  for other implementors.
- There where some bugs where default implemntations called other default
  implementations directly, rather than to the underlying Folder, with the
  result of some ast nodes never being visted even if the user implemented that
  method. (For example, the current Folder never folded struct fields)

This commit solves this situation somewhat radically by making __all__
`fold_...` functions in the module into Folder methods, and implementing
them all in terms of public `noop_...` functions for other implementors to
call out to.

Some public functions had to be renamed to fit the new system, so this is a
breaking change.

---

Also added a few trait implementations to `ast` types
src/libsyntax/ast.rs
src/libsyntax/ext/expand.rs