Made the case-renaming pass non-optional for Mono/.NET generators.

Author: ddobrev

src/CppParser/ParserGen/ParserGen.cs

@@ -182,10 +182,6 @@ public void Preprocess(Driver driver, ASTContext ctx)
 
         public void Postprocess(Driver driver, ASTContext ctx)
         {
-            new CaseRenamePass(
-                RenameTargets.Function | RenameTargets.Method | RenameTargets.Property | RenameTargets.Delegate |
-                RenameTargets.Field | RenameTargets.Variable,
-                RenameCasePattern.UpperCamelCase).VisitASTContext(driver.Context.ASTContext);
         }
 
         public static void Main(string[] args)

src/Generator/Driver.cs

@@ -338,6 +338,12 @@ public void SetupPasses(ILibrary library)
 
             TranslationUnitPasses.AddPass(new GetterSetterToPropertyPass());
             TranslationUnitPasses.AddPass(new StripUnusedSystemTypesPass());
+            if (Options.GeneratorKind == GeneratorKind.CLI ||
+                Options.GeneratorKind == GeneratorKind.CSharp)
+                TranslationUnitPasses.AddPass(new CaseRenamePass(
+                    RenameTargets.Function | RenameTargets.Method | RenameTargets.Property |
+                    RenameTargets.Delegate | RenameTargets.Field | RenameTargets.Variable,
+                    RenameCasePattern.UpperCamelCase));
         }
 
         public void ProcessCode()

src/Generator/Passes/RenamePass.cs

@@ -83,11 +83,20 @@ public bool IsRenameableDecl(Declaration decl)
             }
             if (decl is Parameter) return true;
             if (decl is Enumeration) return true;
-            if (decl is Property) return true;
+            var property = decl as Property;
+            if (property != null) return !property.IsIndexer;
             if (decl is Event) return true;
             if (decl is TypedefDecl) return true;
             if (decl is Namespace && !(decl is TranslationUnit)) return true;
             if (decl is Variable) return true;
+            var field = decl as Field;
+            if (field != null)
+            {
+                var fieldProperty = ((Class) field.Namespace).Properties.FirstOrDefault(
+                    p => p.Field == field);
+                return (fieldProperty != null &&
+                    fieldProperty.IsInRefTypeAndBackedByValueClassField());
+            }
             return false;
         }
 

tests/CLI/CLI.Tests.cs

@@ -23,6 +23,6 @@ public void TestToStringOverride()
     [Test]
     public void TestStdString()
     {
-        Assert.AreEqual("test_test", new Date(0, 0, 0).testStdString("test"));
+        Assert.AreEqual("test_test", new Date(0, 0, 0).TestStdString("test"));
     }
 }

tests/CSharp/CSharp.cs

@@ -169,9 +169,6 @@ public override void Preprocess(Driver driver, ASTContext ctx)
 
         public override void Postprocess(Driver driver, ASTContext ctx)
         {
-            new CaseRenamePass(
-                RenameTargets.Function | RenameTargets.Method | RenameTargets.Property | RenameTargets.Delegate | RenameTargets.Variable,
-                RenameCasePattern.UpperCamelCase).VisitASTContext(driver.Context.ASTContext);
         }
 
         public static void Main(string[] args)