cmd: Simplify boolean expressions (PVS-Studio)

Frédéric Delanoy frederic.delanoy at gmail.com
Tue Oct 13 18:58:38 CDT 2015


Signed-off-by: Frédéric Delanoy <frederic.delanoy at gmail.com>
---
 programs/cmd/builtins.c | 2 +-
 programs/cmd/wcmdmain.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/programs/cmd/builtins.c b/programs/cmd/builtins.c
index 5661c69..e9b9ddd 100644
--- a/programs/cmd/builtins.c
+++ b/programs/cmd/builtins.c
@@ -1005,7 +1005,7 @@ void WCMD_copy(WCHAR * args) {
       if (!srcisdevice) FindClose (hff);
     } else {
       /* Error if the first file was not found */
-      if (!anyconcats || (anyconcats && !writtenoneconcat)) {
+      if (!anyconcats || !writtenoneconcat) {
         WCMD_print_error ();
         errorlevel = 1;
       }
diff --git a/programs/cmd/wcmdmain.c b/programs/cmd/wcmdmain.c
index 7666329..ba7d9ca 100644
--- a/programs/cmd/wcmdmain.c
+++ b/programs/cmd/wcmdmain.c
@@ -879,7 +879,7 @@ static void handleExpansion(WCHAR *cmd, BOOL atExecute, BOOL delayed) {
       if (startchar == '%' && forvaridx != -1 && forloopcontext.variable[forvaridx]) {
         /* Replace the 2 characters, % and for variable character */
         WCMD_strsubstW(p, p + 2, forloopcontext.variable[forvaridx], -1);
-      } else if (!atExecute || (atExecute && startchar == '!')) {
+      } else if (!atExecute || startchar == '!') {
         p = WCMD_expand_envvar(p, startchar);
 
       /* In a FOR loop, see if this is the variable to replace */
-- 
2.6.1




More information about the wine-patches mailing list