Skip to content

Commit

Permalink
Merge pull request #29 from qmuntal/reduce-alloc
Browse files Browse the repository at this point in the history
Avoid using  direct binary.Write when encoding a document
  • Loading branch information
qmuntal committed Oct 31, 2020
2 parents 3e49166 + 5b79d3c commit 8cd922e
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions encode.go
Original file line number Diff line number Diff line change
Expand Up @@ -131,11 +131,12 @@ func (e *Encoder) encodeBinary(doc *Document) error {
if err != nil {
return err
}
binary.Write(e.w, binary.LittleEndian, jsonText)
binary.Write(e.w, binary.LittleEndian, headerPadding)
e.w.Write(jsonText)
e.w.Write(headerPadding)
binary.Write(e.w, binary.LittleEndian, &binHeader)
if binBuffer != nil {
binary.Write(e.w, binary.LittleEndian, binBuffer.Data)
e.w.Write(binBuffer.Data)
}
return binary.Write(e.w, binary.LittleEndian, binPadding)
_, err = e.w.Write(binPadding)
return err
}

0 comments on commit 8cd922e

Please sign in to comment.