RFR: 7272: Method profiling page should show the frame type

Marcus Hirt hirt at openjdk.java.net
Thu Jun 17 21:16:28 UTC 2021


On Thu, 17 Jun 2021 21:12:31 GMT, Marcus Hirt <hirt at openjdk.org> wrote:

>> application/org.openjdk.jmc.flightrecorder.ui/src/main/java/org/openjdk/jmc/flightrecorder/ui/pages/internal/MethodWithFrameTypeLabelProvider.java line 144:
>> 
>>> 142: 			return ((IDisplayable) key).displayUsing(IDisplayable.VERBOSE);
>>> 143: 		}
>>> 144: 		return key.toString();
>> 
>> When giving this a try I'm finding that the key object is a `MethodWithFrameType`, and as such it doesn't get formatted into any human readable way.
>> 
>> ![2021-06-17-151839_1916x1036_scrot](https://user-images.githubusercontent.com/10425301/122470248-d40da580-cf8b-11eb-8286-a3855edf00d7.png)
>
>> Is the design of these icons standardized, and is there a place to find out in jmc what they mean? Should this information be added to the tooltip perhaps? Otherwise I added a couple of comments inline, other than those I think it looks good.
> 
> Yeah, it's the same geometric figures as Eclipse is using. ;)

Good catch with the tooltip! Thanks!

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

PR: https://git.openjdk.java.net/jmc/pull/267


More information about the jmc-dev mailing list