Skip to content

Commit 2f83f29

Browse files
committed
fixup! check PrettyPrintFunctionNameWithArgs return value
1 parent 3c2abf4 commit 2f83f29

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1746,11 +1746,12 @@ static bool PrintFunctionNameWithArgs(Stream &s,
17461746
variable_list_sp->AppendVariablesWithScope(eValueTypeVariableArgument,
17471747
args);
17481748

1749-
if (args.GetSize() > 0) {
1750-
PrettyPrintFunctionNameWithArgs(s, cstr, exe_scope, args);
1751-
} else {
1752-
s.PutCString(cstr);
1753-
}
1749+
if (args.GetSize() > 0)
1750+
return PrettyPrintFunctionNameWithArgs(s, cstr, exe_scope, args);
1751+
1752+
// FIXME: can we just unconditionally call PrettyPrintFunctionNameWithArgs?
1753+
// It should be able to handle the "no arguments" case.
1754+
s.PutCString(cstr);
17541755

17551756
return true;
17561757
}

0 commit comments

Comments
 (0)