Fixed the generated C# when a constructor has a default parameter with type an external specialisation.

Signed-off-by: Dimitar Dobrev <[email protected]>

Author: ddobrev

src/Generator/Driver.cs

@@ -224,6 +224,7 @@ public void SetupPasses(ILibrary library)
             {
                 TranslationUnitPasses.AddPass(new GenerateSymbolsPass());
                 TranslationUnitPasses.AddPass(new TrimSpecializationsPass());
+                TranslationUnitPasses.AddPass(new CheckIgnoredDeclsPass());
             }
 
             library.SetupPasses(this);

src/Generator/Passes/ConstructorToConversionOperatorPass.cs

@@ -12,10 +12,10 @@ public class ConstructorToConversionOperatorPass : TranslationUnitPass
     {
         public override bool VisitMethodDecl(Method method)
         {
-            if (AlreadyVisited(method) || !method.IsGenerated || !method.IsConstructor
-                || method.IsCopyConstructor
+            if (AlreadyVisited(method) || !method.IsGenerated || !method.IsConstructor ||
+                method.IsCopyConstructor ||
                 // conversion operators can only be public
-                || method.Access != AccessSpecifier.Public)
+                method.Access != AccessSpecifier.Public)
                 return false;
 
             var @params = method.Parameters.Where(p => p.Kind == ParameterKind.Regular).ToList();
@@ -29,7 +29,7 @@ public override bool VisitMethodDecl(Method method)
                      (p.DefaultArgument.Declaration == null ||
                       p.DefaultArgument.Declaration.Ignore)));
                 if (nonDefaultParams > 1)
-                    return false;   
+                    return false;
             }
             else
             {

tests/NamespacesDerived/NamespacesDerived.cpp

@@ -62,7 +62,8 @@ Abstract* Derived2::getAbstract()
     return 0;
 }
 
-DerivedFromExternalSpecialization::DerivedFromExternalSpecialization()
+DerivedFromExternalSpecialization::DerivedFromExternalSpecialization(int i,
+                                                                     TemplateWithIndependentFields<HasVirtualInDependency> defaultExternalSpecialization)
 {
 }
 

tests/NamespacesDerived/NamespacesDerived.h

@@ -74,7 +74,9 @@ class DLL_API HasVirtualInDependency : public HasVirtualInCore
 class DLL_API DerivedFromExternalSpecialization : public TemplateWithIndependentFields<Derived>
 {
 public:
-    DerivedFromExternalSpecialization();
+    DerivedFromExternalSpecialization(int i,
+                                      TemplateWithIndependentFields<HasVirtualInDependency> defaultExternalSpecialization =
+                                          TemplateWithIndependentFields<HasVirtualInDependency>());
     ~DerivedFromExternalSpecialization();
     TemplateWithIndependentFields<Base3> returnExternalSpecialization();
 };