From: Laurent Mazet <mazet@softndesign.org>
Date: Fri, 14 Jun 2024 21:55:37 +0000 (+0200)
Subject: cleaning
X-Git-Tag: v1.0~10
X-Git-Url: https://secure.softndesign.org/git/?a=commitdiff_plain;h=dedc81e78752ba6a8dc0f9f680fdd9facba301ef;p=scrabble.git

cleaning
---

diff --git a/function.c b/function.c
index d121f1d..4ecf3e1 100644
--- a/function.c
+++ b/function.c
@@ -555,15 +555,16 @@ char *savedata (int nbplayers, char *lang, char *board, play_t *play, draw_t *dr
     CHECKALLOC (buffer);
 
     int l = sprintf (buffer, "nbplayers: %d\n", nbplayers);
-    l += sprintf (buffer + l, "lang: %s\n", lang);
-    l += sprintf (buffer + l, "board: %s\n", board);
-    l += sprintf (buffer + l, "play: %s\n", play->tab);
-    l += sprintf (buffer + l, "draw: %s\n", draw->tiles);
+    l += sprintf (buffer + l, "lang: \"%s\"\n", lang);
+    l += sprintf (buffer + l, "board: \"%s\"\n", board);
+    l += sprintf (buffer + l, "play: \"%s\"\n", play->tab);
+    l += sprintf (buffer + l, "nbtiles: %d\n", draw->nbtiles);
+    l += sprintf (buffer + l, "tiles: \"%s\"\n", draw->tiles);
     for (i = 0; i < nbplayers; i++) {
-        l += sprintf (buffer + l, "game[%d]: %s\n", i, game[i]->tiles);
+        l += sprintf (buffer + l, "game[%d]: \"%s\"\n", i, game[i]->tiles);
     }
-    l += sprintf (buffer + l, "n: %d\n", n);
-    l += sprintf (buffer + l, "turn[%d]: %s\n", n, turn[n]->tab);
+    l += sprintf (buffer + l, "current: %d\n", n);
+    l += sprintf (buffer + l, "turn[%d]: \"%s\"\n", n, turn[n]->tab);
     for (i = 0; i < nbplayers; i++) {
         l += sprintf (buffer + l, "score[%d]: %d\n", i, score[i]);
     }
@@ -576,9 +577,9 @@ int writedata (char *filename, char *data)
     int ret = 0;
     FILE *fd = fopen (filename, "w");
     if (fd) {
-        fwrite (data, strlen (data), 1, fd);
+        size_t n = fwrite (data, strlen (data), 1, fd);
         fclose (fd);
-        ret = 1;
+        ret = (n == strlen (data));
     }
     return ret;
 }
diff --git a/scrabble.c b/scrabble.c
index e10e942..25f2cb7 100644
--- a/scrabble.c
+++ b/scrabble.c
@@ -233,6 +233,7 @@ int main (int argc, char *argv[])
     drawwindow (game[n], score[n], xdrawwin[n], ydrawwin[n], 1);
     int nbpoints = 0;
     int stop = 0;
+    char *data = NULL;
     while (!stop) {
         char highlight = current = gessletter (play, x, y);
         if ((current == ' ') && (letter != '\0')) {
@@ -293,11 +294,9 @@ int main (int argc, char *argv[])
             stop = 1;
             break;
         case 's':
-            {
-                char *data = savedata (nbplayers, language, boardname, play, draw, game, n, turn, score);
-                writedata ("data.scr", data);
-                free (data);
-            }
+            data = savedata (nbplayers, language, boardname, play, draw, game, n, turn, score);
+            writedata ("data.scr", data);
+            free (data);
             break;
         case 't':
             mode ^= 1;