1
0

Comments & new BLOCK_FACE constants

This commit is contained in:
Tiger Wang 2014-02-18 00:28:31 +00:00
parent 320cc74f0a
commit ced6eb971d

View File

@ -23,23 +23,26 @@ public:
{ {
if (a_Dir == BLOCK_FACE_NONE) if (a_Dir == BLOCK_FACE_NONE)
{ {
// Client sends this if clicked on top or bottom face
return false; return false;
} }
AddFaceDirection(a_BlockX, a_BlockY, a_BlockZ, a_Dir); AddFaceDirection(a_BlockX, a_BlockY, a_BlockZ, a_Dir); // Make sure block that will be occupied is free
BLOCKTYPE Block = a_World->GetBlock(a_BlockX, a_BlockY, a_BlockZ); BLOCKTYPE Block = a_World->GetBlock(a_BlockX, a_BlockY, a_BlockZ);
AddFaceDirection(a_BlockX, a_BlockY, a_BlockZ, a_Dir, true); AddFaceDirection(a_BlockX, a_BlockY, a_BlockZ, a_Dir, true); // We want the clicked block, so go back again
if (Block == E_BLOCK_AIR) if (Block == E_BLOCK_AIR)
{ {
int Dir = 0; int Dir = 0;
// The client uses different values for painting directions and block faces. Our constants are for the block faces, so we convert them here to painting faces
switch (a_Dir) switch (a_Dir)
{ {
case BLOCK_FACE_SOUTH: break; case BLOCK_FACE_ZP: break; // Initialised to zero
case BLOCK_FACE_NORTH: Dir = 2; break; case BLOCK_FACE_ZM: Dir = 2; break;
case BLOCK_FACE_WEST: Dir = 1; break; case BLOCK_FACE_XM: Dir = 1; break;
case BLOCK_FACE_EAST: Dir = 3; break; case BLOCK_FACE_XP: Dir = 3; break;
default: return false; default: ASSERT(!"Unhandled block face when trying spawn painting!"); return false;
} }
static const struct // Define all the possible painting titles static const struct // Define all the possible painting titles