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
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
