8210307: 8210246 broke NMT jtreg tests
    Lindenmaier, Goetz 
    goetz.lindenmaier at sap.com
       
    Mon Sep  3 07:25:14 UTC 2018
    
    
  
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
    
    
More information about the hotspot-runtime-dev
mailing list