aboutsummaryrefslogtreecommitdiffstats
path: root/FICS/gamedb.c
diff options
context:
space:
mode:
authorMarkus Uhlin <markus@nifty-networks.net>2024-05-02 22:23:44 +0200
committerMarkus Uhlin <markus@nifty-networks.net>2024-05-02 22:23:44 +0200
commit0ac49149a76aa0edd46ca7a3012dd3bf76a5224e (patch)
tree426e79a8d0b13afbf68c6dc426dbf50006c4f236 /FICS/gamedb.c
parent30eb9928b036b0399de9c0f33017c7171807fdab (diff)
Revised RemoveHistGame()
Diffstat (limited to 'FICS/gamedb.c')
-rw-r--r--FICS/gamedb.c109
1 files changed, 59 insertions, 50 deletions
diff --git a/FICS/gamedb.c b/FICS/gamedb.c
index f0c3e1c..555bf8e 100644
--- a/FICS/gamedb.c
+++ b/FICS/gamedb.c
@@ -1280,65 +1280,74 @@ PRIVATE long OldestHistGame(char *login)
return 0L;
}
-PRIVATE void RemoveHistGame(char *file, int maxlines)
+PRIVATE void
+RemoveHistGame(char *file, int maxlines)
{
- FILE *fp;
- char GameFile[MAX_FILENAME_SIZE];
- char Opponent[MAX_LOGIN_NAME];
- char line[MAX_LINE_SIZE];
- long When, oppWhen;
- int count = 0;
-
- fp = fopen(file, "r");
- if (fp == NULL)
- return;
+ FILE *fp;
+ char GameFile[MAX_FILENAME_SIZE] = { '\0' };
+ char Opponent[MAX_LOGIN_NAME] = { '\0' };
+ char line[MAX_LINE_SIZE] = { '\0' };
+ int count = 0;
+ long int When, oppWhen;
+
+ if ((fp = fopen(file, "r")) == NULL)
+ return;
- fgets(line, MAX_LINE_SIZE - 1, fp);
- sscanf(line, "%*d %*c %*d %*c %*d %s %*s %*d %*d %*d %*d %*s %*s %ld",
- Opponent, &When);
- count++;
+ fgets(line, ARRAY_SIZE(line), fp);
+ sscanf(line, "%*d %*c %*d %*c %*d %s %*s %*d %*d %*d %*d %*s %*s %ld",
+ Opponent, &When);
+ count++;
- while (!feof(fp)) {
- fgets(line, MAX_LINE_SIZE - 1, fp);
- if (!feof(fp))
- count++;
- }
- fclose(fp);
+ while (!feof(fp)) {
+ fgets(line, ARRAY_SIZE(line), fp);
- stolower(Opponent);
- if (count > maxlines) {
- truncate_file(file, maxlines);
+ if (!feof(fp))
+ count++;
+ }
- oppWhen = OldestHistGame(Opponent);
- if (oppWhen > When || oppWhen <= 0L) {
- sprintf(GameFile, "%s/%ld/%ld", hist_dir, When % 100, When);
- unlink(GameFile);
- }
- }
+ fclose(fp);
+ stolower(Opponent);
+
+ if (count > maxlines) {
+ truncate_file(file, maxlines);
+ oppWhen = OldestHistGame(Opponent);
+
+ if (oppWhen > When || oppWhen <= 0L) {
+ msnprintf(GameFile, sizeof GameFile, "%s/%ld/%ld",
+ hist_dir, (When % 100), When);
+ unlink(GameFile);
+ }
+ }
}
-PUBLIC void RemHist(char *who)
+PUBLIC void
+RemHist(char *who)
{
- FILE *fp;
- char fName[MAX_FILENAME_SIZE];
- char Opp[MAX_LOGIN_NAME];
- long When, oppWhen;
+ FILE *fp;
+ char Opp[MAX_LOGIN_NAME] = { '\0' };
+ char fName[MAX_FILENAME_SIZE] = { '\0' };
+ long int When, oppWhen;
- sprintf(fName, "%s/player_data/%c/%s.%s", stats_dir,
- who[0], who, STATS_GAMES);
- fp = fopen(fName, "r");
- if (fp != NULL) {
- while (!feof(fp)) {
- fscanf(fp, "%*d %*c %*d %*c %*d %s %*s %*d %*d %*d %*d %*s %*s %ld",
- Opp, &When);
- stolower(Opp);
- oppWhen = OldestHistGame(Opp);
- if (oppWhen > When || oppWhen <= 0L) {
- sprintf(fName, "%s/%ld/%ld", hist_dir, When % 100, When);
- unlink(fName);
- }
- }
- }
+ msnprintf(fName, sizeof fName, "%s/player_data/%c/%s.%s", stats_dir,
+ who[0], who, STATS_GAMES);
+
+ fp = fopen(fName, "r");
+
+ if (fp != NULL) {
+ while (!feof(fp)) {
+ fscanf(fp, "%*d %*c %*d %*c %*d %s %*s %*d %*d %*d "
+ "%*d %*s %*s %ld", Opp, &When);
+
+ stolower(Opp);
+ oppWhen = OldestHistGame(Opp);
+
+ if (oppWhen > When || oppWhen <= 0L) {
+ msnprintf(fName, sizeof fName, "%s/%ld/%ld",
+ hist_dir, When % 100, When);
+ unlink(fName);
+ }
+ }
+ }
}
PRIVATE void