RFR: 6787: Simplified stack trace tree model [v2]

Alex Ciminian github.com+348973+cimi at openjdk.java.net
Thu Nov 19 16:12:26 UTC 2020


On Thu, 19 Nov 2020 14:23:01 GMT, Marcus Hirt <hirt at openjdk.org> wrote:

>> application/org.openjdk.jmc.flightrecorder.flameview/src/main/java/org/openjdk/jmc/flightrecorder/flameview/FlameGraphJSONMarshaller.java line 1:
>> 
>>> 1: /*
>> 
>> Normally we use initial capital case and the rest lower case for both acronyms and initialisms. You may see exceptions in the code base, since (IIRC) we used to have a different ruler separating them, and also taking into account how long they were, but eventually I believe we settled on initial capital case, the rest lower to keep things simple.
>
> This is true for methods as well - will not add this to all places.

I got this habit from Go, the style guide specifically states that all acronyms should be uppercase.

It took a while to get used to but I started liking it afterwards ��

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

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


More information about the jmc-dev mailing list