CodeConnect/SourceBrowser — pull request #66 by JoshVarty (2014-12-31 11:55:25+00:00)
Title: Fixes #63 and miscellaneous bugs.

This fixes #63 as well as some undocumented bugs with parameters and local variables:

- Local variable's fully qualified names were being set to their simple name. This meant that using variables both named `data` in different functions would cause one definition to incorrectly link to the other.
- Parameters were linking to the method itself instead of the parameter inside of the method.
- Search was returning local variables and parameters.

In order to differentiate between different local variables, I append a delimeter (::) and the simple name of the variable to the function or property in which they are contained.

Consider the following:

namespace MyNamespace
{
class MyClass
{
void MyMethod() { int myVariable = 4; }
}
}

The fully qualified name of `myVariable` is now `MyNamespace.MyClass.MyMethod::myVariable`. I believe this should work for C# but I'm less familiar with naming rules in VB .Net.

I'll be using a similar approach (but a different delimiter) for #64.