diff --git a/Source/Android/jni/AndroidCommon/AndroidCommon.h b/Source/Android/jni/AndroidCommon/AndroidCommon.h index 733d2882f6..cc86795418 100644 --- a/Source/Android/jni/AndroidCommon/AndroidCommon.h +++ b/Source/Android/jni/AndroidCommon/AndroidCommon.h @@ -19,8 +19,9 @@ jobjectArray VectorToJStringArray(JNIEnv* env, const std::vector& v template jobjectArray VectorToJObjectArray(JNIEnv* env, const std::vector& vector, jclass clazz, F f) { - jobjectArray result = env->NewObjectArray(vector.size(), clazz, nullptr); - for (jsize i = 0; i < vector.size(); ++i) + const auto vector_size = static_cast(vector.size()); + jobjectArray result = env->NewObjectArray(vector_size, clazz, nullptr); + for (jsize i = 0; i < vector_size; ++i) { jobject obj = f(env, vector[i]); env->SetObjectArrayElement(result, i, obj); diff --git a/Source/Core/Common/TraversalClient.cpp b/Source/Core/Common/TraversalClient.cpp index 3198144479..44b8390f90 100644 --- a/Source/Core/Common/TraversalClient.cpp +++ b/Source/Core/Common/TraversalClient.cpp @@ -373,7 +373,7 @@ void TraversalClient::HandleTraversalTest() waitCondition = 0; break; } - else if (rv < sizeof(packet) || raddr.host != m_ServerAddress.host || + else if (rv < int(sizeof(packet)) || raddr.host != m_ServerAddress.host || raddr.host != m_portAlt || packet.requestId != m_TestRequestId) { // irrelevant packet, ignore diff --git a/Source/Core/Core/PowerPC/JitArm64/JitArm64_RegCache.cpp b/Source/Core/Core/PowerPC/JitArm64/JitArm64_RegCache.cpp index c5484da30b..8538f2c040 100644 --- a/Source/Core/Core/PowerPC/JitArm64/JitArm64_RegCache.cpp +++ b/Source/Core/Core/PowerPC/JitArm64/JitArm64_RegCache.cpp @@ -248,7 +248,7 @@ void Arm64GPRCache::FlushRegisters(BitSet32 regs, bool maintain_state, ARM64Reg ASSERT_MSG(DYNA_REC, m_guest_registers[GUEST_GPR_OFFSET + i].GetType() != RegType::Discarded, "Attempted to flush discarded register"); - if (i + 1 < GUEST_GPR_COUNT && regs[i + 1]) + if (i + 1 < int(GUEST_GPR_COUNT) && regs[i + 1]) { // We've got two guest registers in a row to store OpArg& reg1 = m_guest_registers[GUEST_GPR_OFFSET + i]; diff --git a/Source/Core/Core/PowerPC/JitArm64/JitArm64_SystemRegisters.cpp b/Source/Core/Core/PowerPC/JitArm64/JitArm64_SystemRegisters.cpp index fda306984f..3521f9c66b 100644 --- a/Source/Core/Core/PowerPC/JitArm64/JitArm64_SystemRegisters.cpp +++ b/Source/Core/Core/PowerPC/JitArm64/JitArm64_SystemRegisters.cpp @@ -236,9 +236,9 @@ void JitArm64::twx(UGeckoInstruction inst) constexpr std::array conditions{{CC_LT, CC_GT, CC_EQ, CC_VC, CC_VS}}; Common::SmallVector fixups; - for (int i = 0; i < conditions.size(); i++) + for (size_t i = 0; i < conditions.size(); i++) { - if (inst.TO & (1 << i)) + if (inst.TO & (1U << i)) { FixupBranch f = B(conditions[i]); fixups.push_back(f); diff --git a/Source/Core/VideoCommon/VertexManagerBase.cpp b/Source/Core/VideoCommon/VertexManagerBase.cpp index 65772f3dad..ac4446abba 100644 --- a/Source/Core/VideoCommon/VertexManagerBase.cpp +++ b/Source/Core/VideoCommon/VertexManagerBase.cpp @@ -598,7 +598,7 @@ void VertexManagerBase::Flush() std::optional custom_pixel_shader; std::vector custom_pixel_texture_names; std::span custom_pixel_shader_uniforms; - for (int i = 0; i < texture_names.size(); i++) + for (size_t i = 0; i < texture_names.size(); i++) { const std::string& texture_name = texture_names[i]; const u32 texture_unit = texture_units[i]; diff --git a/Source/UnitTests/Common/FloatUtilsTest.cpp b/Source/UnitTests/Common/FloatUtilsTest.cpp index e0172c5643..61b7f80974 100644 --- a/Source/UnitTests/Common/FloatUtilsTest.cpp +++ b/Source/UnitTests/Common/FloatUtilsTest.cpp @@ -85,7 +85,7 @@ TEST(FloatUtils, ApproximateReciprocalSquareRoot) 0x7FF8'0000'0000'0000, 0x7FF8'0000'0000'0000, 0x3FEA'2040'0000'0000, 0x3FA0'3108'0000'0000, 0x7FF8'0000'0000'0000}; - for (int i = 0; i < double_test_values.size(); ++i) + for (size_t i = 0; i < double_test_values.size(); ++i) { u64 ivalue = double_test_values[i]; double dvalue = Common::BitCast(ivalue);