newrelic/rpm — pull request #196 by fimmtiu (2014-11-05 23:00:57+00:00)
Title: Renamed TransactionNamer.name class method.

The TransactionNamer class is rather poorly-behaved; it overrides the built-in `name` method with something completely different, thus preventing anyone from asking the class object what it's called. Observe:

```
1.9.3-p484 :001 > String.name
=> "String"
1.9.3-p484 :002 > NewRelic.name
=> "NewRelic"
1.9.3-p484 :003 > NewRelic::Agent::Instrumentation::ControllerInstrumentation::TransactionNamer.name
ArgumentError: wrong number of arguments (0 for 3)
```

Not a hypothetical problem; it bit me while I was instrumenting classes to find a memory leak. The fix is simple -- just rename the method.