RFR: 8230398: Remove NULL checks before FREE_C_HEAP_ARRAY

Thomas Schatzl thomas.schatzl at oracle.com
Wed Sep 11 08:39:20 UTC 2019


Hi,

On 9/10/19 3:39 PM, Leo Korinth wrote:
> 
[...]
>>
> I will remove NULL assignments of fields in destructors and keep them 
> otherwise.
> 
> I will also add a comment (// handles NULL pointers) to FreeHeap and 
> os::free to not only describe behaviour, but also to establish a contract.
> 
> Also fixed indentation miss in os_perf_windows.cpp
> 
> Incremental:
> http://cr.openjdk.java.net/~lkorinth/8230398/00_01
> 
> Full:
> http://cr.openjdk.java.net/~lkorinth/8230398/01
> 
> 
> running tests...

   still looks good.

Thomas


More information about the hotspot-dev mailing list