From a26675517e6326a7e3d3391f9de79d5efeb8bb90 Mon Sep 17 00:00:00 2001
From: xuri <xuri.me@gmail.com>
Date: Fri, 8 Jan 2021 23:57:13 +0800
Subject: [PATCH] This closes #756, not set the empty string for the cell when
 SetCellValue with nil

---
 cell.go      | 2 +-
 cell_test.go | 2 +-
 drawing.go   | 2 +-
 xmlChart.go  | 8 ++++----
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/cell.go b/cell.go
index 4dd28306..3e635659 100644
--- a/cell.go
+++ b/cell.go
@@ -93,7 +93,7 @@ func (f *File) SetCellValue(sheet, axis string, value interface{}) error {
 	case bool:
 		err = f.SetCellBool(sheet, axis, v)
 	case nil:
-		err = f.SetCellStr(sheet, axis, "")
+		break
 	default:
 		err = f.SetCellStr(sheet, axis, fmt.Sprint(value))
 	}
diff --git a/cell_test.go b/cell_test.go
index 8d3f7748..2122ecac 100644
--- a/cell_test.go
+++ b/cell_test.go
@@ -331,7 +331,7 @@ func TestFormattedValue2(t *testing.T) {
 	f.Styles.CellXfs.Xf = append(f.Styles.CellXfs.Xf, xlsxXf{
 		NumFmtID: nil,
 	})
-	v = f.formattedValue(3, "43528")
+	_ = f.formattedValue(3, "43528")
 
 	// formatted value with empty number format
 	f.Styles.NumFmts = nil
diff --git a/drawing.go b/drawing.go
index de9905e7..f0eb7e95 100644
--- a/drawing.go
+++ b/drawing.go
@@ -844,7 +844,7 @@ func (f *File) drawChartSeriesVal(v formatChartSeries, formatSet *formatChart) *
 // drawChartSeriesMarker provides a function to draw the c:marker element by
 // given data index and format sets.
 func (f *File) drawChartSeriesMarker(i int, formatSet *formatChart) *cMarker {
-	defaultSymbol := map[string]*attrValString{Scatter: &attrValString{Val: stringPtr("circle")}}
+	defaultSymbol := map[string]*attrValString{Scatter: {Val: stringPtr("circle")}}
 	marker := &cMarker{
 		Symbol: defaultSymbol[formatSet.Type],
 		Size:   &attrValInt{Val: intPtr(5)},
diff --git a/xmlChart.go b/xmlChart.go
index fffdddd0..ee2ad294 100644
--- a/xmlChart.go
+++ b/xmlChart.go
@@ -138,25 +138,25 @@ type aSchemeClr struct {
 }
 
 // attrValInt directly maps the val element with integer data type as an
-// attribute。
+// attribute.
 type attrValInt struct {
 	Val *int `xml:"val,attr"`
 }
 
 // attrValFloat directly maps the val element with float64 data type as an
-// attribute。
+// attribute.
 type attrValFloat struct {
 	Val *float64 `xml:"val,attr"`
 }
 
 // attrValBool directly maps the val element with boolean data type as an
-// attribute。
+// attribute.
 type attrValBool struct {
 	Val *bool `xml:"val,attr"`
 }
 
 // attrValString directly maps the val element with string data type as an
-// attribute。
+// attribute.
 type attrValString struct {
 	Val *string `xml:"val,attr"`
 }