Skip to content

Commit ef976cc

Browse files
committed
Set success on plugin commands
1 parent 1f863a6 commit ef976cc

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

lldb/source/Commands/CommandObjectPlugin.cpp

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,7 @@ List only the plugin 'foo' matching a fully qualified name exactly
165165
return;
166166
}
167167
llvm::StringRef pattern = argc ? command[0].ref() : "";
168+
result.SetStatus(eReturnStatusSuccessFinishResult);
168169

169170
int num_matching = ActOnMatchingPlugins(
170171
pattern, [&](const PluginNamespace &plugin_namespace,
@@ -177,9 +178,8 @@ List only the plugin 'foo' matching a fully qualified name exactly
177178
}
178179
});
179180

180-
if (num_matching == 0) {
181+
if (num_matching == 0)
181182
result.AppendErrorWithFormat("Found no matching plugins");
182-
}
183183
}
184184
};
185185

@@ -201,12 +201,12 @@ class CommandObjectPluginEnable : public CommandObjectParsed {
201201
return;
202202
}
203203
llvm::StringRef pattern = argc ? command[0].ref() : "";
204+
result.SetStatus(eReturnStatusSuccessFinishResult);
204205

205206
int num_matching = SetEnableOnMatchingPlugins(pattern, result, true);
206207

207-
if (num_matching == 0) {
208+
if (num_matching == 0)
208209
result.AppendErrorWithFormat("Found no matching plugins to enable");
209-
}
210210
}
211211
};
212212

@@ -228,6 +228,7 @@ class CommandObjectPluginDisable : public CommandObjectParsed {
228228
return;
229229
}
230230
llvm::StringRef pattern = argc ? command[0].ref() : "";
231+
result.SetStatus(eReturnStatusSuccessFinishResult);
231232

232233
int num_matching = SetEnableOnMatchingPlugins(pattern, result, false);
233234

0 commit comments

Comments
 (0)