diff options
| author | Ulrich Sibiller <uli42@gmx.de> | 2020-08-03 16:40:49 +0200 |
|---|---|---|
| committer | Ulrich Sibiller <uli42@gmx.de> | 2021-01-15 19:50:35 +0100 |
| commit | 5735e164e9625137c7d3209428eafab33300df94 (patch) | |
| tree | f892ff7b5d166751ddeab2a1ac1dfd98b83e3325 /nx-X11 | |
| parent | 28fef3bc06e9325202920fc21f42d2506e8f2b2e (diff) | |
| download | nx-libs-5735e164e9625137c7d3209428eafab33300df94.tar.gz nx-libs-5735e164e9625137c7d3209428eafab33300df94.tar.bz2 nx-libs-5735e164e9625137c7d3209428eafab33300df94.zip | |
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."
Diffstat (limited to 'nx-X11')
| -rw-r--r-- | nx-X11/programs/Xserver/hw/nxagent/Colormap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/nx-X11/programs/Xserver/hw/nxagent/Colormap.c b/nx-X11/programs/Xserver/hw/nxagent/Colormap.c index 099c76af4..c173f322c 100644 --- 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); } |