Don't try indexing backward in strings if they are 0-length

ryanuber · ryanuber · commit f0fd2e8b9452 · 2014-04-12T00:29:46.000-07:00
diff --git a/columnize.go b/columnize.go
@@ -95,7 +95,7 @@ func Format(input interface{}, config *Config) (string, error) {
 	}
 
 	// Remove trailing newline without removing leading/trailing space
-	if n := len(result); result[n-1] == '\n' {
+	if n := len(result); n > 0 && result[n-1] == '\n' {
 		result = result[:n-1]
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ func Format(input interface{}, config *Config) (string, error) {`
`95`	`95`	`}`
`96`	`96`
`97`	`97`	`// Remove trailing newline without removing leading/trailing space`
`98`		`- if n := len(result); result[n-1] == '\n' {`
	`98`	`+ if n := len(result); n > 0 && result[n-1] == '\n' {`
`99`	`99`	`result = result[:n-1]`
`100`	`100`	`}`
`101`	`101`