Colormap.c: remove unneccessary check

success would always be true at the beginning of the loop as it is part of the loop condition. Finding from PVS Studio: "V560 A part of conditional expression is always true: success."

Colormap.c: remove unneccessary check
5735e164 · Ulrich Sibiller · 28fef3bc · 5735e164
Commit 5735e164 authored Aug 03, 2020 by Ulrich Sibiller
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

Colormap.c nx-X11/programs/Xserver/hw/nxagent/Colormap.c +1 -1

No files found.
--- a/nx-X11/programs/Xserver/hw/nxagent/Colormap.c
+++ b/nx-X11/programs/Xserver/hw/nxagent/Colormap.c
@@ -573,7 +573,7 @@ Bool nxagentReconnectAllColormap(void *p0)

  for (int cid = 0; (cid < MAXCLIENTS) && success; cid++)
  {
-    if (clients[cid] && success)
+    if (clients[cid])
    {
      FindClientResourcesByType(clients[cid], RT_COLORMAP, nxagentReconnectColormap, &success);
    }