RFR: 7879: Automated Analysis taking very long time to produce results for Class Leak Rule and showing wrong results. [v2]

Brice Dutheil bdutheil at openjdk.org
Wed Oct 5 20:21:32 UTC 2022


On Wed, 5 Oct 2022 18:35:26 GMT, Suchita Chaturvedi <schaturvedi at openjdk.org> wrote:

>> I have a few nits and one necessary change before we can merge this.
>
> @Gunde @bric3 I have tried to implement all the review comments. Please re-review the PR and let me know if there is something which is still need to be corrected.

@Suchitainf There's still these two comment to address in my opinion: 
* https://github.com/openjdk/jmc/pull/419#discussion_r987769827
* https://github.com/openjdk/jmc/pull/419#discussion_r988313757

Note I'm really unsure about the result ordering there, may be @thegreystone or @Gunde may chime in there to make sure this change is ok.

-------------

PR: https://git.openjdk.org/jmc/pull/419


More information about the jmc-dev mailing list