diff options
| author | Markus Uhlin <markus@nifty-networks.net> | 2025-04-08 19:21:57 +0200 |
|---|---|---|
| committer | Markus Uhlin <markus@nifty-networks.net> | 2025-04-08 19:21:57 +0200 |
| commit | 9b8c04bbdfa229d8e16272ccdc6c2f7ec1baffba (patch) | |
| tree | ee38013292167d4bba0f52ef1540ed80d3c69001 | |
| parent | 4d0cdff8c25cf6635991655a7e9b31a1271d4400 (diff) | |
Use of '>=' is better
| -rw-r--r-- | FICS/playerdb.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/FICS/playerdb.c b/FICS/playerdb.c index b2d1806..c85ec0c 100644 --- a/FICS/playerdb.c +++ b/FICS/playerdb.c @@ -587,17 +587,17 @@ ReadV1PlayerFmt(int p, player *pp, FILE *fp, char *file, int version) pp->timeOfReg = array[0]; pp->totalTime = array[1]; - if (pp->num_plan > MAX_PLAN) { + if (pp->num_plan >= MAX_PLAN) { warnx("Player %s is corrupt\nToo many plans (%d)", parray[p].name, pp->num_plan); return; - } else if (pp->num_formula > MAX_FORMULA) { + } else if (pp->num_formula >= MAX_FORMULA) { warnx("Player %s is corrupt\nToo many formulas (%d)", parray[p].name, pp->num_formula); return; - } else if (pp->numAlias > MAX_ALIASES) { + } else if (pp->numAlias >= MAX_ALIASES) { warnx("Player %s is corrupt\nToo many aliases (%d)", parray[p].name, pp->numAlias); @@ -859,7 +859,7 @@ got_attr_value_player(int p, char *attr, char *value, FILE *fp, char *file) * num_plan */ - if ((parray[p].num_plan = atoi(value)) > MAX_PLAN) { + if ((parray[p].num_plan = atoi(value)) >= MAX_PLAN) { warnx("%s: %s: too many plans (%d)", __func__, file, parray[p].num_plan); return -1; @@ -893,7 +893,7 @@ got_attr_value_player(int p, char *attr, char *value, FILE *fp, char *file) * num_formula */ - if ((parray[p].num_formula = atoi(value)) > MAX_FORMULA) { + if ((parray[p].num_formula = atoi(value)) >= MAX_FORMULA) { warnx("%s: %s: too many formulas (%d)", __func__, file, parray[p].num_formula); return -1; @@ -932,7 +932,7 @@ got_attr_value_player(int p, char *attr, char *value, FILE *fp, char *file) * num_alias */ - if ((parray[p].numAlias = atoi(value)) > MAX_ALIASES) { + if ((parray[p].numAlias = atoi(value)) >= MAX_ALIASES) { warnx("%s: %s: too many aliases (%d)", __func__, file, parray[p].numAlias); return -1; |