[Lldb-commits] [PATCH] D15152: Add a new option to Platform::LoadImage to install the image

[Greg Clayton via lldb-commits]

clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.

Add header doc for the new LoadImage and make Platform::LoadImage virtual in case platforms want to override this and this is good to go.


================
Comment at: include/lldb/API/SBProcess.h:301-307
@@ +300,9 @@
+
+    // Load an image to the currently running process.
+    // If both local and remote image is specified then copy the local image to the remote image and
+    // then load it from thew remote path.
+    // If only the local image is specified then the image will be copied to the current working
+    // directory and opened from there.
+    // If only the remote image is specified then open the image from the target (no compying done
+    // in this case)
+    uint32_t
----------------
Add correct header doc for this if we are going to add documentation to the header file.

================
Comment at: include/lldb/Target/Platform.h:1023-1027
@@ -1013,6 +1022,7 @@
         //------------------------------------------------------------------
-        virtual uint32_t
+        uint32_t
         LoadImage (lldb_private::Process* process,
-                   const lldb_private::FileSpec& image_spec,
+                   const lldb_private::FileSpec& local_file,
+                   const lldb_private::FileSpec& remote_file,
                    lldb_private::Error& error);
 
----------------
This can still be virtual just in case other platforms want to do something completely different that what is supported in the default implementation.


http://reviews.llvm.org/D15152