From c036fa05196f90dd445cc82ea3fbba860ebcb987 Mon Sep 17 00:00:00 2001
From: rlar <rlar>
Date: Sun, 1 May 2016 15:53:22 +0200
Subject: [PATCH] nupa_substitute(), cleanup

---
 src/frontend/numparam/xpressn.c | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/src/frontend/numparam/xpressn.c b/src/frontend/numparam/xpressn.c
index c5ad37f57..3e0905203 100644
--- a/src/frontend/numparam/xpressn.c
+++ b/src/frontend/numparam/xpressn.c
@@ -1207,8 +1207,8 @@ nupa_substitute(dico_t *dico, const char *s, char *r)
     const char *kptr;
 
     while ((s < s_end) && !err) {
-        s++;
-        c = s[-1];
+
+        c = *s++;
 
         if (c == '{') {
             /* try ps expression syntax */
@@ -1217,8 +1217,8 @@ nupa_substitute(dico_t *dico, const char *s, char *r)
 
             do
             {
-                kptr++;
-                d = kptr[-1];
+                d = *kptr++;
+
                 if (d == '{')
                     nnest++;
                 else if (d == '}')
@@ -1267,11 +1267,12 @@ nupa_substitute(dico_t *dico, const char *s, char *r)
 
                 do
                 {
-                    kptr++;
-                    if (kptr > s_end)
+                    if (kptr >= s_end)
                         d = '\0';
                     else
-                        d = kptr[-1];
+                        d = *kptr;
+
+                    kptr++;
 
                     if (d == '(')
                         level++;
@@ -1295,11 +1296,12 @@ nupa_substitute(dico_t *dico, const char *s, char *r)
 
                 do
                 {
-                    kptr++;
-                    if (kptr > s_end)
+                    if (kptr >= s_end)
                         d = '\0';
                     else
-                        d = kptr[-1];
+                        d = *kptr;
+
+                    kptr++;
 
                 } while ((kptr <= s_end) && (d > ' '));