about summary refs log tree commit diff
path: root/src/rustllvm/RustWrapper.cpp
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2014-07-31 16:41:36 +0000
committerbors <bors@rust-lang.org>2014-07-31 16:41:36 +0000
commit8c00357f9d60ef67479a85e8df83aa59fe690623 (patch)
treec58173ece36ca6b00e70cd4a355504c13c960968 /src/rustllvm/RustWrapper.cpp
parent9826e801bed408c2497a06c535a5691e1161c85f (diff)
parentda6070dbef81c1028ce3e87fd0102b2abbbc181e (diff)
downloadrust-8c00357f9d60ef67479a85e8df83aa59fe690623.tar.gz
rust-8c00357f9d60ef67479a85e8df83aa59fe690623.zip
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
Diffstat (limited to 'src/rustllvm/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions