-
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
Remove more usages of specialized collections in favor of collection exprs #72918
Changes from 1 commit
2338503
7131928
9e3ae85
539cb73
5d25d39
fef845d
2d21c07
876ebb8
a289dad
8f58021
5625f34
c70dd76
07ff580
3f5ab62
03d5a57
0c6bfa9
43e919e
e9710f5
ed1074e
77ecaac
b877438
fdb859c
dec1564
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -77,5 +77,5 @@ protected static SignatureHelpSymbolParameter Convert( | |
#pragma warning disable CA1822 // Mark members as static - see obsolete message above. | ||
protected IList<TaggedText> GetAwaitableUsage(IMethodSymbol method, SemanticModel semanticModel, int position) | ||
#pragma warning restore CA1822 // Mark members as static | ||
=> SpecializedCollections.EmptyList<TaggedText>(); | ||
=> []; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -480,9 +480,7 @@ private async Task<ImmutableArray<CodeActionOperation>> GetGenerateIntoExistingD | |
{ | ||
string includeUsingsOrImports = null; | ||
if (!areFoldersValidIdentifiers) | ||
{ | ||
folders = SpecializedCollections.EmptyList<string>(); | ||
} | ||
folders = []; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
|
||
// Now actually create the symbol that we want to add to the root namespace. The | ||
// symbol may either be a named type (if we're not generating into a namespace) or | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,9 +23,7 @@ internal sealed class SupportedPlatformData(Solution solution, List<ProjectId> i | |
public IList<SymbolDisplayPart> ToDisplayParts() | ||
{ | ||
if (InvalidProjects == null || InvalidProjects.Count == 0) | ||
{ | ||
return SpecializedCollections.EmptyList<SymbolDisplayPart>(); | ||
} | ||
return []; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
|
||
var builder = new List<SymbolDisplayPart>(); | ||
builder.AddLineBreak(); | ||
|
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.