Skip to content

Commit 80b1beb

Browse files
committed
fix builder grow
1 parent 0363d4a commit 80b1beb

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

common.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,9 @@ func commaTables(scope scope, tables []Table) string {
9595
}
9696

9797
func commaValuesBuilder(scope scope, builder *strings.Builder, values []interface{}) error {
98-
builder.Grow(16 * (len(values) + 2)) // hopefully reduce copying
98+
if len(values) == 0 {
99+
return nil
100+
}
99101
for i, item := range values {
100102
if i > 0 {
101103
builder.WriteString(", ")

insert.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,11 +158,11 @@ func (s insertStatus) GetSQL() (string, error) {
158158
if i > 0 {
159159
valuesBuilder.WriteString(", ")
160160
}
161-
valuesBuilder.WriteString("(")
161+
valuesBuilder.WriteByte('(')
162162
if err := commaValuesBuilder(s.scope, &valuesBuilder, model.GetValues()); err != nil {
163163
return "", err
164164
}
165-
valuesBuilder.WriteString(")")
165+
valuesBuilder.WriteByte(')')
166166
}
167167
}
168168
} else {

0 commit comments

Comments
 (0)