[Lldb-commits] [PATCH] D85243: Factor out common code from the iPhone/AppleTV/WatchOS simulator platform plugins. (NFC)

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere added inline comments.


================
Comment at: lldb/source/Plugins/Platform/MacOSX/PlatformAppleSimulator.cpp:85
+  if (!m_sdk.empty())
+    strm.Printf("  SDK Path: \"%s\"\n", m_sdk.str().c_str());
+  else
----------------
If you use `<<` you could drop the `.str().c_str()`. Alternatively, you could use `Format` which uses the `llvm::fmt` logic.


================
Comment at: lldb/source/Plugins/Platform/MacOSX/PlatformAppleSimulator.h:96
 protected:
+  const char *m_class_name;
+  const char *m_description;
----------------
"While you are here" could we make these StringRefs? 


================
Comment at: lldb/source/Plugins/Platform/MacOSX/PlatformAppleSimulator.h:122
+    lldb_private::Status
+
+    GetSymbolFile(const lldb_private::FileSpec &platform_file,
----------------
Weird formatting. Did you clang-format?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85243/new/

https://reviews.llvm.org/D85243