sas added a comment.

Just a couple nits inline. Also, did you run `clang-format` on your change? I 
see some issues with `if`s and the associated parentheses.



================
Comment at: source/Plugins/Process/elf-core/ProcessElfCore.cpp:215
+  }
+  if (siginfo_signal_found == false) {
+    // If we don't have signal from SIGINFO use the signal from each threads
----------------
Nit: `if (!siginfo_signal_found)`.


================
Comment at: source/Plugins/Process/elf-core/ProcessElfCore.cpp:218
+    // PRSTATUS note.
+    if( prstatus_signal_found == true) {
+      for (std::vector<ThreadData>::iterator it = m_thread_data.begin();
----------------
Similar nit: `if (prstatus_signal_found)`.


https://reviews.llvm.org/D26676



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to