fix: Correct edge checks for grow

Author: linkdotnet

CHANGELOG.md

@@ -6,11 +6,14 @@ All notable changes to **ValueStringBuilder** will be documented in this file. T
 
 ## [Unreleased]
 
-## [0.8.0] - 2023-08-24
-
 ### Added
 - `Trim`, `TrimStart`, `TrimEnd` are added to the string builder
 
+### Fixed
+- Sometimes the grow mechanism did not work properly leading to a panic
+
+## [0.8.0] - 2023-08-24
+
 ### Added
 
 -   The string builder can now be used to append integers, boolean & collection of strings. `sb.AppendInt(1).AppendBool(true).AppendList([]string{"a", "b", "c"})`

stringbuilder.go

@@ -45,7 +45,7 @@ func (s *StringBuilder) AppendLine(text string) *StringBuilder {
 // Appends a single character to the StringBuilder instance
 func (s *StringBuilder) AppendRune(char rune) *StringBuilder {
 	newLen := s.position + 1
-	if newLen > cap(s.data) {
+	if newLen >= cap(s.data) {
 		s.grow(newLen)
 	}
 	s.data[s.position] = char
@@ -79,7 +79,7 @@ func (s *StringBuilder) resize(words ...string) {
 		allWordLength += len(word)
 	}
 	newLen := s.position + allWordLength
-	if newLen > cap(s.data) {
+	if newLen >= cap(s.data) {
 		s.grow(newLen)
 	}
 }
@@ -274,6 +274,12 @@ func (s *StringBuilder) TrimEnd(chars ...rune) *StringBuilder {
 	return s
 }
 
+// Returns the internal array of the string builder. Be careful as this returns the internal slice.
+// Changes to that will reflect in this string builder instance.
+func (s *StringBuilder) AsRuneArray() []rune {
+	return s.data
+}
+
 func (s *StringBuilder) grow(lenToAdd int) {
 	// Grow times 2 until lenToAdd fits
 	newLen := len(s.data)