Fix compiler warns

CMakeLists.txt

@@ -28,7 +28,7 @@ check_symbol_exists("_WIN32" "" MCPE_WIN32)
 
 # Clang
 if(CMAKE_CXX_COMPILER_ID STREQUAL "Clang")
-    add_compile_options(-Wno-inconsistent-missing-override -Wno-enum-compare-switch -Wno-register)
+    add_compile_options(-Wno-inconsistent-missing-override)
 endif()
 
 # Windows Linking
@@ -53,7 +53,7 @@ else()
 endif()
 
 # Disable thread local storage (needed for C++98 & iOS)
-add_compile_options(-DRAPIDJSON_NO_THREAD_LOCAL -DSTBI_NO_THREAD_LOCALS)
+add_compile_definitions(RAPIDJSON_NO_THREAD_LOCAL STBI_NO_THREAD_LOCALS)
 
 # Android Logging
 if(ANDROID)
@@ -76,7 +76,11 @@ else()
     file(CREATE_LINK "${CMAKE_CURRENT_SOURCE_DIR}/game/assets" "${CMAKE_CURRENT_BINARY_DIR}/assets" SYMBOLIC)
 endif()
 
-#VBO Emulation
+# VBO Emulation
 if(USE_GL_VBO_EMULATION)
-    add_compile_options(-DUSE_GL_VBO_EMULATION)
+    add_compile_definitions(USE_GL_VBO_EMULATION)
 endif()
+
+# Disable warnings on third party libraries
+target_compile_options(stb_vorbis PRIVATE -w)
+target_compile_options(stb_image PRIVATE -w)

platforms/ios/CMakeLists.txt

@@ -2,7 +2,8 @@ cmake_minimum_required(VERSION 3.16.0)
 project(reminecraftpe-ios)
 
 target_compile_definitions(reminecraftpe-core PUBLIC HANDLE_CHARS_SEPARATELY USE_GLES)
-target_compile_definitions(stb_image PUBLIC STBI_NO_THREAD_LOCALS)
+
+add_compile_options(-Wno-deprecated-declarations)
 
 # Build
 add_executable(reminecraftpe

platforms/ios/minecraftpeViewController.mm

@@ -297,7 +297,7 @@ - (void)setAnimationFrameInterval:(NSInteger)frameInterval
     if (frameInterval >= 1)
     {
         animationFrameInterval = frameInterval;
-        NSLog(@"Set animationFrameInterval to %d\n", animationFrameInterval);
+        NSLog(@"Set animationFrameInterval to %ld\n", static_cast<long>(animationFrameInterval));
 
         if (animating)
         {

source/client/app/Minecraft.cpp

@@ -467,6 +467,8 @@ void Minecraft::handleBuildAction(const BuildActionIntention& action)
 			}
 			break;
 		}
+		default:
+			break;
 	}
 
 	if (bInteract && action.isInteract() && canInteract)

source/client/network/ClientSideNetworkHandler.cpp

@@ -105,6 +105,8 @@ void ClientSideNetworkHandler::handle(const RakNet::RakNetGUID& rakGuid, LoginSt
 	case LoginStatusPacket::STATUS_SERVER_OUTDATED:
 		m_pMinecraft->setScreen(new DisconnectionScreen("Could not connect: Outdated server!"));
 		break;
+	case LoginStatusPacket::STATUS_SUCCESS:
+		break;
 	}
 }
 
@@ -840,4 +842,4 @@ void ClientSideNetworkHandler::flushAllBufferedUpdates()
 void ClientSideNetworkHandler::handleBlockUpdate(const BlockUpdate& u)
 {
 	m_pLevel->setTileAndData(u.pos, Tile::TransformToValidBlockId(u.tile), u.data);
-}
+}

source/client/renderer/entity/EntityRenderDispatcher.cpp

@@ -126,9 +126,9 @@ EntityRenderer* EntityRenderDispatcher::getRenderer(Entity::RenderType renderTyp
 		case Entity::RENDER_FALLING_TILE:
 			return &m_FallingTileRenderer;
 #endif
+		default:
+			return nullptr;
 	}
-
-	return nullptr;
 }
 
 EntityRenderer* EntityRenderDispatcher::getRenderer(const Entity& entity)

source/client/sound/SoundData.cpp

@@ -54,7 +54,7 @@ bool SoundDesc::_loadPcm(const AppPlatform* platform, const char *name)
     m_codecType = AudioCodec::PCM;
     m_fileData = m_file.data;
     m_header = *(PCMSoundHeader *) m_fileData;
-    m_buffer.m_pData = (void *) (m_fileData + sizeof(PCMSoundHeader));
+    m_buffer.m_pData = (int16_t *) (m_fileData + sizeof(PCMSoundHeader));
     m_buffer.m_dataSize = m_header.m_channels * m_header.m_length * m_header.m_bytes_per_sample;
 
     // Success!
@@ -76,9 +76,7 @@ bool SoundDesc::_loadOgg(const AppPlatform* platform, const char* category, cons
     m_codecType = AudioCodec::OGG;
     m_fileData = m_file.data;
     m_header.m_bytes_per_sample = 2; // Always 2 (16-bit)
-    // Casting to a short** here might cause problems. Let's find out...
-    // Seems like it doesn't. Cool.
-    m_header.m_length = stb_vorbis_decode_memory(m_file.data, (int) m_file.size, &m_header.m_channels, &m_header.m_sample_rate, (short **) &m_buffer.m_pData);
+    m_header.m_length = stb_vorbis_decode_memory(m_file.data, (int) m_file.size, &m_header.m_channels, &m_header.m_sample_rate, (int16_t **) &m_buffer.m_pData);
     if (m_header.m_length == -1)
     {
         LOG_E("An error occurred while trying to decode a sound!");
@@ -125,4 +123,4 @@ void SoundDesc::_unloadAll()
 // Declare Variables
 #define SOUND(category, name, number) SoundDesc SA_##name##number;
 #include "sound_list.h"
-#undef SOUND
+#undef SOUND

source/client/sound/SoundData.hpp

@@ -35,7 +35,7 @@ struct PCMSoundHeader
 
 struct SoundBuffer
 {
-	void* m_pData;
+	int16_t *m_pData;
 	int m_dataSize;
 };
 

source/client/sound/SoundStream.cpp

@@ -36,7 +36,7 @@ bool SoundStream::_stream(int bufferId)
 
     while (size < m_tempPcmBuffer.m_dataSize)
     {
-        result = stb_vorbis_get_samples_short_interleaved(m_decoder, m_info.channels, (short*)m_tempPcmBuffer.m_pData + size, m_tempPcmBuffer.m_dataSize - size);
+        result = stb_vorbis_get_samples_short_interleaved(m_decoder, m_info.channels, (int16_t *)m_tempPcmBuffer.m_pData + size, m_tempPcmBuffer.m_dataSize - size);
         if (result > 0) size += result * m_info.channels;
         else break;
     }

source/nbt/Tag.hpp

@@ -59,7 +59,7 @@ class TagMemoryChunk
 		m_size = size;
 		if (size > 0)
 		{
-			m_buffer = new uint8_t(size);
+			m_buffer = new uint8_t[size];
 			if (m_buffer)
 				memset(m_buffer, 0, size);
 		}
@@ -83,4 +83,4 @@ class TagMemoryChunk
 	unsigned int m_elements;
 	unsigned int m_size;
 	uint8_t* m_buffer;
-};
+};

source/renderer/hal/ogl/RenderContextOGL.cpp

@@ -215,7 +215,7 @@ void RenderContextOGL::drawIndexed(PrimitiveMode primitiveMode, unsigned int cou
 
 void RenderContextOGL::drawIndexed(PrimitiveMode primitiveMode, unsigned int count, unsigned int startOffset, uint8_t indexSize)
 {
-    glDrawElements(modeMap[primitiveMode], count, indexType[indexSize], (const GLvoid*)(startOffset * indexSize));
+    glDrawElements(modeMap[primitiveMode], count, indexType[indexSize], (const GLvoid*)((uintptr_t)startOffset * indexSize));
 }
 
 void RenderContextOGL::setDepthRange(float nearVal, float farVal)
@@ -336,4 +336,4 @@ GLenum mce::getComparisonFunc(ComparisonFunc comparisonFunc)
         LOG_E("Unknown comparisonFunc: %d", comparisonFunc);
         throw std::bad_cast();
     }
-}
+}

source/server/ServerSideNetworkHandler.cpp

@@ -99,7 +99,7 @@ void ServerSideNetworkHandler::onDisconnect(const RakNet::RakNetGUID& guid)
 		// remove it from our world
 		m_pLevel->removeEntity(pPlayer);
 	}
-	else if (pPlayer = getPendingPlayerByGUID(guid))
+	else if ((pPlayer = getPendingPlayerByGUID(guid)))
 	{
 		// Player was still loading
 		m_pendingPlayers.erase(guid);

source/world/entity/SynchedEntityData.cpp

@@ -20,7 +20,7 @@ MAP(Vec3,         TYPE_VEC3,         Vec3())
 SynchedEntityData::SynchedEntityData()
 {
     m_itemsArray = ItemsArray();
-    m_minIdxDirty = INT_MAX;
+    m_minIdxDirty = UINT16_MAX;
     m_maxIdxDirty = 0;
 }
 
@@ -72,7 +72,7 @@ void SynchedEntityData::clear()
 
     m_itemsArray.clear();
     // Mark as clean
-    m_minIdxDirty = INT_MAX;
+    m_minIdxDirty = UINT16_MAX;
     m_maxIdxDirty = 0;
 }
 
@@ -91,7 +91,7 @@ SynchedEntityData::ItemsArray SynchedEntityData::packDirty()
     }
 
     // Mark as clean
-    m_minIdxDirty = INT_MAX;
+    m_minIdxDirty = UINT16_MAX;
     return result;
 }
 
@@ -274,4 +274,4 @@ SynchedEntityData::ItemsArray SynchedEntityData::Unpack(IDataInput& dis)
     }
 
     return result;
-}
+}

source/world/phys/Vec2.cpp

@@ -6,14 +6,14 @@
 	SPDX-License-Identifier: BSD-1-Clause
  ********************************************************************/
 
-#include <climits>
+#include <cfloat>
 
 #include "Vec2.hpp"
 
 const Vec2 Vec2::ZERO = Vec2(0, 0), Vec2::ONE = Vec2(1, 1);
 const Vec2 Vec2::UNIT_X = Vec2(1, 0), Vec2::NEG_UNIT_X = Vec2(-1, 0);
 const Vec2 Vec2::UNIT_Y = Vec2(0, 1), Vec2::NEG_UNIT_Y = Vec2(0, -1);
-const Vec2 Vec2::MIN = Vec2(INT_MIN, INT_MIN), Vec2::MAX = Vec2(INT_MAX, INT_MAX);
+const Vec2 Vec2::MIN = Vec2(FLT_MIN, FLT_MIN), Vec2::MAX = Vec2(FLT_MAX, FLT_MAX);
 
 void Vec2::_init(float x, float y)
 {

source/world/phys/Vec3.cpp

@@ -6,7 +6,7 @@
 	SPDX-License-Identifier: BSD-1-Clause
  ********************************************************************/
 
-#include <climits>
+#include <cfloat>
 
 #include "Vec3.hpp"
 #include "world/level/TilePos.hpp"
@@ -15,7 +15,7 @@ const Vec3 Vec3::ZERO   = Vec3(0, 0, 0), Vec3::ONE        = Vec3(1, 1, 1);
 const Vec3 Vec3::UNIT_X = Vec3(1, 0, 0), Vec3::NEG_UNIT_X = Vec3(-1, 0, 0);
 const Vec3 Vec3::UNIT_Y = Vec3(0, 1, 0), Vec3::NEG_UNIT_Y = Vec3(0, -1, 0);
 const Vec3 Vec3::UNIT_Z = Vec3(0, 0, 1), Vec3::NEG_UNIT_Z = Vec3(0, 0, -1);
-const Vec3 Vec3::MIN = Vec3(INT_MIN, INT_MIN, INT_MIN), Vec3::MAX = Vec3(INT_MAX, INT_MAX, INT_MAX);
+const Vec3 Vec3::MIN = Vec3(FLT_MIN, FLT_MIN, FLT_MIN), Vec3::MAX = Vec3(FLT_MAX, FLT_MAX, FLT_MAX);
 
 void Vec3::_init(float x, float y, float z)
 {

source/world/tile/GrassTile.cpp

@@ -53,6 +53,8 @@ int GrassTile::getTexture(const LevelSource* level, const TilePos& pos, Facing::
 		return TEXTURE_GRASS_TOP;
 	case Facing::DOWN:
 		return TEXTURE_DIRT;
+	default:
+		break;
 	}
 
 	Material* pMat = level->getMaterial(pos.above());

source/world/tile/SandStoneTile.cpp

@@ -21,7 +21,7 @@ int SandStoneTile::getTexture(Facing::Name face) const
 		return TEXTURE_SANDSTONE_BOTTOM;
 	case Facing::UP:
 		return TEXTURE_SANDSTONE_TOP;
+	default:
+		return m_TextureFrame;
 	}
-
-	return m_TextureFrame;
 }

source/world/tile/TntTile.cpp

@@ -28,9 +28,9 @@ int TntTile::getTexture(Facing::Name face) const
 		return m_TextureFrame + 2;
 	case Facing::UP:
 		return m_TextureFrame + 1;
+	default:
+		return m_TextureFrame;
 	}
-
-	return m_TextureFrame;
 }
 
 void TntTile::destroy(Level* level, const TilePos& pos, TileData data)

thirdparty/raknet/DynDNS.cpp

@@ -219,7 +219,7 @@ void DynDNS::Update(void)
 				existingHost[0]=0;
 				// Resolve DNS we are setting. If equal to current then abort
 				RakNetSocket2::DomainNameToIP(host.C_String(), existingHost);
-				if (existingHost && strcmp(existingHost, myIPStr)==0)
+				if (strcmp(existingHost, myIPStr)==0)
 				{
 					// DynDNS considers setting the IP to what it is already set abuse
 					tcp->DeallocatePacket(packet);

thirdparty/raknet/Rand.cpp

@@ -142,8 +142,8 @@ void seedMT( unsigned int seed, unsigned int *state, unsigned int *&next, int &l
 	// so-- that's why the only change I made is to restrict to odd seeds.
 	//
 
-	register unsigned int x = ( seed | 1U ) & 0xFFFFFFFFU, *s = state;
-	register int j;
+	unsigned int x = ( seed | 1U ) & 0xFFFFFFFFU, *s = state;
+	int j;
 
 	for ( left = 0, *s++ = x, j = N; --j;
 		*s++ = ( x *= 69069U ) & 0xFFFFFFFFU )