Merge pull request #1022 from rokups/fixes

tritao · web-flow · commit 1a73c126cf77 · 2017-12-10T13:52:30.000Z
Multiple fixes
diff --git a/.travis.yml b/.travis.yml
@@ -1,4 +1,5 @@
 language: cpp
+sudo: required
 
 os:
   - linux
diff --git a/appveyor.yml b/appveyor.yml
@@ -25,6 +25,7 @@ os: Visual Studio 2017
 
 platform:
   - x86
+  - x64
 
 configuration:
   - Release
diff --git a/build/Compile.sh b/build/Compile.sh
@@ -16,7 +16,7 @@ else
 	BUILD_CONF=release_x86;
 fi
 
-$PREMAKE --file=$CUR_DIR/premake5.lua gmake
+$PREMAKE --file=$CUR_DIR/premake5.lua gmake "$@"
 config=$BUILD_CONF make -C $CUR_DIR/gmake/
 
 BUILD_CONF_DIR="$(tr '[:lower:]' '[:upper:]' <<< ${BUILD_CONF:0:1})${BUILD_CONF:1}"
diff --git a/build/Helpers.lua b/build/Helpers.lua
@@ -84,8 +84,9 @@ function SetupNativeProject()
     buildoptions { msvc_buildflags }
     defines { msvc_cpp_defines }
 
-  filter { "action:gmake" }
+  filter { "system:linux" }
     buildoptions { gcc_buildflags }
+    links { "stdc++" }
 
   filter { "system:macosx", "language:C++" }
     buildoptions { gcc_buildflags, "-stdlib=libc++" }
diff --git a/src/CLI/Generator.cs b/src/CLI/Generator.cs
@@ -4,9 +4,11 @@
 using CppSharp.Passes;
 using System;
 using System.Collections.Generic;
+using System.Diagnostics;
 using System.IO;
 using System.Linq;
 using System.Text;
+using System.Text.RegularExpressions;
 using CppAbi = CppSharp.Parser.AST.CppAbi;
 
 namespace CppSharp
@@ -193,34 +195,7 @@ public void Setup(Driver driver)
 
         private void SetupLinuxOptions(ParserOptions parserOptions)
         {
-            parserOptions.MicrosoftMode = false;
-            parserOptions.NoBuiltinIncludes = true;
-
-            var headersPath = string.Empty;
-
-            // Search for the available GCC versions on the provided headers.
-            var versions = Directory.EnumerateDirectories(Path.Combine(headersPath, "usr/include/c++"));
-
-            if (versions.Count() == 0)
-                throw new Exception("No valid GCC version found on system include paths");
-
-            string gccVersionPath = versions.First();
-            string gccVersion = gccVersionPath.Substring(gccVersionPath.LastIndexOf(Path.DirectorySeparatorChar) + 1);
-
-            string[] systemIncludeDirs = {
-                Path.Combine("usr", "include", "c++", gccVersion),
-                Path.Combine("usr", "include", "x86_64-linux-gnu", "c++", gccVersion),
-                Path.Combine("usr", "include", "c++", gccVersion, "backward"),
-                Path.Combine("usr", "lib", "gcc", "x86_64-linux-gnu", gccVersion, "include"),
-                Path.Combine("usr", "lib", "gcc", "x86_64-pc-linux-gnu", gccVersion, "include"),
-                Path.Combine("usr", "include", "x86_64-linux-gnu"),
-                Path.Combine("usr", "include", "x86_64-pc-linux-gnu"),
-                Path.Combine("usr", "include")
-            };
-
-            foreach (var dir in systemIncludeDirs)
-                parserOptions.AddSystemIncludeDirs(Path.Combine(headersPath, dir));
-
+            parserOptions.SetupLinux();
             parserOptions.AddDefines("_GLIBCXX_USE_CXX11_ABI=" + (options.Cpp11ABI ? "1" : "0"));
         }
 
diff --git a/src/CppParser/ParserGen/ParserGen.cs b/src/CppParser/ParserGen/ParserGen.cs
@@ -98,32 +98,7 @@ private void SetupLinuxOptions(ParserOptions options)
 
             var headersPath = Platform.IsLinux ? string.Empty :
                 Path.Combine(GetSourceDirectory("build"), "headers", "x86_64-linux-gnu");
-
-            // Search for the available GCC versions on the provided headers.
-            var versions = Directory.EnumerateDirectories(Path.Combine(headersPath,
-                "usr/include/c++"));
-
-            if (versions.Count() == 0)
-                throw new Exception("No valid GCC version found on system include paths");
-
-            string gccVersionPath = versions.First();
-            string gccVersion = gccVersionPath.Substring(
-                gccVersionPath.LastIndexOf(Path.DirectorySeparatorChar) + 1);
-
-            string[] systemIncludeDirs = {
-                Path.Combine("usr", "include", "c++", gccVersion),
-                Path.Combine("usr", "include", "x86_64-linux-gnu", "c++", gccVersion),
-                Path.Combine("usr", "include", "c++", gccVersion, "backward"),
-                Path.Combine("usr", "lib", "gcc", "x86_64-linux-gnu", gccVersion, "include"),
-                Path.Combine("usr", "lib", "gcc", "x86_64-pc-linux-gnu", gccVersion, "include"),
-                Path.Combine("usr", "include", "x86_64-linux-gnu"),
-                Path.Combine("usr", "include", "x86_64-pc-linux-gnu"),
-                Path.Combine("usr", "include")
-            };
-
-            foreach (var dir in systemIncludeDirs)
-                options.AddSystemIncludeDirs(Path.Combine(headersPath, dir));
-
+            options.SetupLinux(headersPath);
             options.AddDefines("_GLIBCXX_USE_CXX11_ABI=" + (IsGnuCpp11Abi ? "1" : "0"));
         }
 
@@ -237,4 +212,4 @@ public static void Main(string[] args)
             }
         }
     }
-}
+}
diff --git a/src/Generator.Tests/GeneratorTest.cs b/src/Generator.Tests/GeneratorTest.cs
@@ -28,6 +28,7 @@ public virtual void Setup(Driver driver)
             options.GeneratorKind = kind;
             options.OutputDir = Path.Combine(GetOutputDirectory(), "gen", name);
             options.Quiet = true;
+            options.GenerateDebugOutput = true;
             var testModule = options.AddModule(name);
             testModule.SharedLibraryName = $"{name}.Native";
 
diff --git a/src/Generator/Generators/CSharp/CSharpSources.cs b/src/Generator/Generators/CSharp/CSharpSources.cs
@@ -203,6 +203,27 @@ public override bool VisitDeclContext(DeclarationContext context)
             return true;
         }
 
+        private IEnumerable<Class> EnumerateClasses()
+        {
+            foreach (var tu in TranslationUnits)
+            {
+                foreach (var cls in EnumerateClasses(tu))
+                    yield return cls;
+            }
+        }
+
+        private IEnumerable<Class> EnumerateClasses(DeclarationContext context)
+        {
+            foreach (var cls in context.Classes)
+                yield return cls;
+
+            foreach (var ns in context.Namespaces)
+            {
+                foreach (var cls in EnumerateClasses(ns))
+                    yield return cls;
+            }
+        }
+
         void GenerateNamespaceFunctionsAndVariables(DeclarationContext context)
         {
             var hasGlobalVariables = !(context is Class) && context.Variables.Any(
@@ -213,7 +234,12 @@ void GenerateNamespaceFunctionsAndVariables(DeclarationContext context)
 
             PushBlock(BlockKind.Functions);
             var parentName = SafeIdentifier(context.TranslationUnit.FileNameWithoutExtension);
-            WriteLine("public unsafe partial class {0}", parentName);
+
+            var keyword = "class";
+            var classes = EnumerateClasses().ToList();
+            if (classes.FindAll(cls => cls.IsValueType && cls.Name == parentName && context.QualifiedLogicalName == cls.Namespace.QualifiedLogicalName).Any())
+                keyword = "struct";
+            WriteLine($"public unsafe partial {keyword} {parentName}");
             WriteStartBraceIndent();
 
             PushBlock(BlockKind.InternalsClass);
@@ -3233,4 +3259,4 @@ internal class SymbolNotFoundException : Exception
         public SymbolNotFoundException(string msg) : base(msg)
         {}
     }
-}
+}
diff --git a/src/Generator/Generators/CodeGenerator.cs b/src/Generator/Generators/CodeGenerator.cs
@@ -61,14 +61,6 @@ protected CodeGenerator(BindingContext context, IEnumerable<TranslationUnit> uni
 
         public abstract void Process();
 
-        public override string Generate()
-        {
-            if (Options.IsCSharpGenerator && Options.CompileCode && !Options.GenerateDebugOutput)
-                return base.GenerateUnformatted();
-
-            return base.Generate();
-        }
-
         public virtual void GenerateFilePreamble(CommentKind kind, string generatorName = "CppSharp")
         {
             var lines = new List<string>
@@ -91,16 +83,19 @@ public virtual void GenerateFilePreamble(CommentKind kind, string generatorName
         public virtual void GenerateDeclarationCommon(Declaration decl)
         {
             if (decl.Comment != null)
-            {
                 GenerateComment(decl.Comment);
-                GenerateDebug(decl);
-            }
+
+            GenerateDebug(decl);
         }
 
         public virtual void GenerateDebug(Declaration decl)
         {
             if (Options.GenerateDebugOutput && !string.IsNullOrWhiteSpace(decl.DebugText))
-                WriteLine("// DEBUG: " + decl.DebugText);
+            {
+                var debugText = decl.DebugText;
+                debugText = Regex.Replace(debugText.Trim(), "\r?\n", "\n// DEBUG: ");
+                WriteLine($"// DEBUG: {debugText}");
+            }
         }
 
         #endregion
diff --git a/src/Generator/Passes/CleanInvalidDeclNamesPass.cs b/src/Generator/Passes/CleanInvalidDeclNamesPass.cs
@@ -53,7 +53,6 @@ public override bool VisitDeclaration(Declaration decl)
                 (method == null || method.Kind == CXXMethodKind.Normal))
                 decl.Name = CheckName(decl.Name);
 
-            StringHelpers.CleanupText(ref decl.DebugText);
             return true;
         }
 
diff --git a/src/Generator/Passes/DelegatesPass.cs b/src/Generator/Passes/DelegatesPass.cs
@@ -187,6 +187,15 @@ private DeclarationContext GetDeclContextForDelegates(DeclarationContext @namesp
                     ).GroupBy(d => d.Name).Where(g => g.Any(d => d.HasDeclarations)).ToList();
                 if (groups.Count == 1)
                     parent = groups.Last().Last();
+                else
+                {
+                    foreach (var g in groups)
+                    {
+                        parent = g.ToList().Find(ns => ns.Name == module.LibraryName || ns.Name == module.OutputNamespace);
+                        if (parent != null)
+                            break;
+                    }
+                }
             }
 
             if (parent == null)
diff --git a/src/Generator/Utils/BlockGenerator.cs b/src/Generator/Utils/BlockGenerator.cs
@@ -183,55 +183,6 @@ public virtual string Generate()
             return builder.ToString();
         }
 
-        public StringBuilder GenerateUnformatted()
-        {
-            if (CheckGenerate != null && !CheckGenerate())
-                return new StringBuilder(0);
-
-            if (Blocks.Count == 0)
-                return Text.StringBuilder;
-
-            var builder = new StringBuilder();
-            Block previousBlock = null;
-
-            var blockIndex = 0;
-            foreach (var childBlock in Blocks)
-            {
-                var childText = childBlock.GenerateUnformatted();
-
-                var nextBlock = (++blockIndex < Blocks.Count)
-                    ? Blocks[blockIndex]
-                    : null;
-
-                if (nextBlock != null)
-                {
-                    var nextText = nextBlock.GenerateUnformatted();
-                    if (nextText.Length == 0 &&
-                        childBlock.NewLineKind == NewLineKind.IfNotEmpty)
-                        continue;
-                }
-
-                if (childText.Length == 0)
-                    continue;
-
-                if (previousBlock != null &&
-                    previousBlock.NewLineKind == NewLineKind.BeforeNextBlock)
-                    builder.AppendLine();
-
-                builder.Append(childText);
-
-                if (childBlock.NewLineKind == NewLineKind.Always)
-                    builder.AppendLine();
-
-                previousBlock = childBlock;
-            }
-
-            if (Text.StringBuilder.Length != 0)
-                builder.Append(Text.StringBuilder);
-
-            return builder;
-        }
-
         public bool IsEmpty
         {
             get
@@ -323,11 +274,6 @@ public virtual string Generate()
             return RootBlock.Generate();
         }
 
-        public string GenerateUnformatted()
-        {
-            return RootBlock.GenerateUnformatted().ToString();
-        }
-
         #region Block helpers
 
         public void AddBlock(Block block)
diff --git a/src/Generator/Utils/Utils.cs b/src/Generator/Utils/Utils.cs
@@ -43,20 +43,6 @@ public static string CommonPrefix(this string[] ss)
             return ss[0]; // all strings identical
         }
 
-        public static void CleanupText(ref string debugText)
-        {
-            // Strip off newlines from the debug text.
-            if (string.IsNullOrWhiteSpace(debugText))
-            {
-                debugText = string.Empty;
-                return;
-            }
-
-            // TODO: Make this transformation in the output.
-            debugText = Regex.Replace(debugText, " {2,}", " ");
-            debugText = debugText.Replace("\n", "");
-        }
-
         public static string[] SplitCamelCase(string input)
         {
             var str = Regex.Replace(input, "([A-Z])", " $1", RegexOptions.Compiled);
diff --git a/src/Parser/ParserOptions.cs b/src/Parser/ParserOptions.cs
diff --git a/tests/CSharp/CSharp.cs b/tests/CSharp/CSharp.cs
diff --git a/tests/CSharp/CSharp.h b/tests/CSharp/CSharp.h

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`language: cpp`
	`2`	`+sudo: required`
`2`	`3`
`3`	`4`	`os:`
`4`	`5`	`- linux`
Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,6 @@ public override bool VisitDeclaration(Declaration decl)`
`53`	`53`	`(method == null \|\| method.Kind == CXXMethodKind.Normal))`
`54`	`54`	`decl.Name = CheckName(decl.Name);`
`55`	`55`
`56`		`- StringHelpers.CleanupText(ref decl.DebugText);`
`57`	`56`	`return true;`
`58`	`57`	`}`
`59`	`58`