Merge pull request #480 from tannergooding/main

Add support for DisableRuntimeMarshalling and CallConvMemberFunction

Author: tannergooding

sources/ClangSharp.Interop/ClangSharp.Interop.csproj

@@ -2,10 +2,31 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <NoWarn>$(NoWarn);CA1069</NoWarn>
     <TargetFrameworks>net8.0;netstandard2.0</TargetFrameworks>
   </PropertyGroup>
 
+  <PropertyGroup>
+    <!-- CA1003: Use generic event handler instances -->
+    <!-- CA1008: Enums should have zero value -->
+    <!-- CA1027: Mark enums with FlagsAttribute -->
+    <!-- CA1034: Nested types should not be visible -->
+    <!-- CA1051: Do not declare visible instance fields -->
+    <!-- CA1069: Enums should not have duplicate values -->
+    <!-- CA1305: Specify IFormatProvider -->
+    <!-- CA1508: Avoid dead conditional code -->
+    <!-- CA1707: Identifiers should not contain underscores -->
+    <!-- CA1708: Identifiers should differ by more than case -->
+    <!-- CA1710: Identifiers should have correct suffix -->
+    <!-- CA1711: Identifiers should not have incorrect suffix -->
+    <!-- CA1712: Do not prefix enum values with type name -->
+    <!-- CA1720: Identifiers should not contain type names -->
+    <!-- CA1721: Property names should not match get methods -->
+    <!-- CA1724: Type names should not match namespaces -->
+    <!-- CA1815: Override equals and operator equals on value types -->
+    <!-- CA2225: Operator overloads have named alternates -->
+    <NoWarn>$(NoWarn);CA1003;CA1008;CA1027;CA1034;CA1051;CA1069;CA1305;CA1508;CA1707;CA1708;CA1710;CA1711;CA1712;CA1720;CA1721;CA1724;CA1815;CA2225</NoWarn>
+  </PropertyGroup>
+
   <ItemGroup Condition="'$(TargetFramework)' == 'netstandard2.0'">
     <Compile Remove="clang.cs" />
     <InternalsVisibleTo Include="ClangSharp" Key="$(AssemblyOriginatorPublicKey)" />

sources/ClangSharp.Interop/Configuration.cs

@@ -1,6 +1,9 @@
 // Copyright (c) .NET Foundation and Contributors. All Rights Reserved. Licensed under the MIT License (MIT). See License.md in the repository root for more information.
 
 using System;
+using System.Runtime.InteropServices;
+
+[assembly: DefaultDllImportSearchPaths(DllImportSearchPath.SafeDirectories)]
 
 namespace ClangSharp.Interop;
 
@@ -27,4 +30,4 @@ private static bool GetAppContextData(string name, bool defaultValue)
             return defaultValue;
         }
     }
-}
+}

sources/ClangSharp.Interop/Extensions/CXClientData.cs

@@ -4,14 +4,9 @@
 
 namespace ClangSharp.Interop;
 
-public unsafe partial struct CXClientData : IEquatable<CXClientData>
+public unsafe partial struct CXClientData(IntPtr handle) : IEquatable<CXClientData>
 {
-    public CXClientData(IntPtr handle)
-    {
-        Handle = handle;
-    }
-
-    public IntPtr Handle { get; set; }
+    public IntPtr Handle { get; set; } = handle;
 
     public static explicit operator CXClientData(void* value) => new CXClientData((IntPtr)value);
 

sources/ClangSharp.Interop/Extensions/CXCompileCommands.cs

@@ -6,18 +6,13 @@
 
 namespace ClangSharp.Interop;
 
-public unsafe partial struct CXCompileCommands : IDisposable, IEquatable<CXCompileCommands>, IReadOnlyCollection<CXCompileCommand>
+public unsafe partial struct CXCompileCommands(IntPtr handle) : IDisposable, IEquatable<CXCompileCommands>, IReadOnlyCollection<CXCompileCommand>
 {
-    public CXCompileCommands(IntPtr handle)
-    {
-        Handle = handle;
-    }
-
     public readonly CXCompileCommand this[uint index] => GetCommand(index);
 
     public readonly int Count => (int)Size;
 
-    public IntPtr Handle { get; set; }
+    public IntPtr Handle { get; set; } = handle;
 
     public readonly uint Size => clang.CompileCommands_getSize(this);
 

sources/ClangSharp.Interop/Extensions/CXDiagnostic.cs

@@ -4,13 +4,8 @@
 
 namespace ClangSharp.Interop;
 
-public unsafe partial struct CXDiagnostic : IDisposable, IEquatable<CXDiagnostic>
+public unsafe partial struct CXDiagnostic(IntPtr handle) : IDisposable, IEquatable<CXDiagnostic>
 {
-    public CXDiagnostic(IntPtr handle)
-    {
-        Handle = handle;
-    }
-
     public static CXDiagnosticDisplayOptions DefaultDisplayOptions => (CXDiagnosticDisplayOptions)clang.defaultDiagnosticDisplayOptions();
 
     public readonly uint Category => clang.getDiagnosticCategory(this);
@@ -19,7 +14,7 @@ public CXDiagnostic(IntPtr handle)
 
     public readonly CXDiagnosticSet ChildDiagnostics => (CXDiagnosticSet)clang.getChildDiagnostics(this);
 
-    public IntPtr Handle { get; set; }
+    public IntPtr Handle { get; set; } = handle;
 
     public readonly CXSourceLocation Location => clang.getDiagnosticLocation(this);
 
@@ -54,7 +49,7 @@ public void Dispose()
 
     public readonly CXString Format(CXDiagnosticDisplayOptions options) => clang.formatDiagnostic(this, (uint)options);
 
-    [Obsolete("Use " + nameof(CategoryText) + " instead.")]
+    [Obsolete($"Use {nameof(CategoryText)} instead.")]
     public static CXString GetCategoryName(uint category) => clang.getDiagnosticCategoryName(category);
 
     public readonly CXString GetFixIt(uint fixIt, out CXSourceRange replacementRange)

sources/ClangSharp.Interop/Internals/MarshaledString.cs

@@ -20,7 +20,7 @@ public MarshaledString(string? input)
         }
         else
         {
-            var valueBytes = (input.Length != 0) ? Encoding.UTF8.GetBytes(input) : Array.Empty<byte>();
+            var valueBytes = (input.Length != 0) ? Encoding.UTF8.GetBytes(input) : [];
             length = valueBytes.Length;
             value = Marshal.AllocHGlobal(length + 1);
             Marshal.Copy(valueBytes, 0, value, length);

sources/ClangSharp.Interop/Shims/MemberNotNullWhenAttribute.cs

@@ -3,21 +3,9 @@
 namespace System.Diagnostics.CodeAnalysis;
 
 [AttributeUsage(AttributeTargets.Method | AttributeTargets.Property, Inherited = false, AllowMultiple = true)]
-internal sealed class MemberNotNullWhenAttribute : Attribute
+internal sealed class MemberNotNullWhenAttribute(bool returnValue, params string[] members) : Attribute
 {
-    public MemberNotNullWhenAttribute(bool returnValue, string member)
-    {
-        ReturnValue = returnValue;
-        Members = new[] { member };
-    }
+    public bool ReturnValue { get; } = returnValue;
 
-    public MemberNotNullWhenAttribute(bool returnValue, params string[] members)
-    {
-        ReturnValue = returnValue;
-        Members = members;
-    }
-
-    public bool ReturnValue { get; }
-
-    public string[] Members { get; }
+    public string[] Members { get; } = members;
 }

sources/ClangSharp.Interop/clang.cs

@@ -26,7 +26,7 @@ private static IntPtr OnDllImport(string libraryName, Assembly assembly, DllImpo
             return nativeLibrary;
         }
 
-        if (libraryName.Equals("libclang") && TryResolveClang(assembly, searchPath, out nativeLibrary))
+        if (libraryName.Equals("libclang", StringComparison.Ordinal) && TryResolveClang(assembly, searchPath, out nativeLibrary))
         {
             return nativeLibrary;
         }

sources/ClangSharp.PInvokeGenerator/Abstractions/BitfieldDesc.cs

@@ -4,18 +4,9 @@
 
 namespace ClangSharp.Abstractions;
 
-public struct BitfieldDesc
+internal struct BitfieldDesc
 {
     public Type TypeBacking { get; set; }
 
     public List<BitfieldRegion> Regions { get; set; }
 }
-
-public struct BitfieldRegion
-{
-    public string Name { get; set; }
-
-    public long Offset { get; set; }
-
-    public long Length { get; set; }
-}

sources/ClangSharp.PInvokeGenerator/Abstractions/BitfieldRegion.cs

@@ -0,0 +1,12 @@
+// Copyright © Tanner Gooding and Contributors. Licensed under the MIT License (MIT). See License.md in the repository root for more information.
+
+namespace ClangSharp.Abstractions;
+
+internal struct BitfieldRegion
+{
+    public string Name { get; set; }
+
+    public long Offset { get; set; }
+
+    public long Length { get; set; }
+}