[Lldb-commits] [PATCH] D55472: Speadup memory regions handling and cleanup related code
Greg Clayton via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Dec 19 09:25:01 PST 2018
clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.
One quick change required to avoid STL in public headers.
Comment at: include/lldb/lldb-forward.h:16
Remove due to this header being part of the public API.
Comment at: include/lldb/lldb-forward.h:136
+typedef std::vector<lldb_private::MemoryRegionInfo> MemoryRegionInfos;
After thinking about this a bit, we shouldn't put any STL stuff in headers that are part of the public API as we never want to expose any STL through our API. Can probably move this into MemoryRegionInfo.h, or just use "std::vector<lldb_private::MemoryRegionInfo>" everywhere as long as it is in lldb_private or the global namespace (we only export "lldb::*").
CHANGES SINCE LAST ACTION
More information about the lldb-commits