review request (URGENT): 7051206: JSR 292 method name SwitchPoint.isValid is misleading to unwary users; should be hasBeenInvalidated

[John Coomes]

John Rose (john.r.rose at oracle.com) wrote:
> http://cr.openjdk.java.net/~jrose/7051206
> 
> This is a changed method name (and inverted sense), replacing SwitchPoint.isValid.  Here's the updated javadoc:
> 
> hasBeenInvalidated
> public boolean hasBeenInvalidated()
> Determines if this switch point has been invalidated yet.
> Discussion: Because of the one-way nature of invalidation, once a
> switch point begins to return true for hasBeenInvalidated, it will
> always do so in the future. On the other hand, a valid switch point
> visible to other threads may invalidated at any moment, due to a
                             ^^^ insert "be"
> request by another thread.
> ...

Aside from the trivial typo above, looks good to me.

-John