Skip to content

Commit b2686ff

Browse files
sharwellsharwell
committed
Merge pull request #1979 from sharwell/roslyn7446-workaround
Work around dotnet/roslyn#7446
2 parents 7e27340 + 988017f commit b2686ff

7 files changed

Lines changed: 348 additions & 0 deletions

File tree

‎StyleCop.Analyzers/StyleCop.Analyzers/AnalyzerCategory.cs‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,5 +42,10 @@ internal static class AnalyzerCategory
4242
/// Category definition for spacing rules.
4343
/// </summary>
4444
internal const string SpacingRules = "StyleCop.CSharp.SpacingRules";
45+
46+
/// <summary>
47+
/// Category definition for special purpose rules.
48+
/// </summary>
49+
internal const string SpecialRules = "StyleCop.CSharp.SpecialRules";
4550
}
4651
}

‎StyleCop.Analyzers/StyleCop.Analyzers/SpecialRules/SA0000Roslyn7446Workaround.cs‎

Lines changed: 59 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,59 @@
1+
// Copyright (c) Tunnel Vision Laboratories, LLC. All Rights Reserved.
2+
// Licensed under the Apache License, Version 2.0. See LICENSE in the project root for license information.
3+
4+
namespace StyleCop.Analyzers.SpecialRules
5+
{
6+
using System;
7+
using System.Collections.Immutable;
8+
using System.Reflection;
9+
using Microsoft.CodeAnalysis;
10+
using Microsoft.CodeAnalysis.Diagnostics;
11+
12+
[DiagnosticAnalyzer(LanguageNames.CSharp)]
13+
internal sealed class SA0000Roslyn7446Workaround : DiagnosticAnalyzer
14+
{
15+
/// <summary>
16+
/// The ID for diagnostics produced by the <see cref="SA0000Roslyn7446Workaround"/> analyzer.
17+
/// </summary>
18+
public const string DiagnosticId = "SA0000";
19+
private static readonly LocalizableString Title = new LocalizableResourceString(nameof(SpecialResources.SA0000Title), SpecialResources.ResourceManager, typeof(SpecialResources));
20+
private static readonly LocalizableString MessageFormat = new LocalizableResourceString(nameof(SpecialResources.SA0000MessageFormat), SpecialResources.ResourceManager, typeof(SpecialResources));
21+
private static readonly LocalizableString Description = new LocalizableResourceString(nameof(SpecialResources.SA0000Description), SpecialResources.ResourceManager, typeof(SpecialResources));
22+
private static readonly string HelpLink = "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA0000Roslyn7446Workaround.md";
23+
24+
private static readonly DiagnosticDescriptor Descriptor =
25+
new DiagnosticDescriptor(DiagnosticId, Title, MessageFormat, AnalyzerCategory.SpecialRules, DiagnosticSeverity.Info, AnalyzerConstants.DisabledByDefault, Description, HelpLink);
26+
27+
private static readonly Action<CompilationStartAnalysisContext> CompilationStartAction = HandleCompilationStart;
28+
29+
private static readonly bool CallGetDeclarationDiagnostics;
30+
31+
static SA0000Roslyn7446Workaround()
32+
{
33+
// dotnet/roslyn#7446 was fixed for Roslyn 1.2
34+
CallGetDeclarationDiagnostics = typeof(AdditionalText).GetTypeInfo().Assembly.GetName().Version < new Version(1, 2, 0, 0);
35+
}
36+
37+
/// <inheritdoc/>
38+
public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics { get; } =
39+
ImmutableArray.Create(Descriptor);
40+
41+
/// <inheritdoc/>
42+
public override void Initialize(AnalysisContext context)
43+
{
44+
if (!CallGetDeclarationDiagnostics)
45+
{
46+
return;
47+
}
48+
49+
context.RegisterCompilationStartAction(CompilationStartAction);
50+
}
51+
52+
#pragma warning disable RS1012 // Start action has no registered actions.
53+
private static void HandleCompilationStart(CompilationStartAnalysisContext context)
54+
#pragma warning restore RS1012 // Start action has no registered actions.
55+
{
56+
context.Compilation.GetDeclarationDiagnostics(context.CancellationToken);
57+
}
58+
}
59+
}

‎StyleCop.Analyzers/StyleCop.Analyzers/SpecialRules/SpecialResources.Designer.cs‎

Lines changed: 93 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎StyleCop.Analyzers/StyleCop.Analyzers/SpecialRules/SpecialResources.resx‎

Lines changed: 136 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,136 @@
1+
<?xml version="1.0" encoding="utf-8"?>
2+
<root>
3+
<!--
4+
Microsoft ResX Schema
5+
6+
Version 2.0
7+
8+
The primary goals of this format is to allow a simple XML format
9+
that is mostly human readable. The generation and parsing of the
10+
various data types are done through the TypeConverter classes
11+
associated with the data types.
12+
13+
Example:
14+
15+
... ado.net/XML headers & schema ...
16+
<resheader name="resmimetype">text/microsoft-resx</resheader>
17+
<resheader name="version">2.0</resheader>
18+
<resheader name="reader">System.Resources.ResXResourceReader, System.Windows.Forms, ...</resheader>
19+
<resheader name="writer">System.Resources.ResXResourceWriter, System.Windows.Forms, ...</resheader>
20+
<data name="Name1"><value>this is my long string</value><comment>this is a comment</comment></data>
21+
<data name="Color1" type="System.Drawing.Color, System.Drawing">Blue</data>
22+
<data name="Bitmap1" mimetype="application/x-microsoft.net.object.binary.base64">
23+
<value>[base64 mime encoded serialized .NET Framework object]</value>
24+
</data>
25+
<data name="Icon1" type="System.Drawing.Icon, System.Drawing" mimetype="application/x-microsoft.net.object.bytearray.base64">
26+
<value>[base64 mime encoded string representing a byte array form of the .NET Framework object]</value>
27+
<comment>This is a comment</comment>
28+
</data>
29+
30+
There are any number of "resheader" rows that contain simple
31+
name/value pairs.
32+
33+
Each data row contains a name, and value. The row also contains a
34+
type or mimetype. Type corresponds to a .NET class that support
35+
text/value conversion through the TypeConverter architecture.
36+
Classes that don't support this are serialized and stored with the
37+
mimetype set.
38+
39+
The mimetype is used for serialized objects, and tells the
40+
ResXResourceReader how to depersist the object. This is currently not
41+
extensible. For a given mimetype the value must be set accordingly:
42+
43+
Note - application/x-microsoft.net.object.binary.base64 is the format
44+
that the ResXResourceWriter will generate, however the reader can
45+
read any of the formats listed below.
46+
47+
mimetype: application/x-microsoft.net.object.binary.base64
48+
value : The object must be serialized with
49+
: System.Runtime.Serialization.Formatters.Binary.BinaryFormatter
50+
: and then encoded with base64 encoding.
51+
52+
mimetype: application/x-microsoft.net.object.soap.base64
53+
value : The object must be serialized with
54+
: System.Runtime.Serialization.Formatters.Soap.SoapFormatter
55+
: and then encoded with base64 encoding.
56+
57+
mimetype: application/x-microsoft.net.object.bytearray.base64
58+
value : The object must be serialized into a byte array
59+
: using a System.ComponentModel.TypeConverter
60+
: and then encoded with base64 encoding.
61+
-->
62+
<xsd:schema id="root" xmlns="" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:msdata="urn:schemas-microsoft-com:xml-msdata">
63+
<xsd:import namespace="http://www.w3.org/XML/1998/namespace" />
64+
<xsd:element name="root" msdata:IsDataSet="true">
65+
<xsd:complexType>
66+
<xsd:choice maxOccurs="unbounded">
67+
<xsd:element name="metadata">
68+
<xsd:complexType>
69+
<xsd:sequence>
70+
<xsd:element name="value" type="xsd:string" minOccurs="0" />
71+
</xsd:sequence>
72+
<xsd:attribute name="name" use="required" type="xsd:string" />
73+
<xsd:attribute name="type" type="xsd:string" />
74+
<xsd:attribute name="mimetype" type="xsd:string" />
75+
<xsd:attribute ref="xml:space" />
76+
</xsd:complexType>
77+
</xsd:element>
78+
<xsd:element name="assembly">
79+
<xsd:complexType>
80+
<xsd:attribute name="alias" type="xsd:string" />
81+
<xsd:attribute name="name" type="xsd:string" />
82+
</xsd:complexType>
83+
</xsd:element>
84+
<xsd:element name="data">
85+
<xsd:complexType>
86+
<xsd:sequence>
87+
<xsd:element name="value" type="xsd:string" minOccurs="0" msdata:Ordinal="1" />
88+
<xsd:element name="comment" type="xsd:string" minOccurs="0" msdata:Ordinal="2" />
89+
</xsd:sequence>
90+
<xsd:attribute name="name" type="xsd:string" use="required" msdata:Ordinal="1" />
91+
<xsd:attribute name="type" type="xsd:string" msdata:Ordinal="3" />
92+
<xsd:attribute name="mimetype" type="xsd:string" msdata:Ordinal="4" />
93+
<xsd:attribute ref="xml:space" />
94+
</xsd:complexType>
95+
</xsd:element>
96+
<xsd:element name="resheader">
97+
<xsd:complexType>
98+
<xsd:sequence>
99+
<xsd:element name="value" type="xsd:string" minOccurs="0" msdata:Ordinal="1" />
100+
</xsd:sequence>
101+
<xsd:attribute name="name" type="xsd:string" use="required" />
102+
</xsd:complexType>
103+
</xsd:element>
104+
</xsd:choice>
105+
</xsd:complexType>
106+
</xsd:element>
107+
</xsd:schema>
108+
<resheader name="resmimetype">
109+
<value>text/microsoft-resx</value>
110+
</resheader>
111+
<resheader name="version">
112+
<value>2.0</value>
113+
</resheader>
114+
<resheader name="reader">
115+
<value>System.Resources.ResXResourceReader, System.Windows.Forms, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
116+
</resheader>
117+
<resheader name="writer">
118+
<value>System.Resources.ResXResourceWriter, System.Windows.Forms, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
119+
</resheader>
120+
<data name="SA0000Description" xml:space="preserve">
121+
<value>Visual Studio 2015 Update 1 contains a bug which can cause diagnostics to occasionally not display in the Errors window. When this occurs, it is impossible to use the code fixes to address style violations reported during a build. This analyzer works around the bug (dotnet/roslyn#7446).
122+
123+
When this analyzer is enabled, all diagnostics will eventually be reported in the Error window, but the performance of the analyzers is reduced. The rule is disabled for maximum performance, but can be enabled if users notice errors appearing during a build but not while editing, and they wish to use the code fixes to correct them.
124+
125+
Note that some situations are not affected by the bug:
126+
127+
* When building a project, all relevant warnings are reported even if this rule is disabled.
128+
* The various Fix All operations work properly for the selected scope, even if only a subset of the violations are appearing in the Errors window.</value>
129+
</data>
130+
<data name="SA0000MessageFormat" xml:space="preserve">
131+
<value>n/a</value>
132+
</data>
133+
<data name="SA0000Title" xml:space="preserve">
134+
<value>Workaround incomplete diagnostics in Visual Studio 2015 Update 1</value>
135+
</data>
136+
</root>

‎StyleCop.Analyzers/StyleCop.Analyzers/StyleCop.Analyzers.csproj‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -322,6 +322,12 @@
322322
<DependentUpon>SpacingResources.resx</DependentUpon>
323323
</Compile>
324324
<Compile Include="SpacingRules\TokenSpacingProperties.cs" />
325+
<Compile Include="SpecialRules\SA0000Roslyn7446Workaround.cs" />
326+
<Compile Include="SpecialRules\SpecialResources.Designer.cs">
327+
<AutoGen>True</AutoGen>
328+
<DesignTime>True</DesignTime>
329+
<DependentUpon>SpecialResources.resx</DependentUpon>
330+
</Compile>
325331
</ItemGroup>
326332
<ItemGroup>
327333
<None Include="..\..\build\keys\StyleCopAnalyzers.dev.snk">
@@ -423,6 +429,10 @@
423429
<Generator>ResXFileCodeGenerator</Generator>
424430
<LastGenOutput>SpacingResources.Designer.cs</LastGenOutput>
425431
</EmbeddedResource>
432+
<EmbeddedResource Include="SpecialRules\SpecialResources.resx">
433+
<Generator>ResXFileCodeGenerator</Generator>
434+
<LastGenOutput>SpecialResources.Designer.cs</LastGenOutput>
435+
</EmbeddedResource>
426436
</ItemGroup>
427437
<Import Project="$(MSBuildExtensionsPath32)\Microsoft\Portable\$(TargetFrameworkVersion)\Microsoft.Portable.CSharp.targets" />
428438
<Target Name="EnsureNuGetPackageBuildImports" BeforeTargets="PrepareForBuild">

‎StyleCopAnalyzers.sln‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "documentation", "documentat
3636
documentation\Configuration.md = documentation\Configuration.md
3737
documentation\EnableConfiguration.md = documentation\EnableConfiguration.md
3838
documentation\KnownChanges.md = documentation\KnownChanges.md
39+
documentation\SA0000Roslyn7446Workaround.md = documentation\SA0000Roslyn7446Workaround.md
3940
documentation\SA1000.md = documentation\SA1000.md
4041
documentation\SA1001.md = documentation\SA1001.md
4142
documentation\SA1002.md = documentation\SA1002.md

‎documentation/SA0000Roslyn7446Workaround.md‎

Lines changed: 44 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,44 @@
1+
## SA0000Roslyn7446Workaround
2+
3+
<table>
4+
<tr>
5+
<td>TypeName</td>
6+
<td>SA0000Roslyn7446Workaround</td>
7+
</tr>
8+
<tr>
9+
<td>CheckId</td>
10+
<td>SA0000</td>
11+
</tr>
12+
<tr>
13+
<td>Category</td>
14+
<td>Special Rules</td>
15+
</tr>
16+
</table>
17+
18+
## Cause
19+
20+
Workaround incomplete diagnostics in Visual Studio 2015 Update 1.
21+
22+
## Rule description
23+
24+
Visual Studio 2015 Update 1 contains a bug which can cause diagnostics to occasionally not display in the Errors window.
25+
When this occurs, it is impossible to use the code fixes to address style violations reported during a build. This
26+
analyzer works around the bug [dotnet/roslyn#7446](https://github.com/dotnet/roslyn/issues/7446).
27+
28+
When this analyzer is enabled, all diagnostics will eventually be reported in the Error window, but the performance of
29+
the analyzers is reduced. The rule is disabled for maximum performance, but can be enabled if users notice errors
30+
appearing during a build but not while editing, and they wish to use the code fixes to correct them.
31+
32+
Note that some situations are not affected by the bug:
33+
34+
* When building a project, all relevant warnings are reported even if this rule is disabled.
35+
* The various Fix All operations work properly for the selected scope, even if only a subset of the violations are
36+
appearing in the Errors window.
37+
38+
## How to fix violations
39+
40+
This analyzer does not report any diagnostics.
41+
42+
## How to suppress violations
43+
44+
This analyzer does not report any diagnostics.

0 commit comments

Comments
 (0)