8210307: 8210246 broke NMT jtreg tests

[Thomas Stüfe]

Thank you Goetz.

On Mon, Sep 3, 2018 at 9:25 AM, Lindenmaier, Goetz
<goetz.lindenmaier at sap.com> wrote:
> Hi Thomas,
>
> looks good, thanks for fixing this!
>
> Best regards,
>   Goetz.
>
>> -----Original Message-----
>> From: hotspot-runtime-dev <hotspot-runtime-dev-
>> bounces at openjdk.java.net> On Behalf Of Thomas Stüfe
>> Sent: Montag, 3. September 2018 09:18
>> To: Hotspot dev runtime <hotspot-runtime-dev at openjdk.java.net>
>> Subject: 8210307: 8210246 broke NMT jtreg tests
>>
>> Hi all,
>>
>> may I please have reviews for this tiny and trivial fix:
>>
>> cr: http://cr.openjdk.java.net/~stuefe/webrevs/8210307-8210246-broke-
>> NMT-jtreg-tests/webrev.00/webrev/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8210307
>>
>> 8210246 macrofied (don't know a better name :) the memory types, which
>> is fine, but added additional quotes, which changes NMT output and
>> breaks all tests.
>>
>> Fix is very simple. Just remove the additional quotes, which reverts
>> NMT output to what it was before.
>>
>> Thank you,
>>
>> Thomas