Merge pull request #3 from beef331/master

Fixed dollar operator and tests

stack_strings.nim

@@ -177,8 +177,7 @@ func `$`*(this: StackString): string {.inline.} =
     when stackStringsPreventAllocation:
         {.fatal: "The `$` proc can allocate memory at runtime, see `stackStringsPreventAllocation`".}
 
-    {.suppress: "XDeclaredButNotUsed".}
-    const errMsg = "Conversion of StackString to string with `$` proc. If this was intentional, use `toString` instead."
+    const errMsg {.used.} = "Conversion of StackString to string with `$` proc. If this was intentional, use `toString` instead."
     when fatalOnStackStringDollar:
         {.fatal: errMsg.}
     when warnOnStackStringDollar:
@@ -857,5 +856,5 @@ proc toHeapCstring*(this: StackString): cstring {.inline.} =
     # We don't need a zeroed block of memory because we'll be overwriting it manually
     result = cast[cstring](createU(char, len + 1))
 
-    moveMem(addr result[0], addr this.data[0], len)
+    moveMem(result, addr this.data[0], len)
     result[len] = '\x00'

tests/t_basic.nim

@@ -17,12 +17,12 @@ test "Random tests":
     # The string will be truncated, and the truncated data will be overwritten with zeros
     str1.unsafeSetLen(5)
     check str1 == "Hello"
-    check str1.data == ['H', 'e', 'l', 'l', 'o', '\x00', '\x00', '\x00', '\x00', '\x00', '\x00']
+    check str1.data == ['H', 'e', 'l', 'l', 'o', '\x00', '\x00', '\x00', '\x00', '\x00', '\x00', '\x00']
 
     # If we're sure it's safe to skip overwriting the truncated data with zeros, we can disable it
     str2.unsafeSetLen(2, writeZerosOnTruncate = false)
     check str2 == "Hi"
-    check str2.data == ['H', 'i', ' ', 'w', 'o', 'r', 'l', 'd']
+    check str2.data == ['H', 'i', ' ', 'w', 'o', 'r', 'l', 'd', '\0']
 
     # It works with BackwardsIndex, too
     str3.unsafeSetLen(^1)
@@ -35,3 +35,5 @@ test "Random tests":
 
     str1.unsafeSet(5, 'a')
     check str1.unsafeGet(5) == 'a'
+    discard $str1
+