Pinging this RFR. It still needs a capital R reviewer. http://cr.openjdk.java.net/~emc/8016285/
On 06/21/13 19:21, Eric McCorkle wrote: > On 06/21/13 16:15, Aleksey Shipilev wrote: >> On 06/21/2013 11:57 PM, Eric McCorkle wrote: >>> The webrev is here: >>> http://cr.openjdk.java.net/~emc/8016285/ >> >> Looks generally good (but not a Reviewer). >> >> A few questions though: >> a) Are we em-bracing the brace-less control flow blocks? > > Fixed it. > >> b) Should hasRealParameterData be transient? >> c) Should hasRealParameterData be volatile? (now is being implicitly >> protected by $parameters volatile write/read, but it is fragile) > > Transient yes. Volatile, I'd say I think not, since It's written before > it's read on all possible paths, and all writes are the same value. > But... I set it volatile just to be safe from reordering problems. > > Webrev's been updated. > http://cr.openjdk.java.net/~emc/8016285/ >