[MonoDevelop] Goto class patch

[Lluis Sanchez]

> Exactly. We agree then :-) There is just one thing. I would like the
> method to return some well defined interface. This could be some
> ILocation interface (containing a Region member) which IClass,
> IProperty etc. would implement. 

Yeah, it makes sense.

> The only issue is whether this
> interface should define things to be used for the tool tip
> functionality as well?

I don't think there are special needs for the tooltip. It would check
the type of the returned object and would show all information that can
gather about it. So maybe instead of ILocation something more generic
like ISourceElement would make more sense.

> 
> /Jacob
> 
> > >
> > > > >
> > > > > We could then add some code to get the type as well. But if I am not
> > > > > wrong, the MonodocResolver does this job. So maybe we should use this
> > > > > (and possibly rename it) for the tool tip functionality?
> > > >
> > > > I'm not sure to understand. The GotoDefinitionResolver method in your
> > > > patch returns an IClass, that is, a type. If MonodocResolver does the
> > > > same, there is no need for a new one (after renaming MonodocResolver of
> > > > course).
> > >
> > > It is true that the GotoDefinitionResolver method returns an IClass,
> > > but the patch was only intended as a demonstration to see if people
> > > agreed that such functionality was needed in MD. It should not return
> > > an IClass but something more general.
> > 
> > Agreed.
> > 
> > Lluis.
> > 
> >