fixed crack animation timing in client

src/client.cpp

@@ -176,6 +176,7 @@ void Client::step(float dtime)
 		if(dr != m_env.getDayNightRatio())
 		{
 			//dstream<<"dr="<<dr<<std::endl;
+			dout_client<<DTIME<<"Client: changing day-night ratio"<<std::endl;
 			m_env.setDayNightRatio(dr);
 			m_env.expireMeshes(true);
 		}
@@ -1038,6 +1039,9 @@ bool Client::AsyncProcessPacket(LazyMeshUpdater &mesh_updater)
 		
 		//TimeTaker t1("TOCLIENT_REMOVENODE", g_device);
 		
+		// This will clear the cracking animation after digging
+		((ClientMap&)m_env.getMap()).clearTempMod(p);
+
 		core::map<v3s16, MapBlock*> modified_blocks;
 
 		try

src/client.h

@@ -252,6 +252,15 @@ public:
 		m_env.getMap().updateMeshes(blockpos, m_env.getDayNightRatio());
 	}
 
+	float getAvgRtt()
+	{
+		JMutexAutoLock lock(m_con_mutex);
+		con::Peer *peer = m_con.GetPeerNoEx(PEER_ID_SERVER);
+		if(peer == NULL)
+			return 0.0;
+		return peer->avg_rtt;
+	}
+	
 private:
 	
 	// Virtual methods from con::PeerHandler

src/constants.h

@@ -79,5 +79,8 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 
 #define WATER_LEVEL (0)
 
+// Length of cracking animation in count of images
+#define CRACK_ANIMATION_LENGTH 4
+
 #endif
 

src/main.cpp

@@ -2114,6 +2114,7 @@ int main(int argc, char *argv[])
 			static v3s16 nodepos_old(-32768,-32768,-32768);
 
 			static float dig_time = 0.0;
+			static u16 dig_index = 0;
 
 			if(nodepos != nodepos_old)
 			{
@@ -2146,19 +2147,25 @@ int main(int argc, char *argv[])
 			}
 			if(g_input->getLeftState())
 			{
-				dig_time += dtime;
-				
 				float dig_time_complete = 0.5;
 				MapNode n = client.getNode(nodepos);
 				if(n.d == CONTENT_STONE)
 					dig_time_complete = 1.5;
 				
-				u16 dig_index = (u16)(3.99*dig_time/dig_time_complete);
+				float dig_time_complete0 = dig_time_complete+client.getAvgRtt()*2;
-				if(dig_time > 0.125)
+				if(dig_time_complete0 < 0.0)
+					dig_time_complete0 = 0.0;
+
+				dig_index = (u16)((float)CRACK_ANIMATION_LENGTH
+						* dig_time/dig_time_complete0);
+
+				if(dig_time > 0.125 && dig_index < CRACK_ANIMATION_LENGTH)
 				{
 					//dstream<<"dig_index="<<dig_index<<std::endl;
 					client.setTempMod(nodepos, NodeMod(NODEMOD_CRACK, dig_index));
 				}
+
+				dig_time += dtime;
 			}
 			
 			if(g_input->getRightClicked())