RFR (M): 8201247: Various cleanups in the attach framework

[Chris Plummer]

Hi Christoph,

Can you explain a bit more about "fix handling of null values in 
ArgumentIterator::next". When does this turn up? Is there a test case?

Everything else looks good.

thanks,

Chris

On 4/6/18 8:01 AM, Langer, Christoph wrote:
>
> Hi,
>
> can I please get reviews for a set of clean up changes that I came 
> across when doing some integration work.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8201247 
> <https://bugs.openjdk.java.net/browse/JDK-8201247>
>
> Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8201247.0/ 
> <http://cr.openjdk.java.net/%7Eclanger/webrevs/8201247.0/>
>
> Detailed comments about the changes can be found in the bug.
>
> Thanks & best regards
>
> Christoph
>