RFR: JDK-8013225: Refresh jdk's private ASM to the latest.

Paul Sandoz paul.sandoz at oracle.com
Fri Apr 26 11:43:07 UTC 2013


On Apr 26, 2013, at 1:26 AM, Kumar Srinivasan <kumar.x.srinivasan at oracle.COM> wrote:
> On 4/25/2013 3:53 PM, Mike Duigou wrote:
>> The restyling changes obfustucated things a bit but I didn't see anything of concern in casual review.
>> 
>> I had hoped to see the updated SmallSet that didn't try to implement Iterator directly.
> 
> Remi, Paul and Brian discussed that and struck a deal, maybe Paul/Brain
> can shed some light on that.
> 

If we had stayed with the method names Iterator.forEach and Iterable.forEach (the former was renamed to Iterator.forEachRemaining) then that would have forced a change to SmallSet to fix its rampant layering violation. I don't recall we struck a deal to fix it, but it would be nice if we could do so.

Paul.


More information about the core-libs-dev mailing list