From b8ab03bc6b41dbe8262f87def3ff3513924315d4 Mon Sep 17 00:00:00 2001 From: Tiger Wang Date: Tue, 24 Jul 2018 00:35:20 +0100 Subject: At long last... Piston animations! * Fixes #3198 * Fixes #57 (again lol) --- src/Blocks/BlockPiston.cpp | 158 ++++++++++++++++++++++++++++----------------- src/Blocks/BlockPiston.h | 6 ++ 2 files changed, 105 insertions(+), 59 deletions(-) (limited to 'src/Blocks') diff --git a/src/Blocks/BlockPiston.cpp b/src/Blocks/BlockPiston.cpp index 16dd53daf..3fa421141 100644 --- a/src/Blocks/BlockPiston.cpp +++ b/src/Blocks/BlockPiston.cpp @@ -214,36 +214,58 @@ bool cBlockPistonHandler::CanPushBlock( void cBlockPistonHandler::ExtendPiston(Vector3i a_BlockPos, cWorld & a_World) { - BLOCKTYPE pistonBlock; - NIBBLETYPE pistonMeta; - a_World.GetBlockTypeMeta(a_BlockPos.x, a_BlockPos.y, a_BlockPos.z, pistonBlock, pistonMeta); - - if (IsExtended(pistonMeta)) { - // Already extended, bail out - return; + // Broadcast block action first. Will do nothing if piston cannot in fact push + + BLOCKTYPE pistonBlock; + NIBBLETYPE pistonMeta; + a_World.GetBlockTypeMeta(a_BlockPos.x, a_BlockPos.y, a_BlockPos.z, pistonBlock, pistonMeta); + a_World.BroadcastBlockAction(a_BlockPos, PistonExtendAction, pistonMeta, pistonBlock); } - Vector3i pushDir = MetadataToOffset(pistonMeta); + // Client expects the server to "play" the animation before setting the final blocks + // However, we don't confuse animation with the underlying state of the world, so emulate by delaying 1 tick + // (Probably why vanilla has so many dupe glitches with sand and pistons lolol) - Vector3iSet blocksPushed; - if (!CanPushBlock(a_BlockPos + pushDir, a_World, true, blocksPushed, pushDir)) - { - // Can't push anything, bail out - return; - } + a_World.ScheduleTask(1, [a_BlockPos](cWorld & World) + { + BLOCKTYPE pistonBlock; + NIBBLETYPE pistonMeta; + World.GetBlockTypeMeta(a_BlockPos.x, a_BlockPos.y, a_BlockPos.z, pistonBlock, pistonMeta); - a_World.BroadcastBlockAction(a_BlockPos, 0, pistonMeta, pistonBlock); - a_World.BroadcastSoundEffect("block.piston.extend", a_BlockPos, 0.5f, 0.7f); + if ((pistonBlock != E_BLOCK_PISTON) && !IsSticky(pistonBlock)) + { + // Ensure we operate on a piston to avoid spurious behaviour + // Note that the scheduled task may result in the block type of a_BlockPos changing + return; + } - PushBlocks(blocksPushed, a_World, pushDir); + if (IsExtended(pistonMeta)) + { + // Already extended, bail out + return; + } - // Set the extension and the piston base correctly - Vector3i extensionPos = a_BlockPos + pushDir; - a_World.SetBlock(a_BlockPos.x, a_BlockPos.y, a_BlockPos.z, pistonBlock, pistonMeta | 0x8); - a_World.SetBlock( - extensionPos.x, extensionPos.y, extensionPos.z, - E_BLOCK_PISTON_EXTENSION, pistonMeta | (IsSticky(pistonBlock) ? 8 : 0) + Vector3i pushDir = MetadataToOffset(pistonMeta); + Vector3iSet blocksPushed; + if (!CanPushBlock(a_BlockPos + pushDir, World, true, blocksPushed, pushDir)) + { + // Can't push anything, bail out + return; + } + PushBlocks(blocksPushed, World, pushDir); + + // Set the extension and the piston base correctly + Vector3i extensionPos = a_BlockPos + pushDir; + World.SetBlock(a_BlockPos.x, a_BlockPos.y, a_BlockPos.z, pistonBlock, pistonMeta | 0x8); + World.SetBlock( + extensionPos.x, extensionPos.y, extensionPos.z, + E_BLOCK_PISTON_EXTENSION, pistonMeta | (IsSticky(pistonBlock) ? 8 : 0) + ); + + // Play sound effect only if extended successfully + World.BroadcastSoundEffect("block.piston.extend", a_BlockPos, 0.5f, 0.7f); + } ); } @@ -253,52 +275,70 @@ void cBlockPistonHandler::ExtendPiston(Vector3i a_BlockPos, cWorld & a_World) void cBlockPistonHandler::RetractPiston(Vector3i a_BlockPos, cWorld & a_World) { - BLOCKTYPE pistonBlock; - NIBBLETYPE pistonMeta; - a_World.GetBlockTypeMeta(a_BlockPos.x, a_BlockPos.y, a_BlockPos.z, pistonBlock, pistonMeta); - - if (!IsExtended(pistonMeta)) { - // Already retracted, bail out - return; + BLOCKTYPE pistonBlock; + NIBBLETYPE pistonMeta; + a_World.GetBlockTypeMeta(a_BlockPos.x, a_BlockPos.y, a_BlockPos.z, pistonBlock, pistonMeta); + a_World.BroadcastBlockAction(a_BlockPos, PistonRetractAction, pistonMeta, pistonBlock); } - Vector3i pushDir = MetadataToOffset(pistonMeta); + a_World.ScheduleTask(1, [a_BlockPos](cWorld & World) + { + BLOCKTYPE pistonBlock; + NIBBLETYPE pistonMeta; + World.GetBlockTypeMeta(a_BlockPos.x, a_BlockPos.y, a_BlockPos.z, pistonBlock, pistonMeta); - // Check the extension: - Vector3i extensionPos = a_BlockPos + pushDir; - if (a_World.GetBlock(extensionPos) != E_BLOCK_PISTON_EXTENSION) - { - LOGD("%s: Piston without an extension - still extending, or just in an invalid state?", __FUNCTION__); - return; - } + if ((pistonBlock != E_BLOCK_PISTON) && !IsSticky(pistonBlock)) + { + // Ensure we operate on a piston to avoid spurious behaviour + // Note that the scheduled task may result in the block type of a_BlockPos changing + return; + } - // Remove Extension - a_World.SetBlock(extensionPos.x, extensionPos.y, extensionPos.z, E_BLOCK_AIR, 0); + if (!IsExtended(pistonMeta)) + { + // Already retracted, bail out + return; + } - a_World.SetBlock(a_BlockPos.x, a_BlockPos.y, a_BlockPos.z, pistonBlock, pistonMeta & ~(8)); - a_World.BroadcastBlockAction(a_BlockPos, 1, pistonMeta & ~(8), pistonBlock); - a_World.BroadcastSoundEffect("block.piston.contract", a_BlockPos, 0.5f, 0.7f); + Vector3i pushDir = MetadataToOffset(pistonMeta); - if (!IsSticky(pistonBlock)) - { - // No need for block pulling, bail out - return; - } + // Check the extension: + Vector3i extensionPos = a_BlockPos + pushDir; + if (World.GetBlock(extensionPos) != E_BLOCK_PISTON_EXTENSION) + { + LOGD("%s: Piston without an extension - still extending, or just in an invalid state?", __FUNCTION__); + return; + } - // Get the block to pull - a_BlockPos += pushDir * 2; - // Try to "push" the pulling block in the opposite direction - pushDir *= -1; + // Remove extension, update base state + World.SetBlock(extensionPos.x, extensionPos.y, extensionPos.z, E_BLOCK_AIR, 0); + World.SetBlock(a_BlockPos.x, a_BlockPos.y, a_BlockPos.z, pistonBlock, pistonMeta & ~(8)); - Vector3iSet pushedBlocks; - if (!CanPushBlock(a_BlockPos, a_World, false, pushedBlocks, pushDir)) - { - // Not pushable, bail out - return; - } + // (Retraction is always successful, but play in the task for consistency) + World.BroadcastSoundEffect("block.piston.contract", a_BlockPos, 0.5f, 0.7f); - PushBlocks(pushedBlocks, a_World, pushDir); + if (!IsSticky(pistonBlock)) + { + // No need for block pulling, bail out + return; + } + + // Get the block to pull + Vector3i AdjustedPosition = a_BlockPos + pushDir * 2; + // Try to "push" the pulling block in the opposite direction + pushDir *= -1; + + Vector3iSet pushedBlocks; + if (!CanPushBlock(AdjustedPosition, World, false, pushedBlocks, pushDir)) + { + // Not pushable, bail out + return; + } + + PushBlocks(pushedBlocks, World, pushDir); + } + ); } diff --git a/src/Blocks/BlockPiston.h b/src/Blocks/BlockPiston.h index 21ebbcd39..e604cd38e 100644 --- a/src/Blocks/BlockPiston.h +++ b/src/Blocks/BlockPiston.h @@ -100,6 +100,12 @@ private: typedef std::unordered_set> Vector3iSet; + /** Piston extension block action */ + static const Byte PistonExtendAction = 0U; + + /** Piston retraction block action */ + static const Byte PistonRetractAction = 1U; + /** Returns true if the piston (specified by blocktype) is a sticky piston */ static inline bool IsSticky(BLOCKTYPE a_BlockType) { return (a_BlockType == E_BLOCK_STICKY_PISTON); } -- cgit v1.2.3