Skip to content
This repository was archived by the owner on Dec 19, 2018. It is now read-only.

Commit 2d2c2cd

Browse files
author
N. Taylor Mullen
committed
Add warning disable for unused variable warnings for TagHelper fields.
- Added a 0414 warning disable/restore around the __tagHelperStringValueBuffer since it's the only TagHelper utility field that "may" never be used. - Regenerated baselines for TagHelper test files. #260
1 parent ff3ddfc commit 2d2c2cd

File tree

8 files changed

+18
-0
lines changed

8 files changed

+18
-0
lines changed

src/Microsoft.AspNet.Razor/Generator/Compiler/CodeBuilder/CSharp/Visitors/CSharpTagHelperFieldDeclarationVisitor.cs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,13 @@ protected override void Visit(TagHelperChunk chunk)
3434
// Runtime fields aren't useful during design time.
3535
if (!Context.Host.DesignTimeMode)
3636
{
37+
// Need to disable the warning "X is assigned to but never used." for the value buffer since
38+
// whether it's used depends on how a TagHelper is used.
39+
Writer.WritePragma("warning disable 0414");
3740
WritePrivateField(typeof(TextWriter).FullName,
3841
CSharpTagHelperCodeRenderer.StringValueBufferVariableName,
3942
value: null);
43+
Writer.WritePragma("warning restore 0414");
4044

4145
WritePrivateField(_tagHelperContext.ExecutionContextTypeName,
4246
CSharpTagHelperCodeRenderer.ExecutionContextVariableName,

test/Microsoft.AspNet.Razor.Test/TestFiles/CodeGenerator/CS/Output/BasicTagHelpers.CustomAttributeCodeGenerator.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ namespace TestOutput
77
public class BasicTagHelpers
88
{
99
#line hidden
10+
#pragma warning disable 0414
1011
private System.IO.TextWriter __tagHelperStringValueBuffer = null;
12+
#pragma warning restore 0414
1113
private TagHelperExecutionContext __tagHelperExecutionContext = null;
1214
private TagHelperRunner __tagHelperRunner = new TagHelperRunner();
1315
private TagHelperScopeManager __tagHelperScopeManager = new TagHelperScopeManager();

test/Microsoft.AspNet.Razor.Test/TestFiles/CodeGenerator/CS/Output/BasicTagHelpers.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@ namespace TestOutput
88
public class BasicTagHelpers
99
{
1010
#line hidden
11+
#pragma warning disable 0414
1112
private System.IO.TextWriter __tagHelperStringValueBuffer = null;
13+
#pragma warning restore 0414
1214
private TagHelperExecutionContext __tagHelperExecutionContext = null;
1315
private TagHelperRunner __tagHelperRunner = new TagHelperRunner();
1416
private TagHelperScopeManager __tagHelperScopeManager = new TagHelperScopeManager();

test/Microsoft.AspNet.Razor.Test/TestFiles/CodeGenerator/CS/Output/ComplexTagHelpers.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@ namespace TestOutput
88
public class ComplexTagHelpers
99
{
1010
#line hidden
11+
#pragma warning disable 0414
1112
private System.IO.TextWriter __tagHelperStringValueBuffer = null;
13+
#pragma warning restore 0414
1214
private TagHelperExecutionContext __tagHelperExecutionContext = null;
1315
private TagHelperRunner __tagHelperRunner = new TagHelperRunner();
1416
private TagHelperScopeManager __tagHelperScopeManager = new TagHelperScopeManager();

test/Microsoft.AspNet.Razor.Test/TestFiles/CodeGenerator/CS/Output/ContentBehaviorTagHelpers.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@ namespace TestOutput
88
public class ContentBehaviorTagHelpers
99
{
1010
#line hidden
11+
#pragma warning disable 0414
1112
private System.IO.TextWriter __tagHelperStringValueBuffer = null;
13+
#pragma warning restore 0414
1214
private TagHelperExecutionContext __tagHelperExecutionContext = null;
1315
private TagHelperRunner __tagHelperRunner = new TagHelperRunner();
1416
private TagHelperScopeManager __tagHelperScopeManager = new TagHelperScopeManager();

test/Microsoft.AspNet.Razor.Test/TestFiles/CodeGenerator/CS/Output/SingleTagHelper.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@ namespace TestOutput
88
public class SingleTagHelper
99
{
1010
#line hidden
11+
#pragma warning disable 0414
1112
private System.IO.TextWriter __tagHelperStringValueBuffer = null;
13+
#pragma warning restore 0414
1214
private TagHelperExecutionContext __tagHelperExecutionContext = null;
1315
private TagHelperRunner __tagHelperRunner = new TagHelperRunner();
1416
private TagHelperScopeManager __tagHelperScopeManager = new TagHelperScopeManager();

test/Microsoft.AspNet.Razor.Test/TestFiles/CodeGenerator/CS/Output/TagHelpersInHelper.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,9 @@ public static Template
8989
#line hidden
9090

9191
#line hidden
92+
#pragma warning disable 0414
9293
private System.IO.TextWriter __tagHelperStringValueBuffer = null;
94+
#pragma warning restore 0414
9395
private TagHelperExecutionContext __tagHelperExecutionContext = null;
9496
private TagHelperRunner __tagHelperRunner = new TagHelperRunner();
9597
private TagHelperScopeManager __tagHelperScopeManager = new TagHelperScopeManager();

test/Microsoft.AspNet.Razor.Test/TestFiles/CodeGenerator/CS/Output/TagHelpersInSection.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ namespace TestOutput
77
public class TagHelpersInSection
88
{
99
#line hidden
10+
#pragma warning disable 0414
1011
private System.IO.TextWriter __tagHelperStringValueBuffer = null;
12+
#pragma warning restore 0414
1113
private TagHelperExecutionContext __tagHelperExecutionContext = null;
1214
private TagHelperRunner __tagHelperRunner = new TagHelperRunner();
1315
private TagHelperScopeManager __tagHelperScopeManager = new TagHelperScopeManager();

0 commit comments

Comments
 (0)