Skip to content

Commit 2bb5608

Browse files
shicksbfirsh
authored andcommitted
Fix bug in Mapper.load1kVromBank
This is used by MMC3 and MMC5. I haven't actually noticed any issues while playing, though I suspect it's because *if* the MMC3 game I've been playing (Crystalis) even uses this function at all, it gets by with only using the Tile data, which is being copied correctly.
1 parent db29fa1 commit 2bb5608

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/mappers.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -442,9 +442,9 @@ Mappers[0].prototype = {
442442
var bankoffset = (bank1k % 4) * 1024;
443443
utils.copyArrayElements(
444444
this.nes.rom.vrom[bank4k],
445-
0,
446-
this.nes.ppu.vramMem,
447445
bankoffset,
446+
this.nes.ppu.vramMem,
447+
address,
448448
1024
449449
);
450450

0 commit comments

Comments
 (0)