Fixed bug #697722 Assert error / global

[splint.git] / src / functionClause.c

diff --git a/src/functionClause.c b/src/functionClause.c
index 4f4e1c92127f676a8649b49f2e75f17c99ecc9f2..f63810ce9206a18430392b43e7ddab151820b658 100644 (file)
--- a/src/functionClause.c
+++ b/src/functionClause.c
@@ -1,6 +1,6 @@
 /*
-** LCLint - annotation-assisted static program checker
-** Copyright (C) 1994-2001 University of Virginia,
+** Splint - annotation-assisted static program checker
+** Copyright (C) 1994-2003 University of Virginia,
 **         Massachusetts Institute of Technology
 **
 ** This program is free software; you can redistribute it and/or modify it
@@ -17,15 +17,15 @@
 ** the Free Software Foundation, Inc., 59 Temple Place - Suite 330, Boston,
 ** MA 02111-1307, USA.
 **
-** For information on lclint: lclint-request@cs.virginia.edu
-** To report a bug: lclint-bug@cs.virginia.edu
-** For more information: http://lclint.cs.virginia.edu
+** For information on splint: info@splint.org
+** To report a bug: splint-bug@splint.org
+** For more information: http://www.splint.org
 */
 /*
 ** functionClause.c
 */
 
-# include "lclintMacros.nf"
+# include "splintMacros.nf"
 # include "basic.h"
 
 static /*@only@*/ /*@notnull@*/ /*@special@*/ functionClause  /*@i32 need special? @*/
@@ -52,23 +52,32 @@ extern functionClause functionClause_createModifies (modifiesClause node) /*@*/
 }
 
 extern functionClause functionClause_createState (stateClause node) /*@*/ 
-{ 
-  functionClause res = functionClause_alloc (FCK_STATE);
-  res->val.state = node;
-  return res;
+{
+  if (stateClause_hasEmptyReferences (node) &&
+      (!stateClause_isMetaState (node) ) )
+    {
+      DPRINTF((message("functionClause_createState:: Returning functionClause_undefined" ) ));
+      return functionClause_undefined;
+    }
+  else
+    {
+      functionClause res = functionClause_alloc (FCK_STATE);
+      res->val.state = node;
+      return res;
+    }
 }
 
-extern functionClause functionClause_createEnsures (constraintList node) /*@*/ 
+extern functionClause functionClause_createEnsures (functionConstraint node) /*@*/ 
 { 
   functionClause res = functionClause_alloc (FCK_ENSURES);
-  res->val.ensures = node;
+  res->val.constraint = node;
   return res;
 }
 
-extern functionClause functionClause_createRequires (constraintList node) /*@*/ 
+extern functionClause functionClause_createRequires (functionConstraint node) /*@*/ 
 { 
   functionClause res = functionClause_alloc (FCK_REQUIRES);
-  res->val.requires = node;
+  res->val.constraint = node;
   return res;
 }
 
@@ -97,14 +106,14 @@ extern functionClause functionClause_createWarn (warnClause node) /*@*/
     case FCK_STATE:
       return stateClause_unparse (p->val.state);
     case FCK_ENSURES:
-      return message ("ensures %q", constraintList_unparse (p->val.ensures));
+      return message ("ensures %q", functionConstraint_unparse (p->val.constraint));
     case FCK_REQUIRES:
-      return message ("requires %q", constraintList_unparse (p->val.requires));
+      return message ("requires %q", functionConstraint_unparse (p->val.constraint));
     case FCK_DEAD:
-      BADBRANCH;
+      return cstring_makeLiteral ("<dead clause>");
     }
 
-  BADBRANCH;
+  BADBRANCHRET (cstring_undefined);
 }
 
 extern bool functionClause_matchKind (functionClause p, functionClauseKind kind) /*@*/
@@ -139,42 +148,42 @@ extern stateClause functionClause_takeState (functionClause fc)
   return res;
 }
 
-extern constraintList functionClause_getEnsures (functionClause node)
+extern functionConstraint functionClause_getEnsures (functionClause node)
 {
   llassert (functionClause_isDefined (node));
   llassert (node->kind == FCK_ENSURES);
 
-  return node->val.ensures;
+  return node->val.constraint;
 }
 
-extern constraintList functionClause_takeEnsures (functionClause fc)
+extern functionConstraint functionClause_takeEnsures (functionClause fc)
 {
-  constraintList res;
+  functionConstraint res;
   llassert (functionClause_isDefined (fc));
   llassert (fc->kind == FCK_ENSURES);
 
-  res = fc->val.ensures;
-  fc->val.ensures = NULL;
+  res = fc->val.constraint;
+  fc->val.constraint = NULL;
   fc->kind = FCK_DEAD;
   return res;
 }
 
-extern constraintList functionClause_getRequires (functionClause node)
+extern functionConstraint functionClause_getRequires (functionClause node)
 {
   llassert (functionClause_isDefined (node));
   llassert (node->kind == FCK_REQUIRES);
 
-  return node->val.requires;
+  return node->val.constraint;
 }
 
-extern constraintList functionClause_takeRequires (functionClause fc)
+extern functionConstraint functionClause_takeRequires (functionClause fc)
 {
-  constraintList res;
+  functionConstraint res;
   llassert (functionClause_isDefined (fc));
   llassert (fc->kind == FCK_REQUIRES);
 
-  res = fc->val.requires;
-  fc->val.requires = NULL;
+  res = fc->val.constraint;
+  fc->val.constraint = NULL;
   fc->kind = FCK_DEAD;
   return res;
 }
@@ -219,6 +228,8 @@ extern void functionClause_free (/*@only@*/ functionClause node)
 {
   if (node != NULL)
     {
+      DPRINTF (("free: %s", functionClause_unparse (node)));
+
       switch (node->kind)
        {
        case FCK_GLOBALS:
@@ -234,10 +245,10 @@ extern void functionClause_free (/*@only@*/ functionClause node)
          stateClause_free (node->val.state);
          break;
        case FCK_ENSURES:
-         constraintList_free (node->val.ensures);
+         functionConstraint_free (node->val.constraint);
          break;
        case FCK_REQUIRES:
-         constraintList_free (node->val.requires);
+         functionConstraint_free (node->val.constraint);
          break;
        case FCK_DEAD:
          /* Nothing to release */