================
@@ -1304,9 +1304,11 @@ class CommandObjectProcessSaveCore : public 
CommandObjectParsed {
         FileSpec output_file(command.GetArgumentAtIndex(0));
         FileSystem::Instance().Resolve(output_file);
         SaveCoreStyle corefile_style = m_options.m_requested_save_core_style;
-        Status error =
-            PluginManager::SaveCore(process_sp, output_file, corefile_style,
-                                    m_options.m_requested_plugin_name);
+        CoreDumpOptions core_dump_options(output_file);
+        core_dump_options.SetCoreDumpPluginName(
+            m_options.m_requested_plugin_name);
+        core_dump_options.SetCoreDumpStyle(corefile_style);
+        Status error = PluginManager::SaveCore(process_sp, core_dump_options);
----------------
clayborg wrote:

We need to remove the instance variables `m_requested_plugin_name` and 
`m_requested_save_core_style` and replace with with 
`lldb_private::CoreDumpOptions m_options;`. Then in the code that used to fill 
in the old ivars, we will just call the accessor instead. We can also call the 
accessor to set the output file as well on the `m_options;` ivar.

https://github.com/llvm/llvm-project/pull/98403
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to