Author: teemperor Date: Sun Aug 5 17:04:51 2018 New Revision: 338976 URL: http://llvm.org/viewvc/llvm-project?rev=338976&view=rev Log: Remove duplicated code in CommandObjectQuit
Summary: We already have the same check directly before, so this code can never be reached (as seen in the test coverage). Subscribers: lldb-commits Differential Revision: https://reviews.llvm.org/D50317 Modified: lldb/trunk/source/Commands/CommandObjectQuit.cpp Modified: lldb/trunk/source/Commands/CommandObjectQuit.cpp URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Commands/CommandObjectQuit.cpp?rev=338976&r1=338975&r2=338976&view=diff ============================================================================== --- lldb/trunk/source/Commands/CommandObjectQuit.cpp (original) +++ lldb/trunk/source/Commands/CommandObjectQuit.cpp Sun Aug 5 17:04:51 2018 @@ -86,13 +86,6 @@ bool CommandObjectQuit::DoExecute(Args & return false; } - if (command.GetArgumentCount() > 1) { - result.AppendError("Too many arguments for 'quit'. Only an optional exit " - "code is allowed"); - result.SetStatus(eReturnStatusFailed); - return false; - } - // We parse the exit code argument if there is one. if (command.GetArgumentCount() == 1) { llvm::StringRef arg = command.GetArgumentAtIndex(0); _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits