RFR: 8291801: IGV: Broken button "Hide graphs which are the same as the previous graph" [v2]
Koichi Sakata
ksakata at openjdk.org
Fri Sep 30 08:43:25 UTC 2022
On Thu, 29 Sep 2022 09:36:04 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:
>> Koichi Sakata 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:
>>
>> - Set value to hideDuplicates field
>> - Merge remote-tracking branch 'upstream/master' into JDK-8291801
>> - Remove extra whitespaces
>> - Improve the test
>> - Add the test method
>> - Remove extra code
>> - Fix the broken button "Hide graphs which are the same as the previous graph" in IGV
>
> I could also reproduce the issue on Windows 10, using JDK 11.0.16.
>
> I noticed an interesting pattern (on both Linux and Windows): if I iterate a few times the following sequence, the number of hidden graphs grows monotonically (but non-deterministically), and after a few iterations all graphs expected to be hidden are indeed hidden:
> 1. Open `hello.xml`
> 2. Open any graph of the `virtual void java.lang.Object.<init>()` group
> 3. Click on the "Hide graphs which are the same..." button
> 4. Click on "Remove all graphs and groups" button in the Outline window.
@robcasloz I can fully reproduce them on windows! I'll start to fix the behavior. I didn't expect this to depend on the OS. I highly appreciate your support.
@tobiasholenstein Thank you for your advice! I've seen `Difference.java` before writing this patch, too. So I agree with your thought that the function of `Difference` class is too much for this PR. I'll try to fix the `equals` problem in the `isSameContent` method, change in the timing of determining duplications of graphs.
-------------
PR: https://git.openjdk.org/jdk/pull/10440
More information about the hotspot-compiler-dev
mailing list