diff options
| author | Nada Amin <namin@alum.mit.edu> | 2012-09-28 01:32:44 +0200 |
|---|---|---|
| committer | Paul Phillips <paulp@improving.org> | 2013-01-24 10:50:18 -0800 |
| commit | f2e45fccfe53ff79ae34f9ca6cae1afa0d153e53 (patch) | |
| tree | baa0ccd3754434ce5943b3474a2fda7974f9dcef /src/eclipse | |
| parent | 6fd92774fe1cbee4ec68af359fa57cbd4ad8fc01 (diff) | |
| download | scala-f2e45fccfe53ff79ae34f9ca6cae1afa0d153e53.tar.gz scala-f2e45fccfe53ff79ae34f9ca6cae1afa0d153e53.tar.bz2 scala-f2e45fccfe53ff79ae34f9ca6cae1afa0d153e53.zip | |
Fix class loader issues in instrumentation tests.
The ASM ClassWriter uses a wimpy class loader when computing common
superclasses. This could cause a ClassNotFoundException in the
transform method (at reader.accept). This exception gets swallowed,
resulting in a class that should be instrumented to silently not
be. The fix is to override getCommonSuperClass to use the correct
class loader.
Trivia: This bug was discovered while 'stress-testing' this
instrumentation scheme on the Coursera students, to check that they
implement one method in terms of another in the assignment.
Diffstat (limited to 'src/eclipse')
0 files changed, 0 insertions, 0 deletions