From: Brian Norris <banorris@uci.edu>
Date: Sat, 16 Feb 2013 02:08:09 +0000 (-0800)
Subject: model: improve get_next_thread() comments
X-Git-Url: http://plrg.eecs.uci.edu/git/?p=c11tester.git;a=commitdiff_plain;h=0a0177d3795ceb0eea9e0e803148df36038776a4

model: improve get_next_thread() comments

It's curious that get_next_thread() needs the current ModelAction as a
parameter. It helps to clarify that this argument is somwhat "optional."
---

diff --git a/model.cc b/model.cc
index 2c0ec9ba..655c511d 100644
--- a/model.cc
+++ b/model.cc
@@ -214,8 +214,9 @@ Node * ModelChecker::get_curr_node() const
  * when exploring a new execution ordering), in which case we defer to the
  * scheduler.
  *
- * @param curr The current ModelAction. This action might guide the choice of
- * next thread.
+ * @param curr Optional: The current ModelAction. Only used if non-NULL and it
+ * might guide the choice of next thread (i.e., THREAD_CREATE should be
+ * followed by THREAD_START, or ATOMIC_RMWR followed by ATOMIC_{RMW,RMWC})
  * @return The next chosen thread to run, if any exist. Or else if no threads
  * remain to be executed, return NULL.
  */