promise: rename check_promise() -> has_failed()

[model-checker.git] / promise.cc

diff --git a/promise.cc b/promise.cc
index 943a0f4f9ba09f1d7a7c67521b1c570ca6db0bf2..7b110020cd869514c0f958055fea357f9f3c42ba 100644 (file)
--- a/promise.cc
+++ b/promise.cc
@@ -19,21 +19,21 @@ bool Promise::eliminate_thread(thread_id_t tid)
                return false;
 
        synced_thread[id] = true;
-       unsigned int sync_size = synced_thread.size();
-       int promise_tid = id_to_int(read->get_tid());
-       for (unsigned int i = 1; i < model->get_num_threads(); i++) {
-               if ((i >= sync_size || !synced_thread[i]) && ((int)i != promise_tid) && model->is_enabled(int_to_id(i))) {
-                       return false;
-               }
-       }
-       return true;
+       return has_failed();
 }
 
-bool Promise::check_promise() const
+/**
+ * Check if this promise has failed. A promise can fail when all threads which
+ * could possibly satisfy the promise have been eliminated.
+ *
+ * @return True, if this promise has failed; false otherwise
+ */
+bool Promise::has_failed() const
 {
        unsigned int sync_size = synced_thread.size();
+       int promise_tid = id_to_int(read->get_tid());
        for (unsigned int i = 1; i < model->get_num_threads(); i++) {
-               if ((i >= sync_size || !synced_thread[i]) && model->is_enabled(int_to_id(i))) {
+               if ((i >= sync_size || !synced_thread[i]) && ((int)i != promise_tid) && model->is_enabled(int_to_id(i))) {
                        return false;
                }
        }