From 04eef6d20c0bb7f140f6fe3e7ceb8a985fcbbc9e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Herm=C3=A8s=20B=C3=A9lusca-Ma=C3=AFto?= Date: Sun, 26 Jul 2020 20:32:11 +0200 Subject: [PATCH] [CMD] parser.c, cmd.c/h : Code style and formatting fixes. - Change C_IFFAILURE, C_IFSUCCESS into C_OR, C_AND respectively. --- base/shell/cmd/cmd.c | 4 ++-- base/shell/cmd/cmd.h | 2 +- base/shell/cmd/parser.c | 18 ++++++++++++++---- 3 files changed, 17 insertions(+), 7 deletions(-) diff --git a/base/shell/cmd/cmd.c b/base/shell/cmd/cmd.c index bef99f272c4..2303f9d787a 100644 --- a/base/shell/cmd/cmd.c +++ b/base/shell/cmd/cmd.c @@ -801,7 +801,7 @@ ExecuteCommand( for (Sub = Cmd->Subcommands; Sub; Sub = Sub->Next) Ret = ExecuteCommand(Sub); break; - case C_IFFAILURE: + case C_OR: Sub = Cmd->Subcommands; Ret = ExecuteCommand(Sub); if (Ret != 0) @@ -810,7 +810,7 @@ ExecuteCommand( Ret = ExecuteCommand(Sub->Next); } break; - case C_IFSUCCESS: + case C_AND: Sub = Cmd->Subcommands; Ret = ExecuteCommand(Sub); if (Ret == 0) diff --git a/base/shell/cmd/cmd.h b/base/shell/cmd/cmd.h index 875ce5ec113..f0243414c14 100644 --- a/base/shell/cmd/cmd.h +++ b/base/shell/cmd/cmd.h @@ -301,7 +301,7 @@ INT cmd_move (LPTSTR); INT CommandMsgbox (LPTSTR); /* Prototypes from PARSER.C */ -enum { C_COMMAND, C_QUIET, C_BLOCK, C_MULTI, C_IFFAILURE, C_IFSUCCESS, C_PIPE, C_IF, C_FOR }; +enum { C_COMMAND, C_QUIET, C_BLOCK, C_MULTI, C_OR, C_AND, C_PIPE, C_IF, C_FOR }; typedef struct _PARSED_COMMAND { struct _PARSED_COMMAND *Subcommands; diff --git a/base/shell/cmd/parser.c b/base/shell/cmd/parser.c index abf2353d563..2a7b653be5d 100644 --- a/base/shell/cmd/parser.c +++ b/base/shell/cmd/parser.c @@ -823,8 +823,10 @@ EchoCommand(PARSED_COMMAND *Cmd) if (SubstituteForVars(Cmd->Command.Rest, Buf)) ConOutPrintf(_T("%s"), Buf); break; + case C_QUIET: return; + case C_BLOCK: ConOutChar(_T('(')); Sub = Cmd->Subcommands; @@ -846,15 +848,17 @@ EchoCommand(PARSED_COMMAND *Cmd) } ConOutChar(_T(')')); break; + case C_MULTI: - case C_IFFAILURE: - case C_IFSUCCESS: + case C_OR: + case C_AND: case C_PIPE: Sub = Cmd->Subcommands; EchoCommand(Sub); ConOutPrintf(_T(" %s "), OpString[Cmd->Type - C_OP_LOWEST]); EchoCommand(Sub->Next); break; + case C_IF: ConOutPrintf(_T("if")); if (Cmd->If.Flags & IFFLAG_IGNORECASE) @@ -874,6 +878,7 @@ EchoCommand(PARSED_COMMAND *Cmd) EchoCommand(Sub->Next); } break; + case C_FOR: ConOutPrintf(_T("for")); if (Cmd->For.Switches & FOR_DIRS) ConOutPrintf(_T(" /D")); @@ -947,10 +952,12 @@ do { \ if (!SubstituteForVars(Cmd->Command.Rest, Buf)) return NULL; STRING(Buf); break; + case C_QUIET: CHAR(_T('@')); RECURSE(Cmd->Subcommands); break; + case C_BLOCK: CHAR(_T('(')); for (Sub = Cmd->Subcommands; Sub; Sub = Sub->Next) @@ -961,15 +968,17 @@ do { \ } CHAR(_T(')')); break; + case C_MULTI: - case C_IFFAILURE: - case C_IFSUCCESS: + case C_OR: + case C_AND: case C_PIPE: Sub = Cmd->Subcommands; RECURSE(Sub); PRINTF(_T(" %s "), OpString[Cmd->Type - C_OP_LOWEST]); RECURSE(Sub->Next); break; + case C_IF: STRING(_T("if")); if (Cmd->If.Flags & IFFLAG_IGNORECASE) @@ -989,6 +998,7 @@ do { \ RECURSE(Sub->Next); } break; + case C_FOR: STRING(_T("for")); if (Cmd->For.Switches & FOR_DIRS) STRING(_T(" /D"));