From d86cea5b9a14c8aac805c79422d387ddc8a064b6 Mon Sep 17 00:00:00 2001
From: Brian Norris <banorris@uci.edu>
Date: Mon, 16 Jul 2012 22:44:40 -0700
Subject: [PATCH 1/1] nodestack: re-insert falsely-declared "dead code"

"remove dead code...  loop entrance condition is i<backtrack.size().  Therefore,
this branch can never be taken."

False.

While the entrance condition prevents 'i == backtrack.size()', the loop may
exit with 'i == backtrack.size()', since i++ is executed after the last
iteration. Since we do not expect or want this condition to occur, though, I
transform this to a documented ASSERT().
---
 nodestack.cc | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/nodestack.cc b/nodestack.cc
index 131452d8..aa3a6c97 100644
--- a/nodestack.cc
+++ b/nodestack.cc
@@ -109,6 +109,9 @@ thread_id_t Node::get_next_backtrack()
 	for (i = 0; i < backtrack.size(); i++)
 		if (backtrack[i] == true)
 			break;
+	/* Backtrack set was empty? */
+	ASSERT(i != backtrack.size());
+
 	backtrack[i] = false;
 	numBacktracks--;
 	return int_to_id(i);
-- 
2.34.1