[Lldb-commits] [PATCH] D25569: Minidump plugin: functions parsing memory structures and filtering module list

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 17 02:26:11 PDT 2016

labath added inline comments.

Comment at: source/Plugins/Process/minidump/MinidumpParser.cpp:15
+#include "lldb/Target/MemoryRegionInfo.h"
+#include "lldb/Utility/LLDBAssert.h"
I think this is not needed anymore.

Comment at: source/Plugins/Process/minidump/MinidumpParser.cpp:252
+    bool exists;
+    std::tie(iter, exists) = lowest_addr.emplace(std::make_pair(
+        module_name, std::make_pair(module.base_of_image, &module)));
The reason I suggested `emplace` instead of `insert` was that you could strip one level of `std::make_pair` here.
std::tie(iter, exists) = lowest_addr.emplace( module_name, std::make_pair(module.base_of_image, &module));


More information about the lldb-commits mailing list