SagerNet/sing
1
0
Fork 0
mirror of https://github.com/SagerNet/sing.git synced 2025-04-03 20:07:38 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

Improve buffer write string

Browse source 
Both []byte(string) and copy([]byte, string) are zero-alloc for this operation, but copy([]byte, string) is faster since Go supports it as a feature.
This commit is contained in:
Hellojack 2023-01-22 16:37:56 +08:00 • committed by GitHub
parent bc788b0271
commit 439ecb1a20
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

12
common/buf/buffer.go
View file

@ -229,8 +229,16 @@ func (b *Buffer) WriteRune(s rune) (int, error) {
return b.Write([]byte{byte(s)})
}
func (b *Buffer) WriteString(s string) (int, error) {
return b.Write([]byte(s))
func (b *Buffer) WriteString(s string) (n int, err error) {
if len(s) == 0 {
return
}
if b.IsFull() {
return 0, io.ErrShortBuffer
}
n = copy(b.data[b.end:], s)
b.end += n
return
}
func (b *Buffer) WriteZero() error {

2
common/metadata/serializer.go
View file

@ -200,5 +200,5 @@ func WriteSocksString(buffer *buf.Buffer, str string) error {
if err != nil {
return err
}
return rw.WriteString(buffer, str)
return common.Error(buffer.WriteString(str))
}