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

DefaultFederatedIndexUrls doesn't include https://source.dot.net #199

Open
tannergooding opened this issue Oct 22, 2021 · 2 comments
Open

Comments

@tannergooding
Copy link

As per the title, DefaultFederatedIndexUrls doesn't include https://source.dot.net which leads to various URLs going to dead ends.

Can this be added and can it take precedence over referencesource.microsoft.com (which is largely for .NET Framework only)?

@tannergooding
Copy link
Author

An additional issue here is that this doesn't look to cover or handle TypeForward and the urls get messed up at times.

For example: https://source.terrafx.dev/#TerraFX.Interop.Windows/um/shellapi/SHELLEXECUTEINFOW.cs,22

If you click on IntPtr the URL generated is: https://source.dot.net/System.Runtime/A.html#d99bf6ad49979009 which takes you to Don't use this page directly, pass #symbolId to get redirected.

Simply inserting #System.Runtime isn't sufficient here either nor is #System.Private.Corelib. Ideally it ultimately takes us to https://source.dot.net/#System.Private.CoreLib/IntPtr.cs,d99bf6ad49979009

@KirillOsenkov
Copy link
Owner

Yes I think it's time to switch from referencesource to source.dot.net.

I've also recently added an API to look up which assembly a symbol is actually declared in:

So we could call that and if that returns the better URL, use that. This would be equivalent to following type forwarders

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