home · contact · privacy
In save_game(), error-exit on earliest of writing errors, not after cascade of them.
authorChristian Heller <c.heller@plomlompom.de>
Wed, 21 Aug 2013 03:14:20 +0000 (05:14 +0200)
committerChristian Heller <c.heller@plomlompom.de>
Wed, 21 Aug 2013 03:14:20 +0000 (05:14 +0200)
src/misc.c

index 9063fefac812037624e23e2779c6c015aeb0ecbd..1edd7d2ed56068ac594671a4440ede252e6e2bad 100644 (file)
@@ -136,16 +136,17 @@ extern void save_game(struct World * world)
     FILE * file = fopen("savefile", "w");
     exit_err(0 == file, world,
              "Error saving game: Unable to open savefile for writing.");
-    uint8_t err;
-    err = write_uint32_bigendian(world->seed, file);
-    err = err | write_uint32_bigendian(world->turn, file);
-    err = err | write_uint16_bigendian(world->player->pos.y + 1, file);
-    err = err | write_uint16_bigendian(world->player->pos.x + 1, file);
-    err = err | write_uint8(world->player->hitpoints, file);
-    err = err | write_map_objects(world, world->monster, file);
-    err = err | write_map_objects(world, world->item, file);
-    exit_err(err, world,
-             "Error saving game: Trouble writing to opened savefile.");
+    if (   write_uint32_bigendian(world->seed, file)
+        || write_uint32_bigendian(world->turn, file)
+        || write_uint16_bigendian(world->player->pos.y + 1, file)
+        || write_uint16_bigendian(world->player->pos.x + 1, file)
+        || write_uint8(world->player->hitpoints, file)
+        || write_map_objects(world, world->monster, file)
+        || write_map_objects(world, world->item, file))
+    {
+        exit_err(1, world,
+                 "Error saving game: Trouble writing to opened savefile.");
+    }
     exit_err(fclose(file), world,
              "Error saving game: Unable to close opened savefile.");
 }