Adds SEQ_DECL_SIZE 1 back under clang and a test (#23942)

jmgomez · web-flow · commit 630c304a2dff · 2024-08-12T18:10:17.000+02:00
diff --git a/lib/nimbase.h b/lib/nimbase.h
@@ -475,7 +475,9 @@ typedef char* NCSTRING;
   } name = {{length, (NI) ((NU)length | NIM_STRLIT_FLAG)}, str}
 
 /* declared size of a sequence/variable length array: */
-#if  defined(__GNUC__) || defined(__clang__) || defined(_MSC_VER)
+#if defined(__cplusplus) && defined(__clang__)
+#  define SEQ_DECL_SIZE 1
+#elif defined(__GNUC__) || defined(_MSC_VER)
 #  define SEQ_DECL_SIZE /* empty is correct! */
 #else
 #  define SEQ_DECL_SIZE 1000000
diff --git a/tests/cpp/fam.h b/tests/cpp/fam.h
@@ -0,0 +1,4 @@
+struct Test{
+  ~Test() {
+  }
+};
diff --git a/tests/cpp/tfam.nim b/tests/cpp/tfam.nim
@@ -0,0 +1,7 @@
+discard """
+  targets: "cpp"
+"""
+type 
+  Test {.importcpp, header: "fam.h".} = object
+
+let test = newSeq[Test]()

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +struct Test{
 +  ~Test() {
 +  }
 +};