diff options
author | Paul Phillips <paulp@improving.org> | 2013-10-12 13:22:29 -0700 |
---|---|---|
committer | Paul Phillips <paulp@improving.org> | 2013-10-12 13:23:52 -0700 |
commit | 1edddfa7fd3066b2c336eb16f105283f5e9c18cd (patch) | |
tree | 88d364a3fc4bcf670dba66a94df87d78eb07fbff /test/files/neg | |
parent | 351a3e0da6bf84bcdfdd1b0fb02f16c0d9896cb1 (diff) | |
download | scala-1edddfa7fd3066b2c336eb16f105283f5e9c18cd.tar.gz scala-1edddfa7fd3066b2c336eb16f105283f5e9c18cd.tar.bz2 scala-1edddfa7fd3066b2c336eb16f105283f5e9c18cd.zip |
Eliminate redundant pickling code.
This commit drops about 700 lines of redundant traversal logic.
There had been ad hoc adjustments to the pickling scheme here and
there, probably in pursuit of tiny performance improvements.
For instance, a Block was pickled expr/stats instead of stats/expr,
a TypeDef was pickled rhs/tparams instead of tparams/rhs.
The benefits derived are invisible compared to the cost of having
several hundred lines of tree traversal code duplicated in half a
dozen or more places.
After making Traverser consistent/complete, it was a straightforward
matter to use it for pickling. It is ALSO now possible to write a
vastly cleaner tree printer than the ones presently in trunk, but I
leave this as an exercise for Dear Reviewer.
Diffstat (limited to 'test/files/neg')
0 files changed, 0 insertions, 0 deletions