Fix style (double tabs) and rework a condition

2 files changed, 22 insertions, 23 deletions

diff --git a/gendarme/rules/Gendarme.Rules.Performance/AvoidUnusedPrivateFieldsRule.cs b/gendarme/rules/Gendarme.Rules.Performance/AvoidUnusedPrivateFieldsRule.cs
index c7d74a5c..72a548ce 100644
--- a/gendarme/rules/Gendarme.Rules.Performance/AvoidUnusedPrivateFieldsRule.cs
+++ b/gendarme/rules/Gendarme.Rules.Performance/AvoidUnusedPrivateFieldsRule.cs
@@ -114,18 +114,17 @@ namespace Gendarme.Rules.Performance {
 			// scan all methods, including constructors, to find if the field is used
 			if (fields.Count > 0) {
 				CheckFieldsUsageInType (type);
-			}
 
-			// scan nested types becuase they also have access to private fields of their parent
-			if (type.HasNestedTypes && fields.Count > 0) {
-				foreach (TypeDefinition nested in type.NestedTypes) {
-					CheckFieldsUsageInType (nested);
+				// scan nested types becuase they also have access to private fields of their parent
+				if (type.HasNestedTypes) {
+					foreach (TypeDefinition nested in type.NestedTypes)
+						CheckFieldsUsageInType (nested);
 				}
-			}
 
-			// check remaining (private) fields in the set
-			foreach (FieldDefinition field in fields) {
-				Runner.Report (field, Severity.Medium, Confidence.Normal);
+				// check remaining (private) fields in the set
+				foreach (FieldDefinition field in fields) {
+					Runner.Report (field, Severity.Medium, Confidence.Normal);
+				}
 			}
 			return Runner.CurrentRuleResult;
 		}
@@ -138,14 +137,14 @@ namespace Gendarme.Rules.Performance {
 
 				// don't check the method if it does not access any field
 				if (!OpCodeEngine.GetBitmask (method).Intersect (LoadStoreFields))
-						continue;
+					continue;
 
 				foreach (Instruction ins in method.Body.Instructions) {
-						FieldDefinition fd = ins.GetField ();
-						if (fd == null)
-								continue;
+					FieldDefinition fd = ins.GetField ();
+					if (fd == null)
+						continue;
 
-						fields.Remove (fd);
+					fields.Remove (fd);
 				}
 				
 				if (fields.Count == 0)
diff --git a/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUnusedPrivateFieldsTest.cs b/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUnusedPrivateFieldsTest.cs
index dbb9f153..98fbb3ad 100644
--- a/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUnusedPrivateFieldsTest.cs
+++ b/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUnusedPrivateFieldsTest.cs
@@ -164,23 +164,23 @@ namespace Test.Rules.Performance {
 		}
 
 		class FieldsUsedInNested {
-				private bool field;
+			private bool field;
 
-				private static string staticField;
+			private static string staticField;
 
-				class Nested {
-						public void Foo (FieldsUsedInNested parent)
-						{
-								FieldsUsedInNested.staticField = "bar";
-								parent.field = true;
-						}
+			class Nested {
+				public void Foo (FieldsUsedInNested parent)
+				{
+					FieldsUsedInNested.staticField = "bar";
+					parent.field = true;
 				}
+			}
 		}
 
 		[Test]
 		public void FieldsUsedInNestedType ()
 		{
-				AssertRuleSuccess<FieldsUsedInNested> ();
+			AssertRuleSuccess<FieldsUsedInNested> ();
 		}
 		
 		class CompilerGenerated {