Took a less intrusive approach for disableing high memory usage rules.

5 files changed, 25 insertions, 21 deletions

diff --git a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeActions/ConvertIfStatementToSwitchStatementAction.cs b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeActions/ConvertIfStatementToSwitchStatementAction.cs
index eebc384a..10c4d8d6 100644
--- a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeActions/ConvertIfStatementToSwitchStatementAction.cs
+++ b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeActions/ConvertIfStatementToSwitchStatementAction.cs
@@ -32,9 +32,9 @@ using ICSharpCode.NRefactory.PatternMatching;
 
 namespace ICSharpCode.NRefactory.CSharp.Refactoring
 {
-//	[ContextAction (
-//		"Convert 'if' to 'switch'", 
-//		Description = "Convert 'if' statement to 'switch' statement")]
+	[ContextAction (
+		"Convert 'if' to 'switch'", 
+		Description = "Convert 'if' statement to 'switch' statement")]
 	public class ConvertIfStatementToSwitchStatementAction : SpecializedCodeAction<IfElseStatement>
 	{
 		protected override CodeAction GetAction (RefactoringContext context, IfElseStatement node)
diff --git a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Custom/UnreachableCodeIssue.cs b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Custom/UnreachableCodeIssue.cs
index 3474b299..9d921b9d 100644
--- a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Custom/UnreachableCodeIssue.cs
+++ b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Custom/UnreachableCodeIssue.cs
@@ -32,10 +32,11 @@ using ICSharpCode.NRefactory.Refactoring;
 
 namespace ICSharpCode.NRefactory.CSharp.Refactoring
 {
-//	[IssueDescription("Code is unreachable",
-//		Description = "Code is unreachable.",
-//		Category = IssueCategories.RedundanciesInCode,
-//		Severity = Severity.Warning)]
+	[IssueDescription("Code is unreachable",
+		Description = "Code is unreachable.",
+		Category = IssueCategories.RedundanciesInCode,
+		IsEnabledByDefault = false,
+		Severity = Severity.Warning)]
 	public class UnreachableCodeIssue : GatherVisitorCodeIssueProvider
 	{
 		protected override IGatherVisitor CreateVisitor(BaseRefactoringContext context)
diff --git a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/CodeQuality/FunctionNeverReturnsIssue.cs b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/CodeQuality/FunctionNeverReturnsIssue.cs
index a3586638..bded542a 100644
--- a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/CodeQuality/FunctionNeverReturnsIssue.cs
+++ b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/CodeQuality/FunctionNeverReturnsIssue.cs
@@ -32,11 +32,12 @@ using ICSharpCode.NRefactory.TypeSystem;
 
 namespace ICSharpCode.NRefactory.CSharp.Refactoring
 {
-//	[IssueDescription ("Function never returns",
-//		Description = "Function does not reach its end or a 'return' statement by any of possible execution paths.",
-//		Category = IssueCategories.CodeQualityIssues,
-//		Severity = Severity.Warning,
-//		AnalysisDisableKeyword = "FunctionNeverReturns")]
+	[IssueDescription ("Function never returns",
+		Description = "Function does not reach its end or a 'return' statement by any of possible execution paths.",
+		Category = IssueCategories.CodeQualityIssues,
+		Severity = Severity.Warning,
+		IsEnabledByDefault = false,
+		AnalysisDisableKeyword = "FunctionNeverReturns")]
 	public class FunctionNeverReturnsIssue : GatherVisitorCodeIssueProvider
 	{
 		protected override IGatherVisitor CreateVisitor(BaseRefactoringContext context)
diff --git a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/Opportunities/ConvertIfStatementToSwitchStatementIssue.cs b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/Opportunities/ConvertIfStatementToSwitchStatementIssue.cs
index 1d3dffc3..eebaa383 100644
--- a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/Opportunities/ConvertIfStatementToSwitchStatementIssue.cs
+++ b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/Opportunities/ConvertIfStatementToSwitchStatementIssue.cs
@@ -29,10 +29,11 @@ using System.Linq;
 
 namespace ICSharpCode.NRefactory.CSharp.Refactoring
 {
-//	[IssueDescription("'if' statement can be re-written as 'switch' statement",
-//	                  Description="Convert 'if' to 'switch'",
-//	                  Category = IssueCategories.Opportunities,
-//	                  Severity = Severity.Hint)]
+	[IssueDescription("'if' statement can be re-written as 'switch' statement",
+	                  Description="Convert 'if' to 'switch'",
+	                  Category = IssueCategories.Opportunities,
+	                  IsEnabledByDefault = false,
+	                  Severity = Severity.Hint)]
 	public class ConvertIfStatementToSwitchStatementIssue : GatherVisitorCodeIssueProvider
 	{
 		protected override IGatherVisitor CreateVisitor(BaseRefactoringContext context)
diff --git a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/RedundanciesInCode/RedundantIfElseBlockIssue.cs b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/RedundanciesInCode/RedundantIfElseBlockIssue.cs
index 8cf0dc7a..c039a22c 100644
--- a/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/RedundanciesInCode/RedundantIfElseBlockIssue.cs
+++ b/ICSharpCode.NRefactory.CSharp.Refactoring/CodeIssues/Synced/RedundanciesInCode/RedundantIfElseBlockIssue.cs
@@ -30,11 +30,12 @@ using System.Linq;
 
 namespace ICSharpCode.NRefactory.CSharp.Refactoring
 {
-//	[IssueDescription("Redundant 'else' keyword",
-//	                  Description = "Redundant 'else' keyword.",
-//	                  Category = IssueCategories.RedundanciesInCode,
-//	                  Severity = Severity.Warning,
-//	                  AnalysisDisableKeyword = "RedundantIfElseBlock")]
+	[IssueDescription("Redundant 'else' keyword",
+	                  Description = "Redundant 'else' keyword.",
+	                  Category = IssueCategories.RedundanciesInCode,
+	                  Severity = Severity.Warning,
+	                  IsEnabledByDefault = false,
+	                  AnalysisDisableKeyword = "RedundantIfElseBlock")]
 	public class RedundantIfElseBlockIssue : GatherVisitorCodeIssueProvider
 	{
 		protected override IGatherVisitor CreateVisitor(BaseRefactoringContext context)