Remove Skip()

Author: g41797

CHANGELOG.md

@@ -9,7 +9,6 @@ All notable changes to **ValueStringBuilder** will be documented in this file. T
 ### Added
 
 -   `SetRuneAt` is added to the string builder
--   `Skip` is added to the string builder
 
 ## [0.10.0] - 2023-08-31
 

stringbuilder.go

@@ -349,20 +349,3 @@ func (s *StringBuilder) SetRuneAt(index int, val rune) error {
 
 	return nil
 }
-
-// Change current position
-func (s *StringBuilder) Skip(forward int) error {
-	if forward <= 0 {
-		return fmt.Errorf("forward should always be greater than zero")
-	}
-
-	newPos := s.position + forward
-
-	if newPos >= len(s.data) {
-		return fmt.Errorf("cannot skip after end of string builder")
-	}
-
-	s.position = newPos
-
-	return nil
-}

stringbuilder_test.go

@@ -511,23 +511,3 @@ func TestSetRuneAt(t *testing.T) {
 		t.Errorf("Actual %q, Expected: %q", result, 'e')
 	}
 }
-
-func TestSkip(t *testing.T) {
-	sb := NewStringBuilderFromString("Hello")
-
-	err := sb.Skip(1)
-	if err == nil {
-		t.Errorf("Should throw error but did not")
-	}
-
-	sb.Clear()
-
-	err = sb.Skip(1)
-	if err != nil {
-		t.Errorf("Should not throw error")
-	}
-
-	if sb.position != 1 {
-		t.Errorf("Actual position %d, Expected: 1", sb.position)
-	}
-}