Skip to content

C#: Remove compilation warning#1678

Merged
hvitved merged 1 commit intogithub:masterfrom
calumgrant:cs/remove-analyzer-NRE
Aug 2, 2019
Merged

C#: Remove compilation warning#1678
hvitved merged 1 commit intogithub:masterfrom
calumgrant:cs/remove-analyzer-NRE

Conversation

@calumgrant
Copy link
Contributor

Removes this warning from the compilation of the C# extractor

CSC : warning AD0001: Analyzer 'Microsoft.CodeAnalysis.CSharp.Analyzers.MetaAnalyzers.CSharpDiagnosticAnalyzerApiUsageAnalyzer' threw an exception of type 'System.NullReferenceException' with message 'Object reference not set to an instance of an object.'. 

This would sometimes to accompanied by

MSBUILD : error MSB4166: Child node "2" exited prematurely

causing the Jenkins build to fail, although this has not been observed locally.

The file Semmle.Extraction.ruleset is generated by Visual Studio.

…g a compilation warning (and possibly, instability).
@calumgrant calumgrant added the C# label Aug 2, 2019
@calumgrant calumgrant requested a review from hvitved August 2, 2019 11:16
@calumgrant calumgrant requested a review from a team as a code owner August 2, 2019 11:16
@hvitved hvitved merged commit b03cf6f into github:master Aug 2, 2019
@calumgrant calumgrant deleted the cs/remove-analyzer-NRE branch August 2, 2019 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments