-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Switch to straight metadata reading to build our metadata indices for add-using and add-reference. #11183
Switch to straight metadata reading to build our metadata indices for add-using and add-reference. #11183
Conversation
c782334
to
a000159
Compare
Tagging @tmat and @dotnet/roslyn-ide |
@@ -1,6 +1,8 @@ | |||
// Copyright (c) Microsoft. All Rights Reserved. Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information. | |||
|
|||
using System; | |||
using System.Collections.Generic; | |||
using System.Reflection.Metadata; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needed?
@tmat I've addressed your comments. Let me know what you think about the compiler side of things. |
return info; | ||
} | ||
|
||
// We don't include internals from metadata assemblies. It's less likely that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This comment seems misplaced now. There is no mention of symbol accessibility here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
// Add all child members | ||
foreach (var symbol in symbolsWithSameName) | ||
{ | ||
lookup(symbol, symbolMap); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lookup
doesn't sound like the right name here.
@CyrusNajmabadi A couple more comments. Other than that compiler changes LGTM. |
7d792d1
to
8a8bcfa
Compare
8a8bcfa
to
056b499
Compare
👍 |
test vsi please |
return null; | ||
} | ||
|
||
// Check if we have teh 'Color Color' case. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
spelling
No description provided.