Skip to content

Commit 8a3a030

Browse files
committed
Reference ourselves and fix most issues.
1 parent 4df6833 commit 8a3a030

File tree

53 files changed

+112
-399
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

53 files changed

+112
-399
lines changed

VSDiagnostics/VSDiagnostics/VSDiagnostics.Test/Tests/Utilities/ExtensionsTests.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -528,10 +528,7 @@ private static void AssertMethodIsDefinedInAncestor(string source)
528528
Assert.IsTrue(methodSymbol.IsDefinedInAncestor());
529529
}
530530

531-
private static IEnumerable<MethodDeclarationSyntax> GetMethodNodes(SyntaxTree tree)
532-
{
533-
return tree.GetRoot().DescendantNodes().OfType<MethodDeclarationSyntax>();
534-
}
531+
private static IEnumerable<MethodDeclarationSyntax> GetMethodNodes(SyntaxTree tree) => tree.GetRoot().DescendantNodes().OfType<MethodDeclarationSyntax>();
535532

536533
private static SemanticModel GetSemanticModel(SyntaxTree tree)
537534
{

VSDiagnostics/VSDiagnostics/VSDiagnostics.Test/Tests/Utilities/NamingConventionsTests.cs

Lines changed: 51 additions & 204 deletions
Large diffs are not rendered by default.

VSDiagnostics/VSDiagnostics/VSDiagnostics.Test/VSDiagnostics.Test.csproj

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,7 @@
166166
<ItemGroup>
167167
<Analyzer Include="..\..\packages\Microsoft.CodeAnalysis.Analyzers.1.0.0\analyzers\dotnet\cs\Microsoft.CodeAnalysis.Analyzers.dll" />
168168
<Analyzer Include="..\..\packages\Microsoft.CodeAnalysis.Analyzers.1.0.0\analyzers\dotnet\cs\Microsoft.CodeAnalysis.CSharp.Analyzers.dll" />
169+
<Analyzer Include="..\..\packages\VSDiagnostics.1.9.3\tools\analyzers\VSDiagnostics.dll" />
169170
</ItemGroup>
170171
<Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
171172
<!-- To modify your build process, add your task inside one of the targets below and uncomment it.

VSDiagnostics/VSDiagnostics/VSDiagnostics.Test/packages.config

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,5 @@
1212
<package id="RoslynTester" version="1.6.2" targetFramework="net45" />
1313
<package id="System.Collections.Immutable" version="1.1.36" targetFramework="net45" />
1414
<package id="System.Reflection.Metadata" version="1.0.21" targetFramework="net45" />
15+
<package id="VSDiagnostics" version="1.9.3" targetFramework="net45" />
1516
</packages>

VSDiagnostics/VSDiagnostics/VSDiagnostics/Diagnostics/Async/AsyncMethodWithoutAsyncSuffix/AsyncMethodWithoutAsyncSuffixAnalyzer.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,7 @@ internal static DiagnosticDescriptor Rule
2424

2525
public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics => ImmutableArray.Create(Rule);
2626

27-
public override void Initialize(AnalysisContext context)
28-
{
29-
context.RegisterSyntaxNodeAction(AnalyzeSyntaxNode, SyntaxKind.MethodDeclaration);
30-
}
27+
public override void Initialize(AnalysisContext context) => context.RegisterSyntaxNodeAction(AnalyzeSyntaxNode, SyntaxKind.MethodDeclaration);
3128

3229
private static void AnalyzeSyntaxNode(SyntaxNodeAnalysisContext context)
3330
{

VSDiagnostics/VSDiagnostics/VSDiagnostics/Diagnostics/Async/AsyncMethodWithoutAsyncSuffix/AsyncMethodWithoutAsyncSuffixCodeFix.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,6 @@ public override async Task RegisterCodeFixesAsync(CodeFixContext context)
3535
}
3636

3737
private async Task<Solution> AddSuffixAsync(Document document, MethodDeclarationSyntax methodDeclaration, SyntaxNode root,
38-
CancellationToken cancellationToken)
39-
{
40-
return await RenameHelper.RenameSymbolAsync(document, root, methodDeclaration.Identifier, methodDeclaration.Identifier.Text + "Async", cancellationToken);
41-
}
38+
CancellationToken cancellationToken) => await RenameHelper.RenameSymbolAsync(document, root, methodDeclaration.Identifier, methodDeclaration.Identifier.Text + "Async", cancellationToken);
4239
}
4340
}

VSDiagnostics/VSDiagnostics/VSDiagnostics/Diagnostics/Async/SyncMethodWithAsyncSuffix/SyncMethodWithAsyncSuffixAnalyzer.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,7 @@ internal static DiagnosticDescriptor Rule
2424

2525
public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics => ImmutableArray.Create(Rule);
2626

27-
public override void Initialize(AnalysisContext context)
28-
{
29-
context.RegisterSyntaxNodeAction(AnalyzeSyntaxNode, SyntaxKind.MethodDeclaration);
30-
}
27+
public override void Initialize(AnalysisContext context) => context.RegisterSyntaxNodeAction(AnalyzeSyntaxNode, SyntaxKind.MethodDeclaration);
3128

3229
private static void AnalyzeSyntaxNode(SyntaxNodeAnalysisContext context)
3330
{

VSDiagnostics/VSDiagnostics/VSDiagnostics/Diagnostics/Attributes/AttributeWithEmptyArgumentList/AttributeWithEmptyArgumentListAnalyzer.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,7 @@ internal static DiagnosticDescriptor Rule
2323

2424
public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics => ImmutableArray.Create(Rule);
2525

26-
public override void Initialize(AnalysisContext context)
27-
{
28-
context.RegisterSyntaxNodeAction(AnalyzeCSharpSymbol, SyntaxKind.Attribute);
29-
}
26+
public override void Initialize(AnalysisContext context) => context.RegisterSyntaxNodeAction(AnalyzeCSharpSymbol, SyntaxKind.Attribute);
3027

3128
private void AnalyzeCSharpSymbol(SyntaxNodeAnalysisContext context)
3229
{

VSDiagnostics/VSDiagnostics/VSDiagnostics/Diagnostics/Attributes/EnumCanHaveFlagsAttribute/EnumCanHaveFlagsAttributeAnalyzer.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,7 @@ internal static DiagnosticDescriptor Rule
2323

2424
public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics => ImmutableArray.Create(Rule);
2525

26-
public override void Initialize(AnalysisContext context)
27-
{
28-
context.RegisterSyntaxNodeAction(AnalyzeCSharpSymbol, SyntaxKind.EnumDeclaration);
29-
}
26+
public override void Initialize(AnalysisContext context) => context.RegisterSyntaxNodeAction(AnalyzeCSharpSymbol, SyntaxKind.EnumDeclaration);
3027

3128
private void AnalyzeCSharpSymbol(SyntaxNodeAnalysisContext context)
3229
{

VSDiagnostics/VSDiagnostics/VSDiagnostics/Diagnostics/Attributes/FlagsEnumValuesAreNotPowersOfTwo/FlagsEnumValuesAreNotPowersOfTwoAnalyzer.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,10 +46,7 @@ internal static DiagnosticDescriptor ValuesDontFitRule
4646
public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics
4747
=> ImmutableArray.Create(DefaultRule, ValuesDontFitRule);
4848

49-
public override void Initialize(AnalysisContext context)
50-
{
51-
context.RegisterSyntaxNodeAction(AnalyzeSymbol, SyntaxKind.EnumDeclaration);
52-
}
49+
public override void Initialize(AnalysisContext context) => context.RegisterSyntaxNodeAction(AnalyzeSymbol, SyntaxKind.EnumDeclaration);
5350

5451
private void AnalyzeSymbol(SyntaxNodeAnalysisContext context)
5552
{

0 commit comments

Comments
 (0)