Fix unnecessary network protocol incompatibility in ContentFeatures
authorPerttu Ahola <celeron55@gmail.com>
Thu, 8 Nov 2012 18:43:32 +0000 (20:43 +0200)
committerPerttu Ahola <celeron55@gmail.com>
Thu, 8 Nov 2012 18:43:32 +0000 (20:43 +0200)
src/nodedef.cpp

index 466531efa8fa07a03fdaa14d1288d7001f90b0b5..180219ba854f1c0dd8a17b732d74994c4958666c 100644 (file)
@@ -254,7 +254,6 @@ void ContentFeatures::serialize(std::ostream &os)
        os<<serializeString(liquid_alternative_flowing);
        os<<serializeString(liquid_alternative_source);
        writeU8(os, liquid_viscosity);
-       writeU8(os, liquid_renewable);
        writeU8(os, light_source);
        writeU32(os, damage_per_second);
        node_box.serialize(os);
@@ -264,6 +263,9 @@ void ContentFeatures::serialize(std::ostream &os)
        serializeSimpleSoundSpec(sound_footstep, os);
        serializeSimpleSoundSpec(sound_dig, os);
        serializeSimpleSoundSpec(sound_dug, os);
+       // Stuff below should be moved to correct place in a version that otherwise changes
+       // the protocol version
+       writeU8(os, liquid_renewable);
 }
 
 void ContentFeatures::deSerialize(std::istream &is)
@@ -309,7 +311,6 @@ void ContentFeatures::deSerialize(std::istream &is)
        liquid_alternative_flowing = deSerializeString(is);
        liquid_alternative_source = deSerializeString(is);
        liquid_viscosity = readU8(is);
-       liquid_renewable = readU8(is);
        light_source = readU8(is);
        damage_per_second = readU32(is);
        node_box.deSerialize(is);
@@ -322,6 +323,9 @@ void ContentFeatures::deSerialize(std::istream &is)
        // If you add anything here, insert it primarily inside the try-catch
        // block to not need to increase the version.
        try{
+               // Stuff below should be moved to correct place in a version that
+               // otherwise changes the protocol version
+               liquid_renewable = readU8(is);
        }catch(SerializationError &e) {};
 }