RFR: 8310228: Improve error reporting for uncaught native exceptions on Windows [v3]
Daniel Jeliński
djelinski at openjdk.org
Thu Sep 21 05:16:49 UTC 2023
On Wed, 20 Sep 2023 19:40:46 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
>> Jorn Vernee has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
>>
>> - use executeTestJvm to forward arguments
>> - Merge branch 'master' into UncaughtExWin
>> - address review comments
>> - Merge branch 'master' into UncaughtExWin
>> - remove uncaught exception string
>> - Improve reporting of uncaught native exceptions
>> - improve uncaught exception handling on Windows
>
> src/hotspot/os/windows/os_windows.cpp line 288:
>
>> 286: #endif
>> 287:
>> 288: LONG WINAPI Uncaught_Exception_Handler(struct _EXCEPTION_POINTERS* exceptionInfo);
>
> can you make this method static?
the forward declaration (this line) should be static too, even if the VS compiler doesn't mind.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14523#discussion_r1332470449
More information about the hotspot-runtime-dev
mailing list