Check if the RHS of the bitshift is an int before casting instead of always casting to int

Author: Exanite

sources/ClangSharp.PInvokeGenerator/PInvokeGenerator.VisitStmt.cs

@@ -42,7 +42,7 @@ private void VisitBinaryOperator(BinaryOperator binaryOperator)
         outputBuilder.Write(binaryOperator.OpcodeStr);
         outputBuilder.Write(' ');
 
-        if (binaryOperator.IsShiftOp || binaryOperator.IsShiftAssignOp)
+        if ((binaryOperator.IsShiftOp || binaryOperator.IsShiftAssignOp) && binaryOperator.RHS.Type.Kind != CXType_Int)
         {
             // RHS of shift operation in C# must be an int
             outputBuilder.Write("(int)");

tests/ClangSharp.PInvokeGenerator.UnitTests/CTest.cs

@@ -284,7 +284,13 @@ public static partial class Methods
     public Task UnsignedIntBitshiftTest()
     {
         var inputContents = @"#define LEFT 1 << 1U
-#define RIGHT 1 >> 1U";
+#define RIGHT 1 >> 1U
+#define INT 1 << 1
+#define LONG 1 << 1L
+#define LONGLONG 1 << 1LL
+#define ULONG 1 << 1UL
+#define ULONGLONG 1 << 1ULL
+";
 
         var expectedOutputContents = @"namespace ClangSharp.Test
 {
@@ -295,6 +301,21 @@ public static partial class Methods
 
         [NativeTypeName(""#define RIGHT 1 >> 1U"")]
         public const int RIGHT = 1 >> (int)(1U);
+
+        [NativeTypeName(""#define INT 1 << 1"")]
+        public const int INT = 1 << 1;
+
+        [NativeTypeName(""#define LONG 1 << 1L"")]
+        public const int LONG = 1 << (int)(1);
+
+        [NativeTypeName(""#define LONGLONG 1 << 1LL"")]
+        public const int LONGLONG = 1 << (int)(1L);
+
+        [NativeTypeName(""#define ULONG 1 << 1UL"")]
+        public const int ULONG = 1 << (int)(1U);
+
+        [NativeTypeName(""#define ULONGLONG 1 << 1ULL"")]
+        public const int ULONGLONG = 1 << (int)(1UL);
     }
 }
 ";