8210307: 8210246 broke NMT jtreg tests

[Thomas Stüfe]

No problem, that was quite trivial. I'll wait for the submit tests to
come back and then I'll push.

Thank you,

Thomas

On Mon, Sep 3, 2018 at 10:19 AM, Ioi Lam <ioi.lam at oracle.com> wrote:
> Hi Thomas,
>
> Looks good. Thanks for fixing this. I plead guilty as charged :-(
>
> - Ioi
>
>
>
> On 9/3/18 12:17 AM, Thomas Stüfe wrote:
>>
>> 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
>
>