Merge branch 'warnings' of https://github.com/unknownbrackets/ppsspp into unknownbrackets-warnings

Conflicts:
	Core/ELF/ElfReader.cpp
This commit is contained in:
Henrik Rydgard 2013-11-15 19:11:51 +01:00
commit 6be534ca18
3 changed files with 10 additions and 8 deletions

View file

@ -38,7 +38,9 @@
// __FUNCTION__ is misused a lot below, it's no longer a string literal but a virtual
// variable so this use fails in some compilers. Just define it away for now.
#ifndef _MSC_VER
#define __FUNCTION__ "(n/a)"
#endif
namespace ArmGen
{
@ -166,8 +168,8 @@ void ARMXEmitter::ADDI2R(ARMReg rd, ARMReg rs, u32 val, ARMReg scratch)
// Decompose into two additions.
ADD(rd, rs, Operand2((u8)(val >> 8), 12)); // rotation right by 12*2 == rotation left by 8
ADD(rd, rd, Operand2((u8)(val), 0));
} else if (((-(u32)(s32)val) & 0xFFFF0000) == 0) {
val = -(u32)(s32)val;
} else if ((((u32)-(s32)val) & 0xFFFF0000) == 0) {
val = (u32)-(s32)val;
SUB(rd, rs, Operand2((u8)(val >> 8), 12));
SUB(rd, rd, Operand2((u8)(val), 0));
} else {

View file

@ -69,9 +69,9 @@ bool ElfReader::LoadRelocations(Elf32_Rel *rels, int numRelocs)
//0 = code
//1 = data
if (readwrite >= ARRAY_SIZE(segmentVAddr)) {
if (readwrite >= (int)ARRAY_SIZE(segmentVAddr)) {
if (numErrors < 10) {
ERROR_LOG(LOADER, "Bad segment number %i", readwrite);
ERROR_LOG_REPORT(LOADER, "Bad segment number %i", readwrite);
}
numErrors++;
continue;
@ -82,9 +82,9 @@ bool ElfReader::LoadRelocations(Elf32_Rel *rels, int numRelocs)
// It appears that misaligned relocations are allowed.
// Will they work correctly on big-endian?
if (!Memory::IsValidAddress(addr)) {
if (((addr & 3) && type != R_MIPS_32) || !Memory::IsValidAddress(addr)) {
if (numErrors < 10) {
WARN_LOG(LOADER, "Suspicious address %08x, skipping reloc", addr);
WARN_LOG_REPORT(LOADER, "Suspicious address %08x, skipping reloc, type = %d", addr, type);
} else if (numErrors == 10) {
WARN_LOG(LOADER, "Too many bad relocations, skipping logging");
}

View file

@ -169,8 +169,8 @@ void MIPSState::DoState(PointerWrap &p) {
p.Do(lo);
p.Do(fpcond);
if (s <= 1) {
u32 fcr0_unusued = 0;
p.Do(fcr0_unusued);
u32 fcr0_unused = 0;
p.Do(fcr0_unused);
}
p.Do(fcr31);
p.Do(rng.m_w);