string trimright now always removes nulls

Not Tcl compatible, but makes sense

Signed-off-by: Steve Bennett <steveb@workware.net.au>

1 files changed, 2 insertions, 3 deletions

diff --git a/jim.c b/jim.c
index 598ac3f..1b4f51d 100644
--- a/jim.c
+++ b/jim.c
@@ -2227,15 +2227,14 @@ static const char *trim_left(const char *str, const char *trimchars)
     return str + strspn(str, trimchars);
 }
 
+/* Note that trim_right() always trims null characters */
 static void trim_right(char *str, const char *trimchars)
 {
     char *p = str + strlen(str) - 1;
     char *end = str - 1;
-    int c;
 
     while (p != end) {
-        c = *p;
-        if (strchr(trimchars, c) == 0) {
+        if (*p && strchr(trimchars, *p) == NULL) {
             break;
         }
         p--;