presidentbeef/brakeman — pull request #128 by presidentbeef (2012-08-14 22:46:24+00:00)
Title: Remove method_missing functionality from Sexp

The original `Sexp` class has a "feature" where `method_missing` ends up being a call to `find_node`. `find_node` also has the "feature" of being able to delete nodes when they are found.

Brakeman does not use this feature, and the use of `method_missing` hides bugs since any method can be called on a `Sexp` instance without raising an exception. It's also super-confusing.

It appears that RubyParser uses this feature for calling exactly one method, `Sexp#resbody`, so I've just made that a regular method.