Removed the option for copy ctors and prevented usage of ignored ones.

Signed-off-by: Dimitar Dobrev <dpldobrev@yahoo.com>

src/CppParser/Bindings/ParserGen.cs

@@ -53,8 +53,7 @@ namespace CppSharp
             options.Abi = Abi;
             options.LibraryName = "CppSharp.CppParser.dll";
             options.GeneratorKind = Kind;
-            options.GenerateCopyConstructors = true;
-            options.Headers.AddRange(new string[]
+            options.Headers.AddRange(new[]
             {
                 "AST.h",
                 "Sources.h",

src/Generator/AST/Utils.cs

@@ -41,9 +41,6 @@ namespace CppSharp.AST
             // Ignore copy constructor if a base class don't has or has a private copy constructor
             if (method.IsCopyConstructor)
             {
-                if (!options.GenerateCopyConstructors)
-                    return true;
-
                 var baseClass = @class;
                 while (baseClass != null && baseClass.HasBaseClass)
                 {

src/Generator/Generators/CLI/CLITypePrinter.cs

@@ -319,15 +319,25 @@ namespace CppSharp.Generators.CLI
         {
             var names = new List<string>();
 
+            string rootNamespace = null;
             if (Options.GenerateLibraryNamespace)
-                names.Add(Driver.Options.OutputNamespace);
+                names.Add(rootNamespace = Driver.Options.OutputNamespace);
 
             if (!string.IsNullOrEmpty(decl.Namespace.QualifiedName))
+            {
                 names.Add(decl.Namespace.QualifiedName);
+                if (string.IsNullOrEmpty(rootNamespace))
+                    rootNamespace = decl.Namespace.QualifiedName;
+            }
 
             names.Add(decl.Visit(this));
 
-            return string.Join("::", names);
+            var result = string.Join("::", names);
+            var translationUnit = decl.Namespace as TranslationUnit;
+            if (translationUnit != null && translationUnit.HasFunctions &&
+                rootNamespace == translationUnit.FileNameWithoutExtension)
+                return "::" + result;
+            return result;
         }
 
         public string VisitClassDecl(Class @class)

src/Generator/Generators/CSharp/CSharpTextTemplate.cs

@@ -553,7 +553,7 @@ namespace CppSharp.Generators.CSharp
                     if (@class.IsStatic || ctor.IsMoveConstructor)
                         continue;
 
-                    if (!ctor.IsGenerated && !(Options.GenerateCopyConstructors && ctor.IsCopyConstructor))
+                    if (!ctor.IsGenerated)
                         continue;
 
                     if (ctor.IsDefaultConstructor && !@class.HasNonTrivialDefaultConstructor)
@@ -1971,15 +1971,10 @@ namespace CppSharp.Generators.CSharp
                 PushBlock(CSharpBlockKind.Method);
                 WriteLine("private static {0}.Internal* __CopyValue({0}.Internal native)", className);
                 WriteStartBraceIndent();
-                if (@class.HasNonTrivialCopyConstructor && Options.GenerateCopyConstructors)
+                var copyCtorMethod = @class.Methods.FirstOrDefault(method =>
+                    method.IsCopyConstructor);
+                if (@class.HasNonTrivialCopyConstructor && copyCtorMethod != null && copyCtorMethod.IsGenerated)
                 {
-                    // Find a valid copy constructor overload.
-                    var copyCtorMethod = @class.Methods.FirstOrDefault(method =>
-                        method.IsCopyConstructor);
-
-                    if (copyCtorMethod == null)
-                        throw new NotSupportedException("Expected a valid copy constructor");
-
                     // Allocate memory for a new native object and call the ctor.
                     WriteLine("var ret = Marshal.AllocHGlobal({0});", @class.Layout.Size);
                     WriteLine("{0}.Internal.{1}(ret, new global::System.IntPtr(&native));",

src/Generator/Options.cs

@@ -96,7 +96,6 @@ namespace CppSharp
         public bool GenerateInternalImports;
         public bool GenerateClassMarshals;
         public bool GenerateInlines;
-        public bool GenerateCopyConstructors;
         public bool UseHeaderDirectories;
 
         /// <summary>

src/Generator/Passes/FindSymbolsPass.cs

@@ -4,6 +4,18 @@ namespace CppSharp.Passes
 {
     public class FindSymbolsPass : TranslationUnitPass
     {
+        public FindSymbolsPass()
+        {
+            Options.VisitClassBases = false;
+            Options.VisitFunctionParameters = false;
+            Options.VisitFunctionReturnType = false;
+            Options.VisitNamespaceEnums = false;
+            Options.VisitNamespaceTemplates = false;
+            Options.VisitNamespaceTypedefs = false;
+            Options.VisitTemplateArguments = false;
+            Options.VisitClassFields = false;
+        }
+
         public override bool VisitDeclaration(Declaration decl)
         {
             if (!base.VisitDeclaration(decl))

tests/CSharp/CSharp.cs

@@ -128,7 +128,6 @@ namespace CppSharp.Tests
         {
             driver.Options.GenerateInterfacesForMultipleInheritance = true;
             driver.Options.GeneratePropertiesAdvanced = true;
-            driver.Options.GenerateCopyConstructors = true;
             // To ensure that calls to constructors in conversion operators
             // are not ambiguous with multiple inheritance pass enabled.
             driver.Options.GenerateConversionOperators = true;

tests/Common/Common.cs

@@ -23,7 +23,6 @@ namespace CppSharp.Tests
 
         public override void SetupPasses(Driver driver)
         {
-            driver.Options.GenerateCopyConstructors = true;
             driver.Options.MarshalCharAsManagedChar = true;
             driver.Options.GenerateProperties = true;
             driver.Options.GenerateConversionOperators = true;

tests/TypeMaps/TypeMaps.cs

@@ -15,7 +15,6 @@ namespace CppSharp.Tests
 
         public override void SetupPasses(Driver driver)
         {
-            driver.Options.GenerateCopyConstructors = true;
             driver.Options.MarshalCharAsManagedChar = true;
             driver.Options.GenerateProperties = true;
             driver.Options.GenerateConversionOperators = true;

tests/VTables/VTables.cpp

@@ -38,6 +38,10 @@ BaseClassVirtual::BaseClassVirtual()
 {
 }
 
+BaseClassVirtual::BaseClassVirtual(const BaseClassVirtual& other)
+{
+}
+
 int BaseClassVirtual::virtualCallRetInt(BaseClassVirtual* base)
 {
     return base->retInt();

tests/VTables/VTables.h

@@ -23,6 +23,7 @@ class DLL_API BaseClassVirtual
 {
 public:
     BaseClassVirtual();
+    BaseClassVirtual(const BaseClassVirtual& other);
     static int virtualCallRetInt(BaseClassVirtual* base);
     virtual int retInt();
     static BaseClassVirtual getBase();