I agree. Thanks for the comments. -- John On Oct 24, 2010, at 3:49 AM, Rémi Forax wrote:
> Le 24/10/2010 12:20, john.r.r...@oracle.com a écrit : >> Changeset: e3a3904eea5d >> Author: jrose >> Date: 2010-10-24 03:20 -0700 >> URL: http://hg.openjdk.java.net/mlvm/mlvm/jdk/rev/e3a3904eea5d >> >> indy, meth: roll up current EG discussions to date into javadoc >> >> ! indy-args-6984311.patch >> + indy-bsm-postjdk7.patch >> ! indy-dep-6982754.patch >> ! meth-api-6981777.patch >> ! series >> > > Hi John, > In VolatileCallSite, invalidate could use setTargetVolatile instead of > setTarget() because the fallback type is already checked. > Also, invalidate that takes a List should not be an overload of > invalidate() and use a wildcard. I propose: > > public static void invalidateAll(List<? extends VolatileCallSite> sites) { > ... } > > > Rémi > _______________________________________________ > mlvm-dev mailing list > mlvm-dev@openjdk.java.net > http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev _______________________________________________ mlvm-dev mailing list mlvm-dev@openjdk.java.net http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev
