RFR: 8248817: Windows: Improving common cross-platform code
kim.barrett at oracle.com
Thu Jul 30 11:54:50 UTC 2020
> On Jul 30, 2020, at 2:56 AM, David Holmes <david.holmes at oracle.com> wrote:
> Changeset: c35fba4bce35
> Author: dholmes
> Date: 2020-07-30 02:47 -0400
> URL: https://hg.openjdk.java.net/jdk/jdk/rev/c35fba4bce35
> 8250810: Push missing parts of JDK-8248817
> Summary: Push changes from JDK-8248817 that were accidentally excluded from the commit.
> Reviewed-by: kbarrett, dholmes
> Contributed-by: Ludovic Henry <luhenry at microsoft.com>
> ! src/hotspot/os/windows/os_windows.cpp
> ! src/hotspot/os_cpu/windows_x86/thread_windows_x86.cpp
For the record, I only reviewed the atomics changes (I always referred to
that change with my comments and approval; sorry that apparently wasn't
clear), so I think the other two parts (pushed under JDK-8250810) were only
reviewed by David. Not a disaster, but potentially an oops.
Just in general, I think it's better to split unrelated changes like this,
rather than having an omnibus bug/webrev. The changes were split into
multiple webrevs, but all under the same bug, which led to the confusion
about who had reviewed what.
> On 30/07/2020 4:40 pm, David Holmes wrote:
>> Hi Ludovic,
>> On 30/07/2020 11:45 am, Ludovic Henry wrote:
>>> Hi David,
>>> Looking at the change pushed , it seems like you pushed only part of the whole change (the bits related to atomics). It's missing the other two webrevs linked in the first email ( and ). What would be the best way to proceed? Would you like me to submit another review?
>> Aaarghhh! Sorry. I'll file a new bug and push the other bits under it.
>>>  https://hg.openjdk.java.net/jdk/jdk/rev/bda65def14de
>>>  http://cr.openjdk.java.net/~burban/luhenry/8248817-exception-handling/
>>>  http://cr.openjdk.java.net/~burban/luhenry/8248817-frames/
More information about the hotspot-runtime-dev