From ad50f54ce77067a61bd7d0b717f1c6341565bb4d Mon Sep 17 00:00:00 2001
From: rlar <rlar>
Date: Wed, 3 Aug 2016 21:25:37 +0200
Subject: [PATCH] variables, drop 'found', use identity found == (*p != NULL)

---
 src/frontend/variable.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/frontend/variable.c b/src/frontend/variable.c
index 5b60725f9..9ebc84f7c 100644
--- a/src/frontend/variable.c
+++ b/src/frontend/variable.c
@@ -405,7 +405,6 @@ cp_remvar(char *varname)
 {
     struct variable *v, **p;
     struct variable *uv1;
-    bool found = TRUE;
     int i;
 
     uv1 = cp_usrvars();
@@ -427,7 +426,6 @@ cp_remvar(char *varname)
     if (!v) {
         /* Gotta make up a var struct for cp_usrset()... */
         v = var_alloc_num(copy(varname), 0, NULL);
-        found = FALSE;
     }
 
     /* Note that 'unset history' doesn't do anything here... Causes
@@ -453,7 +451,7 @@ cp_remvar(char *varname)
 
     case US_OK:
         /* Normal case. */
-        if (found) {
+        if (*p) {
             *p = v->va_next;
         }
         break;
@@ -461,7 +459,7 @@ cp_remvar(char *varname)
     case US_DONTRECORD:
         /* 'curplot' 'curplotname' 'curplottitle' 'curplotdate' */
         /* Do nothing... */
-        if (found)
+        if (*p)
             fprintf(cp_err, "cp_remvar: Internal Error: var %d\n", *varname);
         break;
 
@@ -469,7 +467,7 @@ cp_remvar(char *varname)
         /* 'plots' and any var in plot_cur->pl_env */
         /* Badness... */
         fprintf(cp_err, "Error: %s is read-only.\n", v->va_name);
-        if (found)
+        if (*p)
             fprintf(cp_err, "cp_remvar: Internal Error: var %d\n", *varname);
         break;