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