Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some symbols don't resolve that should #174

Open
ericstj opened this issue Jul 29, 2024 · 1 comment
Open

Some symbols don't resolve that should #174

ericstj opened this issue Jul 29, 2024 · 1 comment

Comments

@ericstj
Copy link
Member

ericstj commented Jul 29, 2024

@stephentoub and I were looking at MemoryExtensions.Count. For example: https://source.dot.net/#System.Private.CoreLib/src/libraries/System.Private.CoreLib/src/System/MemoryExtensions.cs,4009

You cannot search for this method, but you can navigate to it from the class name. When clicking on the name of the member it does nothing: https://source.dot.net/#System.Private.CoreLib/src/libraries/System.Private.CoreLib/src/System/MemoryExtensions.cs,b6c053206fec6870

The method body does also not resolve the called method.
image

Perhaps roslyn is having difficulty with syntax in this file (eg: we need a newer roslyn in the indexer?) or maybe some other problem is making it not possible to index. I tried to find logs mentioning any problems with these types from the index build but didn't see anything. Maybe we could try opening the stage1 solution to see if there's an issue.

@PathogenDavid
Copy link

I've encountered a similar issues with other symbols. Although in my cases the search still works (but clicking on symbols doesn't) so maybe there's multiple issues at play here.

Copy+pasted from my other comment: #138 (comment)

It seems things still aren't quite right with WinForms. For example references in System.Windows.Forms to methods/types in System.Windows.Forms.Primitives do not resolve.

For example, see Application.SetHighDpiMode. Clicking on ScaleHelper.SetProcessHighDpiMode does not navigate to them as you'd expect.

You can manually navigate to it by searching for it, so the type was indexed it's just the cross reference link is missing for some reason. (The references are also broken in the other direction -- SetProcessHighDpiMode is never referenced outside of documentation according to the source browser.)

This is true of many other things in this file, such as RunMessageLoop, all the PInvoke methods, msoloop, etc. (RunMessageLoop stands out as it appears to be from the same assembly, so not sure what's up with it.)

And today I found another not in WinForms, hence this comment:

In System.String.GetHashCode you can navigate to Marvin, but not ComputeHash32. Similar to the WinForms examples above, if you manually navigate to Marvin.ComputeHash32 and inspect its references, the System.String.GetHashCode reference is missing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants