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