#234 helloworld sample double counts

[Philip Aston]

The helloworld.py sample script double counts because it instruments grinder.logger.info, and the logback implementation of info(String) calls an overloaded version of info.

A fix using selective instrumentation would require instrumenting the logger, and picking out the info method. (We can no longer use the method handle because the Jython instrumenter doesn't support selective instrumentation).

E.g.

class InfoStringMethod(Test.InstrumentationFilter):
    def matches(self, m):
        return len(m.parameterTypes) == 1 and m.name == "info"

# Instrument the info() method with our Test.
test1.record(grinder.logger, InfoStringMethod())

This is pretty ugly for a hello world.

We should find a simpler target for the sample. E.g. System.gc().

[Philip Aston]

Hmmm. Having difficulty reproducing this, even if I revert to 3.11, or to HEAD on 23/3/13.