mirrors/v
1
0
Fork 0
mirror of https://github.com/vlang/v.git synced 2025-09-16 16:02:29 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions 13

strings: fix using array_push_many inside write_string + gc, not using array_push_many_noscan (part 1)

Browse source
This commit is contained in:
Delyan Angelov 2023-12-01 18:26:27 +02:00
parent eec9c7f242
commit be51143ac5
No known key found for this signature in database
GPG key ID: 66886C0F12D595ED
2 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
vlib/builtin/array_d_gcboehm_opt.v
View file

@ -255,7 +255,7 @@ fn (mut a array) push_noscan(val voidptr) {
// push_many implements the functionality for pushing another array. // push_many implements the functionality for pushing another array.
// `val` is array.data and user facing usage is `a << [1,2,3]` // `val` is array.data and user facing usage is `a << [1,2,3]`
@[unsafe] @[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 } { if size <= 0 || val == unsafe { nil } {
return return
} }

4
vlib/strings/builder.c.v
View file

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