Rewrite some expressions to avoid type checker edge cases
This commit is contained in:
parent
28241e3042
commit
6afb3be98c
|
@ -513,7 +513,9 @@ namespace BizHawk.Emulation.Cores.Arcades.MAME
|
|||
{
|
||||
return _peek(addr, firstOffset, size);
|
||||
},
|
||||
read == "rom" ? (Action<long, byte>) null : (addr, val) => _poke(addr, val, firstOffset, size),
|
||||
read == "rom"
|
||||
? null
|
||||
: (long addr, byte val) => _poke(addr, val, firstOffset, size),
|
||||
dataWidth));
|
||||
}
|
||||
}
|
||||
|
|
|
@ -213,15 +213,23 @@ namespace BizHawk.Emulation.Cores.Nintendo.GBHawk
|
|||
break;
|
||||
|
||||
case 0xFF76:
|
||||
byte ret1 = audio.SQ1_output >= Audio.DAC_OFST ? (byte)(audio.SQ1_output - Audio.DAC_OFST) : 0;
|
||||
byte ret2 = audio.SQ2_output >= Audio.DAC_OFST ? (byte)(audio.SQ2_output - Audio.DAC_OFST) : 0;
|
||||
var ret1 = audio.SQ1_output >= Audio.DAC_OFST
|
||||
? (byte) (audio.SQ1_output - Audio.DAC_OFST)
|
||||
: (byte) 0;
|
||||
var ret2 = audio.SQ2_output >= Audio.DAC_OFST
|
||||
? (byte) (audio.SQ2_output - Audio.DAC_OFST)
|
||||
: (byte) 0;
|
||||
if (is_GBC) { ret = (byte)(ret1 | (ret2 << 4)); }
|
||||
else { ret = 0xFF; }
|
||||
break;
|
||||
|
||||
case 0xFF77:
|
||||
byte retN = audio.NOISE_output >= Audio.DAC_OFST ? (byte)(audio.NOISE_output - Audio.DAC_OFST) : 0;
|
||||
byte retW = audio.WAVE_output >= Audio.DAC_OFST ? (byte)(audio.WAVE_output - Audio.DAC_OFST) : 0;
|
||||
var retN = audio.NOISE_output >= Audio.DAC_OFST
|
||||
? (byte) (audio.NOISE_output - Audio.DAC_OFST)
|
||||
: (byte) 0;
|
||||
var retW = audio.WAVE_output >= Audio.DAC_OFST
|
||||
? (byte) (audio.WAVE_output - Audio.DAC_OFST)
|
||||
: (byte) 0;
|
||||
if (is_GBC) { ret = (byte)(retN | (retW << 4)); }
|
||||
else { ret = 0xFF; }
|
||||
break;
|
||||
|
|
|
@ -23,7 +23,9 @@ namespace BizHawk.Emulation.Cores.Nintendo.GBHawk
|
|||
}
|
||||
else
|
||||
{
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8) ? 0xFF : Core.bus_value;
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8)
|
||||
? (byte) 0xFF
|
||||
: Core.bus_value;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
@ -58,12 +58,16 @@ namespace BizHawk.Emulation.Cores.Nintendo.GBHawk
|
|||
}
|
||||
else
|
||||
{
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8) ? 0xFF : Core.bus_value;
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8)
|
||||
? (byte) 0xFF
|
||||
: Core.bus_value;
|
||||
}
|
||||
}
|
||||
else
|
||||
{
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8) ? 0xFF : Core.bus_value;
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8)
|
||||
? (byte) 0xFF
|
||||
: Core.bus_value;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
@ -58,13 +58,17 @@ namespace BizHawk.Emulation.Cores.Nintendo.GBHawk
|
|||
}
|
||||
else
|
||||
{
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8) ? 0xFF : Core.bus_value;
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8)
|
||||
? (byte) 0xFF
|
||||
: Core.bus_value;
|
||||
}
|
||||
|
||||
}
|
||||
else
|
||||
{
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8) ? 0xFF : Core.bus_value;
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8)
|
||||
? (byte) 0xFF
|
||||
: Core.bus_value;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
@ -69,7 +69,9 @@ namespace BizHawk.Emulation.Cores.Nintendo.GBHawk
|
|||
}
|
||||
else
|
||||
{
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8) ? 0xFF : Core.bus_value;
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8)
|
||||
? (byte) 0xFF
|
||||
: Core.bus_value;
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -85,7 +87,9 @@ namespace BizHawk.Emulation.Cores.Nintendo.GBHawk
|
|||
}
|
||||
else
|
||||
{
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8) ? 0xFF : Core.bus_value;
|
||||
return Core.cpu.TotalExecutedCycles > (Core.bus_access_time + 8)
|
||||
? (byte) 0xFF
|
||||
: Core.bus_value;
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue