On 09/27/2017 02:38 PM, Paul Sandoz wrote: >> Looks good, i just need to look a little more closely at the >> ConcurrentSkipListMap changes. >> > > > 328 * Note that, with VarHandles, a boolean result of > 329 * compareAndSet must be declared even if not used. > This should not be necessary, since for such methods the return is not > polymorphic. Did you observe some performance or code generation issues? >
It was necessary at the time I started rewriting this in June, but apparently not now. I'll get rid of the annoying unnecessary declarations. Thanks to whoever fixed this! -Doug
