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.
>>
>> 
>
>> 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