vieter-v
/
v
2
0
Fork 0
Code Issues Pull Requests Releases Activity

strings: fix for Builder.str() using b.len instead of b.buf.len

pull/9872/head
Delyan Angelov 2021-04-25 09:07:46 +03:00
parent 7f5c3cc1f8
commit aff21c976b
No known key found for this signature in database
GPG Key ID: 66886C0F12D595ED
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
vlib/strings/builder.v
View File

@ -74,7 +74,7 @@ pub fn (mut b Builder) go_back(n int) {
fn bytes2string(b []byte) string {
mut copy := b.clone()
copy << byte(`\0`)
copy << byte(0)
return unsafe { tos(copy.data, copy.len - 1) }
}
@ -109,7 +109,7 @@ pub fn (mut b Builder) writeln(s string) {
// }
unsafe { b.buf.push_many(s.str, s.len) }
// b.buf << []byte(s) // TODO
b.buf << `\n`
b.buf << byte(`\n`)
b.len += s.len + 1
}
@ -139,8 +139,9 @@ pub fn (b &Builder) after(n int) string {
// accumulated data that was in the string builder, before the
// .str() call.
pub fn (mut b Builder) str() string {
b.buf << `\0`
s := unsafe { (&byte(memdup(b.buf.data, b.len))).vstring_with_len(b.len) }
b.buf << byte(0)
bcopy := unsafe { &byte(memdup(b.buf.data, b.buf.len)) }
s := unsafe { bcopy.vstring_with_len(b.len) }
b.len = 0
b.buf.trim(0)
return s