diff options
author | Bond_009 <bond.009@outlook.com> | 2022-01-02 12:11:54 +0100 |
---|---|---|
committer | Alexander Harkness <me@bearbin.net> | 2022-01-13 00:48:19 +0100 |
commit | 1a310981826b800b1a4a405eaef0612cf7b1cb70 (patch) | |
tree | 869bc47f6ef261a4f7e9359b2808d685cc24c366 | |
parent | Suspicious packets: give 'em the silent treatment (#5375) (diff) | |
download | cuberite-1a310981826b800b1a4a405eaef0612cf7b1cb70.tar cuberite-1a310981826b800b1a4a405eaef0612cf7b1cb70.tar.gz cuberite-1a310981826b800b1a4a405eaef0612cf7b1cb70.tar.bz2 cuberite-1a310981826b800b1a4a405eaef0612cf7b1cb70.tar.lz cuberite-1a310981826b800b1a4a405eaef0612cf7b1cb70.tar.xz cuberite-1a310981826b800b1a4a405eaef0612cf7b1cb70.tar.zst cuberite-1a310981826b800b1a4a405eaef0612cf7b1cb70.zip |
-rw-r--r-- | SetFlags.cmake | 4 | ||||
-rw-r--r-- | src/ByteBuffer.cpp | 10 | ||||
-rw-r--r-- | src/Entities/Pawn.cpp | 2 |
3 files changed, 9 insertions, 7 deletions
diff --git a/SetFlags.cmake b/SetFlags.cmake index 869307d91..51b998247 100644 --- a/SetFlags.cmake +++ b/SetFlags.cmake @@ -158,7 +158,7 @@ function(set_exe_flags TARGET) # TODO: actually fix the warnings instead of disabling them # or at least disable on a file-level basis: -Wno-missing-noreturn -Wno-padded -Wno-implicit-fallthrough - -Wno-double-promotion + -Wno-double-promotion -Wno-reserved-identifier # This is a pretty useless warning, we've already got -Wswitch which is what we need: -Wno-switch-enum @@ -169,7 +169,7 @@ function(set_exe_flags TARGET) -Wno-documentation-unknown-command -Wno-reserved-id-macro -Wno-error=unused-command-line-argument ) - if(CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL 7) + if(CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL 7 AND CMAKE_CXX_COMPILER_VERSION VERSION_LESS 13) target_compile_options( ${TARGET} PRIVATE diff --git a/src/ByteBuffer.cpp b/src/ByteBuffer.cpp index 127227180..80be9baee 100644 --- a/src/ByteBuffer.cpp +++ b/src/ByteBuffer.cpp @@ -117,8 +117,8 @@ bool cByteBuffer::Write(const void * a_Bytes, size_t a_Count) size_t CurFreeSpace = GetFreeSpace(); #ifndef NDEBUG size_t CurReadableSpace = GetReadableSpace(); + size_t WrittenBytes = 0; #endif - size_t WrittenBytes = 0; if (CurFreeSpace < a_Count) { @@ -135,7 +135,9 @@ bool cByteBuffer::Write(const void * a_Bytes, size_t a_Count) memcpy(m_Buffer + m_WritePos, Bytes, TillEnd); Bytes += TillEnd; a_Count -= TillEnd; - WrittenBytes = TillEnd; + #ifndef NDEBUG + WrittenBytes = TillEnd; + #endif } m_WritePos = 0; } @@ -145,7 +147,9 @@ bool cByteBuffer::Write(const void * a_Bytes, size_t a_Count) { memcpy(m_Buffer + m_WritePos, Bytes, a_Count); m_WritePos += a_Count; - WrittenBytes += a_Count; + #ifndef NDEBUG + WrittenBytes += a_Count; + #endif } ASSERT(GetFreeSpace() == CurFreeSpace - WrittenBytes); diff --git a/src/Entities/Pawn.cpp b/src/Entities/Pawn.cpp index e227dca6f..760c9c0e6 100644 --- a/src/Entities/Pawn.cpp +++ b/src/Entities/Pawn.cpp @@ -324,7 +324,6 @@ void cPawn::HandleFalling(void) /* We initialize these with what the foot is really IN, because for sampling we will move down with the epsilon above */ bool IsFootInWater = IsBlockWater(BlockAtFoot); - bool IsFootInLiquid = IsFootInWater || IsBlockLava(BlockAtFoot) || (BlockAtFoot == E_BLOCK_COBWEB); // okay so cobweb is not _technically_ a liquid... bool IsFootOnSlimeBlock = false; /* The "cross" we sample around to account for the player width/girth */ @@ -377,7 +376,6 @@ void cPawn::HandleFalling(void) if (j == 0) { IsFootInWater |= IsBlockWater(BlockType); - IsFootInLiquid |= IsFootInWater || IsBlockLava(BlockType) || (BlockType == E_BLOCK_COBWEB); // okay so cobweb is not _technically_ a liquid... IsFootOnSlimeBlock |= (BlockType == E_BLOCK_SLIME_BLOCK); } |