Skip to content

Commit 50b1276

Browse files
committed
[lldb] Qualify auto in the lldb driver (NFC)
The LLVM coding guidelines say to "[u]se auto & for values and auto * for pointers unless you need to make a copy."
1 parent e994c5d commit 50b1276

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

lldb/tools/driver/Driver.cpp

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) {
206206
}
207207

208208
if (auto *arg = args.getLastArg(OPT_core)) {
209-
auto arg_value = arg->getValue();
209+
auto *arg_value = arg->getValue();
210210
SBFileSpec file(arg_value);
211211
if (!file.Exists()) {
212212
error.SetErrorStringWithFormat(
@@ -232,7 +232,7 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) {
232232
}
233233

234234
if (auto *arg = args.getLastArg(OPT_file)) {
235-
auto arg_value = arg->getValue();
235+
auto *arg_value = arg->getValue();
236236
SBFileSpec file(arg_value);
237237
if (file.Exists()) {
238238
m_option_data.m_args.emplace_back(arg_value);
@@ -249,7 +249,7 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) {
249249
}
250250

251251
if (auto *arg = args.getLastArg(OPT_arch)) {
252-
auto arg_value = arg->getValue();
252+
auto *arg_value = arg->getValue();
253253
if (!lldb::SBDebugger::SetDefaultArchitecture(arg_value)) {
254254
error.SetErrorStringWithFormat(
255255
"invalid architecture in the -a or --arch option: '%s'", arg_value);
@@ -258,7 +258,7 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) {
258258
}
259259

260260
if (auto *arg = args.getLastArg(OPT_script_language)) {
261-
auto arg_value = arg->getValue();
261+
auto *arg_value = arg->getValue();
262262
m_debugger.SetScriptLanguage(m_debugger.GetScriptingLanguage(arg_value));
263263
}
264264

@@ -267,7 +267,7 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) {
267267
}
268268

269269
if (auto *arg = args.getLastArg(OPT_attach_name)) {
270-
auto arg_value = arg->getValue();
270+
auto *arg_value = arg->getValue();
271271
m_option_data.m_process_name = arg_value;
272272
}
273273

@@ -276,7 +276,7 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) {
276276
}
277277

278278
if (auto *arg = args.getLastArg(OPT_attach_pid)) {
279-
auto arg_value = arg->getValue();
279+
auto *arg_value = arg->getValue();
280280
char *remainder;
281281
m_option_data.m_process_pid = strtol(arg_value, &remainder, 0);
282282
if (remainder == arg_value || *remainder != '\0') {
@@ -287,7 +287,7 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) {
287287
}
288288

289289
if (auto *arg = args.getLastArg(OPT_repl_language)) {
290-
auto arg_value = arg->getValue();
290+
auto *arg_value = arg->getValue();
291291
m_option_data.m_repl_lang =
292292
SBLanguageRuntime::GetLanguageTypeFromString(arg_value);
293293
if (m_option_data.m_repl_lang == eLanguageTypeUnknown) {
@@ -304,7 +304,7 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) {
304304

305305
if (auto *arg = args.getLastArg(OPT_repl_)) {
306306
m_option_data.m_repl = true;
307-
if (auto arg_value = arg->getValue())
307+
if (auto *arg_value = arg->getValue())
308308
m_option_data.m_repl_options = arg_value;
309309
}
310310

@@ -313,7 +313,7 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) {
313313
for (auto *arg : args.filtered(OPT_source_on_crash, OPT_one_line_on_crash,
314314
OPT_source, OPT_source_before_file,
315315
OPT_one_line, OPT_one_line_before_file)) {
316-
auto arg_value = arg->getValue();
316+
auto *arg_value = arg->getValue();
317317
if (arg->getOption().matches(OPT_source_on_crash)) {
318318
m_option_data.AddInitialCommand(arg_value, eCommandPlacementAfterCrash,
319319
true, error);
@@ -365,7 +365,7 @@ SBError Driver::ProcessArgs(const opt::InputArgList &args, bool &exiting) {
365365

366366
// Any argument following -- is an argument for the inferior.
367367
if (auto *arg = args.getLastArgNoClaim(OPT_REM)) {
368-
for (auto value : arg->getValues())
368+
for (auto *value : arg->getValues())
369369
m_option_data.m_args.emplace_back(value);
370370
}
371371
} else if (args.getLastArgNoClaim() != nullptr) {

0 commit comments

Comments
 (0)