Mps br/timing: (cosmetic):

message.c: rename MessageClocked() as MessageIsClocked()
check.h: better name in future improvement comments

Copied from Perforce
 Change: 166801
 ServerID: perforce.ravenbrook.com

2 files changed, 5 insertions, 5 deletions

diff --git a/mps/code/check.h b/mps/code/check.h
index f5681e0bb71..e58a9d04ced 100644
--- a/mps/code/check.h
+++ b/mps/code/check.h
@@ -107,7 +107,7 @@ enum {
  * and <design/interface-c/#check.space>.
  *
  * @@@@ This is a test, not a CHECK macro -- it does not assert.
- * It should be renamed MATCHSIG.  RHSK 2006-12-13.
+ * It should be renamed TESTSIG.  RHSK 2006-12-13.
  */
 
 #define CHECKT(type, val)       ((val) != NULL && (val)->sig == type ## Sig)
@@ -240,7 +240,7 @@ enum {
  * satisfy rule.impl.trick.]
  *
  * @@@@ These are tests, not CHECK macros -- they do not assert.
- * They should be renamed MATCHTYPE etc.  RHSK 2006-12-13.
+ * They should be renamed TESTTYPE etc.  RHSK 2006-12-13.
  */
 
 /* compile-time check */
diff --git a/mps/code/message.c b/mps/code/message.c
index 424b0d7ed90..75b01d328d8 100644
--- a/mps/code/message.c
+++ b/mps/code/message.c
@@ -45,7 +45,7 @@ Bool MessageTypeCheck(MessageType type)
 
 /* See .message.clocked.  Currently finalization messages are the */
 /* only ones that can be numerous. */
-#define MessageClocked(message) ((message)->class->type \
+#define MessageIsClocked(message) ((message)->class->type \
                                  != MessageTypeFINALIZATION)
 
 Bool MessageCheck(Message message)
@@ -56,7 +56,7 @@ Bool MessageCheck(Message message)
   CHECKL(RingCheck(&message->queueRing));
   /* postedClock is uncheckable for clocked message types, */
   /* but must be 0 for unclocked message types: */
-  CHECKL(MessageClocked(message) || (message->postedClock == 0));
+  CHECKL(MessageIsClocked(message) || (message->postedClock == 0));
 
   return TRUE;
 }
@@ -132,7 +132,7 @@ void MessagePost(Arena arena, Message message)
     /* .message.clocked: Reading the clock with ClockNow() */
     /* involves an mpslib call, so we avoid it for message */
     /* types that may be numerous. */
-    if(MessageClocked(message)) {
+    if(MessageIsClocked(message)) {
       message->postedClock = ClockNow();
     }
     RingAppend(&arena->messageRing, &message->queueRing);