diff options
author | Markus Uhlin <markus@nifty-networks.net> | 2025-04-05 13:12:22 +0200 |
---|---|---|
committer | Markus Uhlin <markus@nifty-networks.net> | 2025-04-05 13:12:22 +0200 |
commit | c24bfddeae8d9997cdedf643b551577ff6af136c (patch) | |
tree | 252bb772348a7001480085978d8dcb5d54a68b72 /FICS | |
parent | 66fca5136293ae4e02d7ec7b387c3ba3ebf87d7b (diff) |
Do the assignment in the if
Diffstat (limited to 'FICS')
-rw-r--r-- | FICS/gamedb.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/FICS/gamedb.c b/FICS/gamedb.c index abb31ff..a21d61e 100644 --- a/FICS/gamedb.c +++ b/FICS/gamedb.c @@ -1067,15 +1067,15 @@ got_attr_value(int g, char *attr, char *value, FILE *fp, char *file) } else if (!strcmp(attr, "type:")) { garray[g].type = atoi(value); } else if (!strcmp(attr, "halfmoves:")) { - garray[g].numHalfMoves = atoi(value); - - if (garray[g].numHalfMoves == 0) + if ((garray[g].numHalfMoves = atoi(value)) == 0) return 0; else if (garray[g].numHalfMoves < 0 || (size_t)garray[g].numHalfMoves > INT_MAX / sizeof(move_t)) { warnx("%s: num half moves out-of-bounds (%d)", __func__, garray[g].numHalfMoves); return -1; + } else { + /* null */; } garray[g].moveListSize = garray[g].numHalfMoves; |