RFR: 8185746: Remove Mutex destructor assertion
Kim Barrett
kim.barrett at oracle.com
Wed Aug 2 23:31:54 UTC 2017
> On Aug 2, 2017, at 7:23 PM, David Holmes <david.holmes at oracle.com> wrote:
>
> Hi Kim,
>
> On 3/08/2017 8:41 AM, Kim Barrett wrote:
>> Please review this small change to improve the debugging experience
>> when a mutex is destroyed in a bad state.
>> I've removed the assert in ~Mutex, which was making the same checks as
>> in ~Monitor, but providing much less information.
>
> Okay. Please can you add a comment to the ~Mutex noting that eg:
>
> // Defer any state checking to ~Monitor
> Mutex::~Mutex() { }
Well, I was waffling over just removing it completely.
More information about the hotspot-runtime-dev
mailing list