Add new function: CopySheet

- Fix build break by remapping Interface data structure
- Update unit test
- Remove CodeQL configuration

Author: xuri

.github/workflows/codeql-analysis.yml

@@ -166,6 +166,7 @@ public void TestStyle()
             },
             f.GetRows("Sheet1")
         );
+        Assert.Null(Record.Exception(() => f.CopySheet(0, f.NewSheet("Sheet3"))));
         Assert.Null(Record.Exception(() => f.SaveAs("Book1.xlsx")));
         Assert.Empty(f.Close());
 
@@ -186,6 +187,8 @@ public void TestStyle()
         Assert.Equal(expected, err.Message);
         err = Assert.Throws<RuntimeError>(() => f.AddVBAProject(Array.Empty<byte>()));
         Assert.Equal(expected, err.Message);
+        err = Assert.Throws<RuntimeError>(() => f.CopySheet(1, 2));
+        Assert.Equal(expected, err.Message);
         err = Assert.Throws<RuntimeError>(() => f.GetCellValue("Sheet1", "A1"));
         Assert.Equal(expected, err.Message);
         err = Assert.Throws<RuntimeError>(() => f.GetRows("Sheet1"));

Excelize.Tests/UnitTest.cs

@@ -191,6 +191,9 @@ internal static extern TypesC.CellNameToCoordinatesResult CellNameToCoordinates(
         [DllImport(LibraryName, CallingConvention = CallingConvention.Cdecl)]
         internal static extern IntPtr Close(long fileIdx);
 
+        [DllImport(LibraryName, CallingConvention = CallingConvention.Cdecl)]
+        internal static extern IntPtr CopySheet(long fileIdx, int from, int to);
+
         [DllImport(LibraryName, CallingConvention = CallingConvention.Cdecl)]
         internal static extern TypesC.IntErrorResult ColumnNameToNumber(
             [MarshalAs(UnmanagedType.LPUTF8Str)] string name
@@ -366,12 +369,12 @@ public static object CsValToCInterface(object value)
                 value switch
                 {
                     int _ => new Interface { Type = 1, Integer = (int)value },
-                    string _ => new Interface { Type = 2, String = (string)value },
-                    double _ => new Interface { Type = 3, Float = (double)value },
-                    float _ => new Interface { Type = 3, Float = (float)value },
-                    long _ => new Interface { Type = 3, Float = (long)value },
-                    short _ => new Interface { Type = 3, Float = (short)value },
-                    bool _ => new Interface { Type = 4, Boolean = (bool)value },
+                    string _ => new Interface { Type = 3, String = (string)value },
+                    double _ => new Interface { Type = 4, Float = (double)value },
+                    float _ => new Interface { Type = 4, Float = (float)value },
+                    long _ => new Interface { Type = 4, Float = (long)value },
+                    short _ => new Interface { Type = 4, Float = (short)value },
+                    bool _ => new Interface { Type = 5, Boolean = (bool)value },
                     _ => new Interface { Type = 0 },
                 },
                 new TypesC.Interface()
@@ -2356,11 +2359,26 @@ public string Close()
             return Marshal.PtrToStringUTF8(Lib.Close(FileIdx));
         }
 
+        /// <summary>
+        /// Duplicate a worksheet by gave source and target worksheet index.
+        /// Note that currently doesn't support duplicate workbooks that contain
+        /// tables, charts or pictures.
+        /// </summary>
+        /// <param name="buffer">The contents buffer of the file</param>
+        /// <exception cref="RuntimeError">Return None if no error occurred,
+        /// otherwise raise a RuntimeError with the message.</exception>
+        public void CopySheet(int from, int to)
+        {
+            string err = Marshal.PtrToStringUTF8(Lib.CopySheet(FileIdx, from, to));
+            if (!string.IsNullOrEmpty(err))
+                throw new RuntimeError(err);
+        }
+
         /// <summary>
         /// Get formatted value from cell by given worksheet name and cell
         /// reference in spreadsheet. The return value is converted to the
-        /// 'string' data type. If the cell format can be applied to the value
-        /// of a cell, the applied value will be returned, otherwise the
+        /// <c>string</c> data type. If the cell format can be applied to the
+        /// value of a cell, the applied value will be returned, otherwise the
         /// original value will be returned. All cells' values will be the same
         /// in a merged range.
         /// </summary>

Excelize/Excelize.cs

@@ -19,6 +19,7 @@ public unsafe struct Interface
     {
         public int Type;
         public int Integer;
+        public int Integer32;
         public sbyte* String;
         public double Float;
 

Excelize/TypesC.cs

@@ -203,6 +203,7 @@ public struct Interface
     {
         public int Type;
         public int Integer;
+        public int Integer32;
         public string String;
         public double Float;
         public bool Boolean;