From 05bcc0fb0d3e60da4b8dd5bd7e0ea563eb4ca943 Mon Sep 17 00:00:00 2001 From: Chris Date: Wed, 28 Feb 2018 15:36:31 -0800 Subject: [PATCH] Make multi-line arrays always use trailing commas (#217) This makes ArraysWithOneElementPerLine output arrays with commas after every element. ``` A = [1,2,3] ``` Now becomes: ``` A = [ 1, 2, 3, ] ``` --- marshal.go | 2 +- marshal_test.go | 2 +- tomltree_write.go | 6 ++---- 3 files changed, 4 insertions(+), 6 deletions(-) diff --git a/marshal.go b/marshal.go index b5a2415..f584ba4 100644 --- a/marshal.go +++ b/marshal.go @@ -187,7 +187,7 @@ func (e *Encoder) QuoteMapKeys(v bool) *Encoder { // A = [ // 1, // 2, -// 3 +// 3, // ] func (e *Encoder) ArraysWithOneElementPerLine(v bool) *Encoder { e.arraysOneElementPerLine = v diff --git a/marshal_test.go b/marshal_test.go index 291a80d..00cbbf3 100644 --- a/marshal_test.go +++ b/marshal_test.go @@ -775,7 +775,7 @@ func TestMarshalArrayOnePerLine(t *testing.T) { B = [ 1, 2, - 3 + 3, ] C = [1] `) diff --git a/tomltree_write.go b/tomltree_write.go index d322a97..b5600a5 100644 --- a/tomltree_write.go +++ b/tomltree_write.go @@ -91,12 +91,10 @@ func tomlValueStringRepresentation(v interface{}, indent string, arraysOneElemen stringBuffer.WriteString("[\n") - for i, value := range values { + for _, value := range values { stringBuffer.WriteString(valueIndent) stringBuffer.WriteString(value) - if i != len(values)-1 { - stringBuffer.WriteString(`,`) - } + stringBuffer.WriteString(`,`) stringBuffer.WriteString("\n") }