flying ships (#1048)

* attempt fallback for bad encoding headers

* Fix levitation duration

Ship flag was persisted and somehow never cleared.
Resolves https://bugs.eressea.de/view.php?id=2997

scripts/tests/e2/e2features.lua

@@ -253,8 +253,9 @@ function test_levitate()
   u:add_order("ARBEITE")
   levitate_ship(u.ship, u, 2, 1)
   assert_equal(32, u.ship.flags)
+  assert_not_nil(u.ship:get_curse('flyingship'))
   process_orders()
-  assert_equal(0, u.ship.flags)
+  assert_nil(u.ship:get_curse('flyingship'))
 end
 
 function test_levitation_ring()

src/kernel/save.c

@@ -1468,6 +1468,7 @@ ship *read_ship(gamedata *data)
 
     if (data->version >= FOSS_VERSION) {
         READ_INT(store, &sh->flags);
+        sh->flags &= SFL_SAVEMASK;
     }
 
     /* Attribute rekursiv einlesen */

src/kernel/ship.h

@@ -51,7 +51,7 @@ typedef struct ship_type {
 #define SF_FISHING (1<<4)         /* was on an ocean, can fish */
 #define SF_FLYING  (1<<5)         /* the ship can fly */
 
-#define SFL_SAVEMASK (SF_FLYING)
+#define SFL_SAVEMASK 0
 #define INCOME_FISHING 10
 
 typedef struct ship {

src/spells/flyingship.c

@@ -103,27 +103,13 @@ static int flyingship_read(gamedata * data, curse * c, void *target)
 {
     ship *sh = (ship *)target;
     c->data.v = sh;
-    if (data->version < FOSS_VERSION) {
-        sh->flags |= SF_FLYING;
-        return 0;
-    }
-    assert(sh->flags & SF_FLYING);
-    return 0;
-}
-
-static int flyingship_age(curse * c)
-{
-    ship *sh = (ship *)c->data.v;
-    if (sh && c->duration == 1) {
-        sh->flags &= ~SF_FLYING;
-        return 1;
-    }
+    sh->flags |= SF_FLYING;
     return 0;
 }
 
 const struct curse_type ct_flyingship = { "flyingship",
 CURSETYP_NORM, 0, NO_MERGE, cinfo_ship, NULL, flyingship_read,
-NULL, NULL, flyingship_age
+NULL, NULL, NULL
 };
 
 void register_flyingship(void)