]> andersk Git - splint.git/blobdiff - src/constraintExpr.c
Fixed /*@i@*/ splintme errors.
[splint.git] / src / constraintExpr.c
index 3d96627a0c0420ec9b7b5cfc2f28009a1689614e..f6284c6ea5d0de7719b63cf4b4546601559fec6d 100644 (file)
 # include "exprChecks.h"
 # include "exprNodeSList.h"
 
-/*@-czechfcns@*/
 
-/*@access exprNode@*/ /* !!! NO! Don't do this recklessly! */
-// /*@-nullderef@*/ /* !!! DRL needs to fix this code! */
-// /*@-nullstate@*/ /* !!! DRL needs to fix this code! */
-// /*@-temptrans@*/ /* !!! DRL needs to fix this code! */
-
-/*@only@*/ /*@notnull@*/
-constraintExpr constraintExpr_makeIntLiteral (long i);
 static ctype constraintExpr_getOrigType (constraintExpr p_e);
 static bool constraintExpr_hasTypeChange(constraintExpr p_e) /*@*/;
 
@@ -301,7 +293,8 @@ static bool isZeroBinaryOp (constraintExpr expr)
          
          constraintExpr_free(expr1);
          constraintExpr_free(expr);
-         
+
+         llassert (constraintExpr_isDefined(temp) );
          return temp;
        }
       else
@@ -356,7 +349,9 @@ static bool isZeroBinaryOp (constraintExpr expr)
        }
     }
    DPRINTF ((message ("After combine %s", constraintExpr_unparse(expr) ) ) );
-  return expr;
+
+   llassert(constraintExpr_isDefined(expr) );
+   return expr;
 }
 
 /*@special@*/
@@ -432,6 +427,7 @@ constraintExpr constraintExpr_copy (constraintExpr expr)
   return ret;
 }
 
+/*@access exprNode@*/
 constraintExpr constraintExpr_makeExprNode (exprNode e)
 {
  sRef s;
@@ -485,12 +481,13 @@ constraintExpr constraintExpr_makeExprNode (exprNode e)
         ret = constraintExpr_parseMakeBinaryOp (ce1, tok, ce2);         
        }
 
-     
-     /*@i333*/
-     /* uncomment this block to activate the cheesy heuristic
-       for handling sizeof expressions
+
+        /* define this block to activate the cheesy heuristic
+       for handling sizeof expressions*/
+#if 0     
+
        
-     / *
+     /*
        drl 8-11-001
        
        We handle expressions containing sizeof with the rule
@@ -498,7 +495,7 @@ constraintExpr constraintExpr_makeExprNode (exprNode e)
 
        This is the total wronge way to do this but...
        it may be better than nothing
-     * /
+     */
 
      
       
@@ -517,7 +514,8 @@ constraintExpr constraintExpr_makeExprNode (exprNode e)
           ret =  oldconstraintExpr_makeTermExprNode (e);
           }
        }
-     */
+#endif
+     
      else
         ret = oldconstraintExpr_makeTermExprNode (e);
    
@@ -557,7 +555,6 @@ constraintExpr constraintExpr_makeExprNode (exprNode e)
    case XPR_COMMA:
      t = exprData_getPairA(data);
      ret = constraintExpr_makeExprNode(t);
-     /*@i3434*/ /* drl: I'm not sure if this is right.  I'm adding a break to quiet Splint */
      break;
    default:
      ret = oldconstraintExpr_makeTermExprNode (e);
@@ -566,6 +563,11 @@ constraintExpr constraintExpr_makeExprNode (exprNode e)
   return ret;
 }
 
+/*@noaccess exprNode@*/
+
+
+
+
 /*@only@*/ constraintExpr constraintExpr_makeTermExprNode (/*@exposed@*/ exprNode e)
 {
   return  oldconstraintExpr_makeTermExprNode(e);
@@ -1051,17 +1053,17 @@ bool constraintExpr_same (constraintExpr expr1, constraintExpr expr2)
   BADEXIT;
 }
 
-bool constraintExpr_search (/*@observer@*/ constraintExpr c, /*@observer@*/ constraintExpr old)
+bool 
+constraintExpr_search (/*@observer@*/ constraintExpr c, 
+                      /*@observer@*/ constraintExpr old)
 {
   bool ret = FALSE;
   constraintExprKind kind;
   constraintExpr temp;
   
-  if ( constraintExpr_similar (c, old) )
+  if (constraintExpr_similar (c, old))
     {
-      DPRINTF((message ("Found  %q",
-                       constraintExpr_unparse(old)
-                       )));
+      DPRINTF (("Found  %q", constraintExpr_unparse (old)));
       return TRUE;
     }
 
@@ -1479,10 +1481,7 @@ static /*@only@*/ constraintExpr constraintExpr_simplifyunaryExpr (/*@only@*/ co
   DPRINTF ((message ("Doing constraintExpr_simplify:%s", constraintExpr_unparse (c) ) ) );  
   
 
-  /*@i22*/
   
-  /* drl: I think this is an Splint bug */
-
   llassert ( constraintExpr_isDefined (c) );
   if (constraintExpr_isUndefined (c) )
     {
@@ -1542,11 +1541,11 @@ cstring constraintExpr_unparse (/*@temp@*/ /*@observer@*/ constraintExpr ex) /*@
 
             if (context_getFlag (FLG_PARENCONSTRAINT) )
              {
-               st = message ("(%q) ", constraintTerm_print (constraintExprData_termGetTerm (ex->data)));
+               st = message ("(%q) ", constraintTerm_unparse (constraintExprData_termGetTerm (ex->data)));
              }
            else
              {
-               st = message ("%q", constraintTerm_print (constraintExprData_termGetTerm (ex->data)));
+               st = message ("%q", constraintTerm_unparse (constraintExprData_termGetTerm (ex->data)));
              }
       break;
     case unaryExpr:
@@ -1887,32 +1886,29 @@ doFixResultTerm (/*@only@*/ constraintExpr e, /*@exposed@*/ exprNode fcnCall)
 {
   constraintTerm t;
   sRef s;
-
-  
   constraintExprData data;
   constraintExprKind kind;
-  
   constraintExpr ret;
-
-  llassert(constraintExpr_isDefined (e) );
+  
+  llassert (constraintExpr_isDefined (e) );
   
   data = e->data;
   kind = e->kind;
-
-  llassert(kind == term);
-
+  
+  llassert (kind == term);
+  
   t = constraintExprData_termGetTerm (data);
-  llassert (constraintTerm_isDefined(t) );
-
+  llassert (constraintTerm_isDefined (t));
+  
   ret = e;
-  switch (constraintTerm_getKind(t) )
+
+  switch (constraintTerm_getKind (t))
     {
-    case EXPRNODE:
-      break;
-    case INTLITERAL:
+    case CTT_EXPR:
+    case CTT_INTLITERAL:
       break;
       
-    case SREF:
+    case CTT_SREF:
       s = constraintTerm_getSRef(t);
       if (sRef_isResult (s))
        {
@@ -1932,7 +1928,11 @@ doFixResultTerm (/*@only@*/ constraintExpr e, /*@exposed@*/ exprNode fcnCall)
   return ret;
   
 }
-/*
+
+#if 0
+
+/*to be used for structure checking */
+
 / *@only@* / static constraintExpr
 doSRefFixInvarConstraintTerm (/ *@only@* / constraintExpr e,
  sRef s, ctype ct)
@@ -1956,21 +1956,21 @@ doSRefFixInvarConstraintTerm (/ *@only@* / constraintExpr e,
 
   switch (constraintTerm_getKind(t))
     {
-    case EXPRNODE:
-      DPRINTF((message ("%q @ %q ", constraintTerm_print(t),
+    case CTT_EXPR:
+      DPRINTF((message ("%q @ %q ", constraintTerm_unparse(t),
                        fileloc_unparse (constraintTerm_getFileloc(t) ) ) ));
       break;
-    case INTLITERAL:
-      DPRINTF((message (" %q ", constraintTerm_print (t)) ));
+    case CTT_INTLITERAL:
+      DPRINTF((message (" %q ", constraintTerm_unparse (t)) ));
       break;
       
-    case SREF:
+    case CTT_SREF:
       / * evans 2001-07-24: constants should use the original term * /
       if (!constraintTerm_canGetValue (t))
        {
          sRef snew;
          DPRINTF ((message("Doing sRef_fixInvarConstraint for %q ", 
-                            constraintTerm_print (t) ) ));
+                            constraintTerm_unparse (t) ) ));
 
          snew = fixSref (ct, s, constraintTerm_getSRef(t));
 
@@ -1991,7 +1991,7 @@ doSRefFixInvarConstraintTerm (/ *@only@* / constraintExpr e,
   return ret;
   
 }
-*/
+#endif
  
 /*drl moved from constriantTerm.c 5/20/001*/
 /*@only@*/ static constraintExpr 
@@ -2025,20 +2025,19 @@ doSRefFixConstraintParamTerm (/*@only@*/ constraintExpr e, /*@observer@*/ /*@tem
 
   switch (constraintTerm_getKind(t))
     {
-    case EXPRNODE:
-      DPRINTF((message ("%q @ %q ", constraintTerm_print(t),
+    case CTT_EXPR:
+      DPRINTF((message ("%q @ %q ", constraintTerm_unparse(t),
                        fileloc_unparse (constraintTerm_getFileloc(t) ) ) ));
       break;
-    case INTLITERAL:
-      DPRINTF((message (" %q ", constraintTerm_print (t)) ));
+    case CTT_INTLITERAL:
+      DPRINTF((message (" %q ", constraintTerm_unparse (t)) ));
       break;
-      
-    case SREF:
+    case CTT_SREF:
       /* evans 2001-07-24: constants should use the original term */
       if (!constraintTerm_canGetValue (t))
        {
          DPRINTF ((message("Doing sRef_fixConstraintParam for %q ", 
-                            constraintTerm_print (t) ) ));
+                            constraintTerm_unparse (t) ) ));
          ret = sRef_fixConstraintParam (constraintTerm_getSRef(t), arglist);
          
          constraintExpr_free (e);
@@ -2092,6 +2091,8 @@ bool constraintExpr_isBinaryExpr (/*@observer@*/ constraintExpr c)
 /*drl added 8/08/001 */
 bool constraintExpr_isTerm (/*@observer@*/ constraintExpr c) /*@*/
 {
+  llassert(constraintExpr_isDefined (c) );
+  
   if (c->kind == term)
     return TRUE;
 
@@ -2103,6 +2104,8 @@ bool constraintExpr_isTerm (/*@observer@*/ constraintExpr c) /*@*/
 {
   constraintTerm term;
   
+  llassert(constraintExpr_isDefined (c) );
+  
   llassert(constraintExpr_isTerm(c) );
 
   term = constraintExprData_termGetTerm(c->data);
@@ -2150,11 +2153,20 @@ static constraintExpr  binaryExpr_undump (FILE *f)
 
   str = fgets(os, MAX_DUMP_LINE_LENGTH, f);
 
+  if (! mstring_isDefined(str) )
+    {
+      llfatalbug(message("Library file is corrupted") );
+    }
   
   binaryOp = (constraintExprBinaryOpKind) reader_getInt(&str);
   
   str = fgets(os, MAX_DUMP_LINE_LENGTH, f);
 
+  if (! mstring_isDefined(str) )
+    {
+      llfatalbug(message("Library file is corrupted") );
+    }
+
   reader_checkChar (&str, 'e');
   reader_checkChar (&str, '1');
   
@@ -2205,6 +2217,11 @@ static  constraintExpr  unaryExpr_undump ( FILE *f)
   os = str;
   str = fgets(os, MAX_DUMP_LINE_LENGTH, f);
 
+  if (! mstring_isDefined(str) )
+    {
+      llfatalbug(message("Library file is corrupted") );
+    }
+
   unaryOp = (constraintExprUnaryOpKind) reader_getInt(&str);
   
   expr = constraintExpr_undump (f);
@@ -2222,6 +2239,8 @@ void  constraintExpr_dump (/*@observer@*/ constraintExpr expr,  FILE *f)
   constraintTerm t;
   
 
+  llassert(constraintExpr_isDefined(expr) );
+  
   DPRINTF((message("constraintExpr_dump:: dumping constraintExpr %s",
                   constraintExpr_unparse(expr)
                   ) ));
@@ -2260,6 +2279,11 @@ void  constraintExpr_dump (/*@observer@*/ constraintExpr expr,  FILE *f)
   
   s = fgets(os, MAX_DUMP_LINE_LENGTH, f);
 
+  if (! mstring_isDefined(s) )
+    {
+      llfatalbug(message("Library file is corrupted") );
+    }
+
   kind = (constraintExprKind) reader_getInt(&s);
 
   free (os);
@@ -2321,6 +2345,9 @@ int constraintExpr_getDepth (constraintExpr ex)
 
 bool  constraintExpr_canGetCType (constraintExpr e) /*@*/
 {
+  if (constraintExpr_isUndefined(e) )
+    return FALSE;
+  
   if (e->kind == term)
     {
       return TRUE;
@@ -2336,7 +2363,9 @@ bool  constraintExpr_canGetCType (constraintExpr e) /*@*/
 ctype constraintExpr_getCType (constraintExpr e) /*@*/
 {
   constraintTerm t;
+
+  llassert(constraintExpr_isDefined(e) );
+
   llassert(constraintExpr_canGetCType(e) );
 
   switch (e->kind)
@@ -2364,6 +2393,7 @@ ctype constraintExpr_getCType (constraintExpr e) /*@*/
 
 static bool constraintExpr_hasTypeChange(constraintExpr e)
 {
+  llassert(constraintExpr_isDefined(e) );
   if (constraintExpr_isDefined((e)) && (e->ct == TRUE) )
     {
       return TRUE;
@@ -2389,6 +2419,7 @@ static bool constraintExpr_hasTypeChange(constraintExpr e)
 static ctype constraintExpr_getOrigType (constraintExpr e)
 {
 
+  llassert(constraintExpr_isDefined(e) );
   llassert(constraintExpr_hasTypeChange(e) );
   
   
@@ -2420,6 +2451,8 @@ static /*@only@*/ constraintExpr constraintExpr_div (/*@only@*/ constraintExpr e
   return e;
 }
 
+
+/*@access exprNode@*/ 
 static /*@only@*/ constraintExpr  constraintTerm_simpleDivTypeExprNode(/*@only@*/ constraintExpr e, ctype ct)
 {
   exprData data;
@@ -2427,6 +2460,8 @@ static /*@only@*/ constraintExpr  constraintTerm_simpleDivTypeExprNode(/*@only@*
   lltok tok;
   constraintTerm t;
 
+  llassert(constraintExpr_isDefined(e) );
+  
   DPRINTF((
           message("constraintTerm_simpleDivTypeExprNode e=%s, ct=%s",
                   constraintExpr_print(e), ctype_unparse(ct)
@@ -2436,6 +2471,9 @@ static /*@only@*/ constraintExpr  constraintTerm_simpleDivTypeExprNode(/*@only@*
   t = constraintExprData_termGetTerm(e->data);
   
   expr = constraintTerm_getExprNode(t);
+
+  llassert(constraintExpr_isDefined(e) );
+  llassert(exprNode_isDefined(expr) );
   
   if (expr->kind == XPR_OP)
     {
@@ -2446,6 +2484,9 @@ static /*@only@*/ constraintExpr  constraintTerm_simpleDivTypeExprNode(/*@only@*
       tok = exprData_getOpTok (data);
       if (lltok_isMult(tok) )
        {
+         llassert(exprNode_isDefined(t1) && exprNode_isDefined(t2) );
+         /*drl 3/2/2003 we know this from the fact that it's a
+           multiplication operation...*/
          
          if  ((t1->kind == XPR_SIZEOF) || (t1->kind == XPR_SIZEOFT) )
            {
@@ -2482,11 +2523,20 @@ static /*@only@*/ constraintExpr  constraintTerm_simpleDivTypeExprNode(/*@only@*
                }
              else
                {
-                 ct2 = qtype_getType (exprData_getType(exprData_getSingle (t2->edata)->edata ) );
+                 exprNode exprTemp;
+                 exprData eDTemp;
+                 
+                 exprTemp = exprData_getSingle (t2->edata);
+
+                 llassert(exprNode_isDefined(exprTemp) );
+                 eDTemp = exprTemp->edata;
+                 
+                 ct2 = qtype_getType (exprData_getType(eDTemp ) );
+                 
                }
              if (ctype_match (ctype_makePointer(ct2),ct) )
                {
-                 /* sloopy way to do this... */ /*@i22*/
+                 /*a bit of a sloopy way to do this but... */
                                  constraintExpr_free(e);
                  return constraintExpr_makeExprNode(t1);
                }
@@ -2500,13 +2550,15 @@ static /*@only@*/ constraintExpr  constraintTerm_simpleDivTypeExprNode(/*@only@*
     }
   return (constraintExpr_div (e, ct) );
 }
+/*@noaccess exprNode@*/ 
 
 static /*@only@*/ constraintExpr simpleDivType (/*@only@*/ constraintExpr e, ctype ct)
 {
-  /*@i333*/
   DPRINTF(( (message("simpleDiv got %s ", constraintExpr_unparse(e) ) )
            ));
 
+  llassert(constraintExpr_isDefined(e) );
+  
   switch (e->kind)
     {
     case term:
@@ -2584,6 +2636,8 @@ bool  constraintExpr_isConstantOnly ( constraintExpr e )
                     constraintExpr_unparse(e) ) )
            ));
 
+  llassert(constraintExpr_isDefined(e) );
+
   switch (e->kind)
     {
     case term:
This page took 0.055443 seconds and 4 git commands to generate.