Hi Tim,

> > Something is very, very odd here. I really don't understand 
> > why we'd either want or have to go to the source file for
> > finding the defining class for some method.
> Nor me. This is clearly a bad failure of code reviewing in a crucial
> area. 

I don't think so. The code is working nicely that particular place just
seems to be a bug.

>What if the sources are missing/non-connected ?

Actually, and that's one of the weirdnesses, this will work just fine. If
there are no sources we'll use the class being passed along as an argument.
So if anything it seems like the code may be working better if the sources
aren't even there ;-)

Maybe to elaborate, I simply don't understand why we'd want to go to the
file at that point at all. Maybe Anthony can elaborate on what the reason
for this behavior is/was.

Cheers,
  - Andreas