Hi Martin, Thanks for input. I have added the missing trailing newline to the test. Please find below updated webrev details:-
Webrev: http://cr.openjdk.java.net/~rpatil/8186171/webrev.01/ Regards, Deepak From: Martin Buchholz <marti...@google.com> Sent: Wednesday, June 6, 2018 11:33 PM To: Deepak Kejriwal <deepak.kejri...@oracle.com> Cc: Doug Lea <d...@cs.oswego.edu>; core-libs-dev <core-libs-dev@openjdk.java.net> Subject: Re: RFR: JDK8U Backport of 8186171: HashMap: Entry.setValue may not work after Iterator.remove() called for previous entries On Wed, Jun 6, 2018 at 11:01 AM, Martin Buchholz <mailto:marti...@google.com> wrote: Alright, I leave it to you. Your current change is fine. Except, add that trailing newline to the test!