Particle spawner: Fix missing parameters

The changes done to clientpackethandler.cpp and server.cpp move the
compatibility code into a protocol version check so that older code
parts can easily be removed in the future (e.g. bump of minimal version).

src/network/clientpackethandler.cpp

@@ -994,14 +994,22 @@ void Client::handleCommand_AddParticleSpawner(NetworkPacket* pkt)
 	p.amount             = readU16(is);
 	p.time               = readF32(is);
 
-	// older protocols do not support tweening, and send only
+	bool missing_end_values = false;
-	// static ranges, so we can't just use the normal serialization
+	if (m_proto_ver >= 42) {
-	// functions for the older values.
+		// All tweenable parameters
+		p.pos.deSerialize(is);
+		p.vel.deSerialize(is);
+		p.acc.deSerialize(is);
+		p.exptime.deSerialize(is);
+		p.size.deSerialize(is);
+	} else {
 		p.pos.start.legacyDeSerialize(is);
 		p.vel.start.legacyDeSerialize(is);
 		p.acc.start.legacyDeSerialize(is);
 		p.exptime.start.legacyDeSerialize(is);
 		p.size.start.legacyDeSerialize(is);
+		missing_end_values = true;
+	}
 
 	p.collisiondetection = readU8(is);
 	p.texture.string     = deSerializeString32(is);
@@ -1017,8 +1025,6 @@ void Client::handleCommand_AddParticleSpawner(NetworkPacket* pkt)
 	p.glow = readU8(is);
 	p.object_collision = readU8(is);
 
-	bool legacy_format = true;
-
 	// This is kinda awful
 	do {
 		u16 tmp_param0 = readU16(is);
@@ -1028,6 +1034,7 @@ void Client::handleCommand_AddParticleSpawner(NetworkPacket* pkt)
 		p.node.param2 = readU8(is);
 		p.node_tile   = readU8(is);
 
+		if (m_proto_ver < 42) {
 			// v >= 5.6.0
 			f32 tmp_sbias = readF32(is);
 			if (is.eof())
@@ -1048,6 +1055,10 @@ void Client::handleCommand_AddParticleSpawner(NetworkPacket* pkt)
 			p.exptime.end.deSerialize(is);
 			p.size.end.deSerialize(is);
 
+			missing_end_values = false;
+		}
+		// else: fields are already read by deSerialize() very early
+
 		// properties for legacy texture field
 		p.texture.deSerialize(is, m_proto_ver, true);
 
@@ -1077,11 +1088,9 @@ void Client::handleCommand_AddParticleSpawner(NetworkPacket* pkt)
 			newtex.deSerialize(is, m_proto_ver);
 			p.texpool.push_back(newtex);
 		}
-
-		legacy_format = false;
 	} while(0);
 
-	if (legacy_format) {
+	if (missing_end_values) {
 		// there's no tweening data to be had, so we need to set the
 		// legacy params to constant values, otherwise everything old
 		// will tween to zero

src/network/networkprotocol.h

@@ -213,6 +213,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 	PROTOCOL VERSION 42:
 		TOSERVER_UPDATE_CLIENT_INFO added
 		new fields for TOCLIENT_SET_LIGHTING and TOCLIENT_SET_SKY
+		Send forgotten TweenedParameter properties
 		[scheduled bump for 5.7.0]
 */
 
@@ -538,10 +539,22 @@ enum ToClientCommand
 
 	TOCLIENT_ADD_PARTICLESPAWNER = 0x47,
 	/*
-		-- struct range<T> { T min, T max, f32 bias };
+		using range<T> = RangedParameter<T> {
-		-- struct tween<T> { T start, T end };
+			T min, max
+			f32 bias
+		}
+		using tween<T> = TweenedParameter<T> {
+			u8 style
+			u16 reps
+			f32 beginning
+			T start, end
+		}
+
 		u16 amount
 		f32 spawntime
+		if PROTOCOL_VERSION >= 42 {
+			tween<T> pos, vel, acc, exptime, size
+		} else {
 			v3f minpos
 			v3f maxpos
 			v3f minvel
@@ -552,6 +565,7 @@ enum ToClientCommand
 			f32 maxexptime
 			f32 minsize
 			f32 maxsize
+		}
 		u8 bool collisiondetection
 		u32 len
 		u8[len] texture
@@ -562,6 +576,7 @@ enum ToClientCommand
 		u8 glow
 		u8 object_collision
 
+		if PROTOCOL_VERSION < 42 {
 			f32 pos_start_bias
 			f32 vel_start_bias
 			f32 acc_start_bias
@@ -574,6 +589,7 @@ enum ToClientCommand
 			--     f32 pos_end_bias
 			range<v3f> vel_end
 			range<v3f> acc_end
+		}
 
 		tween<range<v3f>> drag
 		-- i.e. v3f drag_start_min

src/server.cpp

@@ -1649,7 +1649,18 @@ void Server::SendAddParticleSpawner(session_t peer_id, u16 protocol_version,
 	NetworkPacket pkt(TOCLIENT_ADD_PARTICLESPAWNER, 100, peer_id);
 
 	pkt << p.amount << p.time;
-	{ // serialize legacy fields
+
+	if (protocol_version >= 42) {
+		// Serialize entire thing
+		std::ostringstream os(std::ios_base::binary);
+		p.pos.serialize(os);
+		p.vel.serialize(os);
+		p.acc.serialize(os);
+		p.exptime.serialize(os);
+		p.size.serialize(os);
+		pkt.putRawString(os.str());
+	} else {
+		// serialize legacy fields only (compatibility)
 		std::ostringstream os(std::ios_base::binary);
 		p.pos.start.legacySerialize(os);
 		p.vel.start.legacySerialize(os);
@@ -1672,6 +1683,8 @@ void Server::SendAddParticleSpawner(session_t peer_id, u16 protocol_version,
 	pkt << p.node.param0 << p.node.param2 << p.node_tile;
 
 	{ // serialize new fields
+		std::ostringstream os(std::ios_base::binary);
+		if (protocol_version < 42) {
 			// initial bias for older properties
 			pkt << p.pos.start.bias
 				<< p.vel.start.bias
@@ -1679,14 +1692,14 @@ void Server::SendAddParticleSpawner(session_t peer_id, u16 protocol_version,
 				<< p.exptime.start.bias
 				<< p.size.start.bias;
 
-		std::ostringstream os(std::ios_base::binary);
-
 			// final tween frames of older properties
 			p.pos.end.serialize(os);
 			p.vel.end.serialize(os);
 			p.acc.end.serialize(os);
 			p.exptime.end.serialize(os);
 			p.size.end.serialize(os);
+		}
+		// else: fields are already written by serialize() very early
 
 		// properties for legacy texture field
 		p.texture.serialize(os, protocol_version, true);