« Return to Thread: Request for review (XS): Wrong filter predicate of visible locals in SA JSJavaFrame

Re: Request for review (XS): Wrong filter predicate of visible locals in SA JSJavaFrame

by staffan.larsen :: Rate this Message:

Hi Kris,

This fix looks good to me. I've created a bug for you: 7162726. I'll be happy to push the fix for you once the review is done (need one Reviewer).

Thanks,

/Staffan

On 19 apr 2012, at 06:49, Krystal Mok wrote:

Hi all,

There seems to be a bug in the filtering logic of visible local variable in Serviceability Agent's JSJavaFrame.

I've made a patch here: [1]. The diff is against tip of jdk8/jdk8/hotspot

An example of running CLHSDB before and after applying the patch is avaiable [2].

Could anyone please review this for me?

- Kris

[1]: https://gist.github.com/2418558#file_getlocal.patch
[2]: https://gist.github.com/2418558#file_clhsdb_session2

« Return to Thread: Request for review (XS): Wrong filter predicate of visible locals in SA JSJavaFrame