diff options
| author | Markus Uhlin <markus@nifty-networks.net> | 2025-03-09 00:01:15 +0100 |
|---|---|---|
| committer | Markus Uhlin <markus@nifty-networks.net> | 2025-03-09 00:01:15 +0100 |
| commit | 2de83a811647603f59b3d4808979ec0c52e8d3f2 (patch) | |
| tree | 1ce1acd5b637fb4cbbb40ff5f75b34723329e77c /FICS/gameproc.c | |
| parent | 22e493e5c085f81f23376f3522f88bf082c6948f (diff) | |
CheckRepetition: calc string length once (found by PVS-Studio)
Diffstat (limited to 'FICS/gameproc.c')
| -rw-r--r-- | FICS/gameproc.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/FICS/gameproc.c b/FICS/gameproc.c index b4a538e..1af99de 100644 --- a/FICS/gameproc.c +++ b/FICS/gameproc.c @@ -855,11 +855,16 @@ CheckRepetition(int p, int g) for (move_num = garray[g].game_state.lastIrreversable; move_num < garray[g].numHalfMoves - 1; move_num++) { + size_t len[3]; + pos = GetFENpos(g, move_num); + len[0] = strlen(pos1); + len[1] = strlen(pos2); + len[2] = strlen(pos); - if (strlen(pos1) == strlen(pos) && !strcmp(pos1, pos)) + if (len[0] == len[2] && !strcmp(pos1, pos)) flag1++; - if (strlen(pos2) == strlen(pos) && !strcmp(pos2, pos)) + if (len[1] == len[2] && !strcmp(pos2, pos)) flag2++; } |