treewide: Replace <name>_cnt by n_<name>s and <name>_cap by allocated_<name>.
[pspp] / src / language / control / do-if.c
index 6b6dc16d4de600362035e9d39c35afa1f0e5e456..b361fba920702e4939395857481207f09bb8d71d 100644 (file)
@@ -75,7 +75,7 @@ struct do_if_trns
   {
     struct dataset *ds;         /* The dataset */
     struct clause *clauses;     /* Clauses. */
-    size_t clause_cnt;          /* Number of clauses. */
+    size_t n_clauses;           /* Number of clauses. */
     int past_END_IF_index;      /* Transformation just past last clause. */
   };
 
@@ -99,7 +99,7 @@ cmd_do_if (struct lexer *lexer, struct dataset *ds)
 {
   struct do_if_trns *do_if = xmalloc (sizeof *do_if);
   do_if->clauses = NULL;
-  do_if->clause_cnt = 0;
+  do_if->n_clauses = 0;
   do_if->ds = ds;
 
   ctl_stack_push (&do_if_class, do_if);
@@ -124,9 +124,12 @@ int
 cmd_else (struct lexer *lexer UNUSED, struct dataset *ds)
 {
   struct do_if_trns *do_if = ctl_stack_top (&do_if_class);
-  assert (ds == do_if->ds);
+
   if (do_if == NULL || !must_not_have_else (do_if))
     return CMD_CASCADING_FAILURE;
+
+  assert (ds == do_if->ds);
+
   add_else (do_if);
   return CMD_SUCCESS;
 }
@@ -186,8 +189,8 @@ must_not_have_else (struct do_if_trns *do_if)
 static bool
 has_else (struct do_if_trns *do_if)
 {
-  return (do_if->clause_cnt != 0
-          && do_if->clauses[do_if->clause_cnt - 1].condition == NULL);
+  return (do_if->n_clauses != 0
+          && do_if->clauses[do_if->n_clauses - 1].condition == NULL);
 }
 
 /* Parses a DO IF or ELSE IF expression and appends the
@@ -198,7 +201,7 @@ parse_clause (struct lexer *lexer, struct do_if_trns *do_if, struct dataset *ds)
 {
   struct expression *condition;
 
-  condition = expr_parse (lexer, ds, EXPR_BOOLEAN);
+  condition = expr_parse_bool (lexer, NULL, ds);
   if (condition == NULL)
     return CMD_CASCADING_FAILURE;
 
@@ -215,12 +218,12 @@ add_clause (struct do_if_trns *do_if, struct expression *condition)
 {
   struct clause *clause;
 
-  if (do_if->clause_cnt > 0)
+  if (do_if->n_clauses > 0)
     add_transformation (do_if->ds, break_trns_proc, NULL, do_if);
 
   do_if->clauses = xnrealloc (do_if->clauses,
-                              do_if->clause_cnt + 1, sizeof *do_if->clauses);
-  clause = &do_if->clauses[do_if->clause_cnt++];
+                              do_if->n_clauses + 1, sizeof *do_if->clauses);
+  clause = &do_if->clauses[do_if->n_clauses++];
   clause->condition = condition;
   clause->target_index = next_transformation (do_if->ds);
 }
@@ -245,7 +248,7 @@ do_if_trns_proc (void *do_if_, struct ccase **c, casenumber case_num UNUSED)
   struct do_if_trns *do_if = do_if_;
   struct clause *clause;
 
-  for (clause = do_if->clauses; clause < do_if->clauses + do_if->clause_cnt;
+  for (clause = do_if->clauses; clause < do_if->clauses + do_if->n_clauses;
        clause++)
     {
       if (clause->condition != NULL)
@@ -269,7 +272,7 @@ do_if_trns_free (void *do_if_)
   struct do_if_trns *do_if = do_if_;
   struct clause *clause;
 
-  for (clause = do_if->clauses; clause < do_if->clauses + do_if->clause_cnt;
+  for (clause = do_if->clauses; clause < do_if->clauses + do_if->n_clauses;
        clause++)
     expr_free (clause->condition);
   free (do_if->clauses);