https://git.reactos.org/?p=reactos.git;a=commitdiff;h=0db5fce47d74655c0b6896...
commit 0db5fce47d74655c0b6896ce402b22e7cbf22d71 Author: Hermès Bélusca-Maïto hermes.belusca-maito@reactos.org AuthorDate: Wed Dec 29 03:17:57 2021 +0100 Commit: Hermès Bélusca-Maïto hermes.belusca-maito@reactos.org CommitDate: Wed Dec 29 04:02:35 2021 +0100
[ATTRIB] Fix bFound assignment, as it was originally intended by the PR reviewer.
See https://github.com/reactos/reactos/pull/2142#discussion_r360645296
This fixes the two warnings: warning C4552: '!=': result of expression not used --- base/applications/cmdutils/attrib/attrib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/base/applications/cmdutils/attrib/attrib.c b/base/applications/cmdutils/attrib/attrib.c index fc53e68dfbc..ff6185a84ea 100644 --- a/base/applications/cmdutils/attrib/attrib.c +++ b/base/applications/cmdutils/attrib/attrib.c @@ -139,7 +139,7 @@ PrintAttribute(
wcscpy(pszFileName, findData.cFileName); wcscat(pszFileName, L"\"); - bFound != PrintAttribute(szFullName, pszFile, bRecurse, bDirectories); + bFound |= PrintAttribute(szFullName, pszFile, bRecurse, bDirectories); } while (FindNextFileW(hFind, &findData)); FindClose(hFind); @@ -243,7 +243,7 @@ ChangeAttribute(
wcscpy(pszFileName, findData.cFileName); wcscat(pszFileName, L"\"); - bFound != ChangeAttribute(szFullName, pszFile, bRecurse, bDirectories, + bFound |= ChangeAttribute(szFullName, pszFile, bRecurse, bDirectories, dwMask, dwAttrib); } while (FindNextFileW(hFind, &findData));