strings: fix using array_push_many inside write_string + gc, not usin…

…g array_push_many_noscan (part 1)
vlang · Dec 1, 2023 · be51143 · be51143
1 parent eec9c7f
commit be51143
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/vlib/builtin/array_d_gcboehm_opt.v b/vlib/builtin/array_d_gcboehm_opt.v
@@ -255,7 +255,7 @@ fn (mut a array) push_noscan(val voidptr) {
 // push_many implements the functionality for pushing another array.
 // `val` is array.data and user facing usage is `a << [1,2,3]`
 @[unsafe]
-fn (mut a3 array) push_many_noscan(val voidptr, size int) {
+pub fn (mut a3 array) push_many_noscan(val voidptr, size int) {
 	if size <= 0 || val == unsafe { nil } {
 		return
 	}

diff --git a/vlib/strings/builder.c.v b/vlib/strings/builder.c.v
@@ -134,7 +134,11 @@ pub fn (mut b Builder) write_string(s string) {
 	if s.len == 0 {
 		return
 	}
-	unsafe { b.push_many(s.str, s.len) }
+	$if gcboehm ? {
+		unsafe { b.push_many_noscan(s.str, s.len) }
+	} $else {
+		unsafe { b.push_many(s.str, s.len) }
+	}
 	// for c in s {
 	// b.buf << c
 	// }