summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBond_009 <bond.009@outlook.com>2022-01-02 12:11:54 +0100
committerAlexander Harkness <me@bearbin.net>2022-01-13 00:48:19 +0100
commit1a310981826b800b1a4a405eaef0612cf7b1cb70 (patch)
tree869bc47f6ef261a4f7e9359b2808d685cc24c366
parentSuspicious packets: give 'em the silent treatment (#5375) (diff)
downloadcuberite-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.cmake4
-rw-r--r--src/ByteBuffer.cpp10
-rw-r--r--src/Entities/Pawn.cpp2
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);
}