On Thu, 20 Apr 2023 08:36:53 GMT, Afshin Zafari <d...@openjdk.org> wrote:
>> JVMTI does not abort on OOM it reports an error, so we definitely do not
>> want a terminating allocator!
>>
>> jvmtiError
>> JvmtiEnv::CreateRawMonitor(const char* name, jrawMonitorID* monitor_ptr) {
>> JvmtiRawMonitor* rmonitor = new JvmtiRawMonitor(name);
>> NULL_CHECK(rmonitor, JVMTI_ERROR_OUT_OF_MEMORY);
>
> The new operator is removed.
We decide this at the call site though by adding the nothrow parameter. Adding
an overloaded operator new without a nothrow parameter that we're not supposed
to call seems very marginally useful. ie not useful.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13498#discussion_r1172958617
