On May 26, 2011, at 1:21 PM, Charles Oliver Nutter wrote:

> isInvalid seems a bit double-negativy to me.

That's my hesitation.  Consider the use case:

  void maybeCleanUpMess(SwitchPoint sp) {
    if (!mysp.isInvalid())  return;
    ... // do cleanup after invalidation
  }

It reads better as:
    if (mysp.isValid())  return;

-- John
_______________________________________________
mlvm-dev mailing list
mlvm-dev@openjdk.java.net
http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev

Reply via email to