Skip to content

Conversation

@tambry
Copy link
Contributor

@tambry tambry commented Oct 21, 2025

These all have member initializers of the same value so they're redundant.

Fixes: 47b9aad

These all have member initializers of the same value so they're redundant.

Fixes: 47b9aad
@tambry tambry self-assigned this Oct 21, 2025
@tambry tambry requested a review from JDevlieghere as a code owner October 21, 2025 16:02
@llvmbot llvmbot added the lldb label Oct 21, 2025
@llvmbot
Copy link
Member

llvmbot commented Oct 21, 2025

@llvm/pr-subscribers-lldb

Author: Raul Tambre (tambry)

Changes

These all have member initializers of the same value so they're redundant.

Fixes: 47b9aad


Full diff: https://github.com/llvm/llvm-project/pull/164451.diff

1 Files Affected:

  • (modified) lldb/include/lldb/Target/Process.h (+1-4)
diff --git a/lldb/include/lldb/Target/Process.h b/lldb/include/lldb/Target/Process.h
index dc75d98acea70..8289eaf3691cc 100644
--- a/lldb/include/lldb/Target/Process.h
+++ b/lldb/include/lldb/Target/Process.h
@@ -127,10 +127,7 @@ class ProcessAttachInfo : public ProcessInstanceInfo {
 public:
   ProcessAttachInfo() = default;
 
-  ProcessAttachInfo(const ProcessLaunchInfo &launch_info)
-      : m_resume_count(0), m_wait_for_launch(false), m_ignore_existing(true),
-        m_continue_once_attached(false), m_detach_on_error(true),
-        m_async(false) {
+  ProcessAttachInfo(const ProcessLaunchInfo &launch_info) {
     ProcessInfo::operator=(launch_info);
     SetProcessPluginName(launch_info.GetProcessPluginName());
     SetResumeCount(launch_info.GetResumeCount());

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants