From 9f5492becd2c8f189983912029893c439951ad1f Mon Sep 17 00:00:00 2001
From: Andreas Rumpf <rumpf_a@web.de>
Date: Sat, 18 Mar 2017 10:42:05 +0100
Subject: [PATCH] strutils.nim: make removeSuffix robust on empty strings

---
 lib/pure/strutils.nim | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/lib/pure/strutils.nim b/lib/pure/strutils.nim
index a87df7f529..9383675f46 100644
--- a/lib/pure/strutils.nim
+++ b/lib/pure/strutils.nim
@@ -2229,20 +2229,16 @@ proc removeSuffix*(s: var string, chars: set[char] = Newlines) {.
   ##   userInput == "Hello World"
   ##   otherInput.removeSuffix({'!', '?'})
   ##   otherInput == "Hello!?"
-
+  if s.len == 0: return
   var last = len(s) - 1
-
   if chars == Newlines:
     if s[last] == '\10':
       last -= 1
-
     if s[last] == '\13':
       last -= 1
-
   else:
     if s[last] in chars:
       last -= 1
-
   s.setLen(last + 1)
 
 proc removeSuffix*(s: var string, c: char) {.
@@ -2263,13 +2259,10 @@ proc removeSuffix*(s: var string, suffix: string) {.
   ##     answers = "yeses"
   ##   answers.removeSuffix("es")
   ##   answers == "yes"
-
   var newLen = s.len
-
   if s.endsWith(suffix):
     newLen -= len(suffix)
-
-  s.setLen(newLen)
+    s.setLen(newLen)
 
 when isMainModule:
   doAssert align("abc", 4) == " abc"