Add option to include method modifiers

src/arguments.cpp

@@ -353,6 +353,9 @@ Error Arguments::parse(const char* args) {
             CASE("meminfolog")
                 _log_meminfo_on_dump = true;
 
+            CASE("includemm")
+                _includemm = true;
+
             DEFAULT()
                 if (_unknown_arg == NULL) _unknown_arg = arg;
         }

src/arguments.h

@@ -157,6 +157,7 @@ class Arguments {
     bool _sched;
     bool _live;
     bool _fdtransfer;
+    bool _includemm;
     const char* _fdtransfer_path;
     int _style;
     CStack _cstack;
@@ -221,7 +222,8 @@ class Arguments {
         _title(NULL),
         _minwidth(0),
         _reverse(false),
-        _log_meminfo_on_dump(false) {
+        _log_meminfo_on_dump(false),
+        _includemm(false) {
     }
 
     ~Arguments();

src/frameName.cpp

@@ -95,6 +95,7 @@ FrameName::FrameName(Arguments& args, int style, int epoch, Mutex& thread_names_
 {
     // Require printf to use standard C format regardless of system locale
     _saved_locale = uselocale(newlocale(LC_NUMERIC_MASK, "C", (locale_t)0));
+    _includemm = args._includemm;
 
     buildFilter(_include, args._buf, args._include);
     buildFilter(_exclude, args._buf, args._exclude);
@@ -168,6 +169,22 @@ void FrameName::javaMethodName(jmethodID method) {
     char* class_name = NULL;
     char* method_name = NULL;
     char* method_sig = NULL;
+    jint modifiers = 0;
+
+    // Based on: https://docs.oracle.com/javase/specs/jvms/se7/html/jvms-4.html#:~:text=Table%C2%A04.5.%C2%A0Method%20access%20and%20property%20flags
+    std::map<int, std::string> access_flags;
+    access_flags[0x0001] = "public";
+    access_flags[0x0002] = "private";
+    access_flags[0x0004] = "protected";
+    access_flags[0x0008] = "static";
+    access_flags[0x0010] = "final";
+    access_flags[0x0020] = "synchronized";
+    access_flags[0x0040] = "bridge";
+    access_flags[0x0080] = "varargs";
+    access_flags[0x0100] = "native";
+    access_flags[0x0400] = "abstract";
+    access_flags[0x0800] = "strict";
+    access_flags[0x1000] = "synthetic";
 
     jvmtiEnv* jvmti = VM::jvmti();
     jvmtiError err;
@@ -177,6 +194,14 @@ void FrameName::javaMethodName(jmethodID method) {
         (err = jvmti->GetClassSignature(method_class, &class_name, NULL)) == 0) {
         // Trim 'L' and ';' off the class descriptor like 'Ljava/lang/Object;'
         javaClassName(class_name + 1, strlen(class_name) - 2, _style);
+        if (_includemm) {
+            jvmti->GetMethodModifiers(method, &modifiers);
+            for (std::map<int, std::string>::const_iterator iterator = access_flags.begin(); iterator != access_flags.end(); iterator++) {
+                if (modifiers & iterator->first) {
+                    _str.append(".").append(iterator->second);
+                }
+            }
+        }
         _str.append(".").append(method_name);
         if (_style & STYLE_SIGNATURES) {
             if (_style & STYLE_NO_SEMICOLON) {

src/frameName.h

@@ -75,6 +75,7 @@ class FrameName {
     Mutex& _thread_names_lock;
     ThreadMap& _thread_names;
     locale_t _saved_locale;
+    bool _includemm;
 
     void buildFilter(std::vector<Matcher>& vector, const char* base, int offset);
     const char* decodeNativeSymbol(const char* name);