change 255 to unused mapper id 1000

Author: zohassadar

build.js

@@ -12,7 +12,7 @@ const mappers = { // https://www.nesdev.org/wiki/Mapper
     3: 'CNROM',
     4: 'MMC3',
     5: 'MMC5',
-    255: 'Autodetect MMC1/CNROM',
+    1000: 'Autodetect MMC1/CNROM',
 };
 
 // options handling
@@ -51,7 +51,7 @@ console.log(`using ${nativeCC65 ? 'system' : 'wasm'} ca65/ld65`);
 
 // mapper options
 
-const mapper = args.find((d) => d.startsWith('-m'))?.slice(2) ?? 255;
+const mapper = args.find((d) => d.startsWith('-m'))?.slice(2) ?? 1000;
 
 if (!mappers[mapper]) {
     console.error(

src/constants.asm

@@ -1,5 +1,5 @@
 .ifndef INES_MAPPER ; is set via ca65 flags
-INES_MAPPER := 255 ; 1 (MMC1), 3 (CNROM), 4 (MMC3), 5 (MMC5),  and 255 (autodetect 1/3)
+INES_MAPPER := 1000 ; 1 (MMC1), 3 (CNROM), 4 (MMC3), 5 (MMC5),  and 1000 (autodetect 1/3)
 .endif
 
 .ifndef SAVE_HIGHSCORES

src/header.asm

@@ -12,8 +12,8 @@
 INES_MIRROR = 0 ; 0 = horizontal mirroring, 1 = vertical mirroring (ignored in MMC1)
 INES_SRAM   = 1 ; 1 = battery backed SRAM at $6000-7FFF
 
-; Override INES_MAPPER for mode 255 (auto detect)
-.if INES_MAPPER = 255
+; Override INES_MAPPER for mode 1000 (auto detect)
+.if INES_MAPPER = 1000
     .if CNROM_OVERRIDE
     _INES_MAPPER = 3 ; Test CNROM on Emulator/Flashcart
     .else

src/io.asm

@@ -62,7 +62,7 @@ MMC5_CHR_BANK0 := $5123 ; 4kb page index
 MMC5_CHR_BANK1 := $5127
 
 .macro RESET_MMC1
-.if INES_MAPPER = 1 .or INES_MAPPER = 255
+.if INES_MAPPER = 1 .or INES_MAPPER = 1000
 :       inc :-  ; increments inc ($aa), writing a negative value to prg
                 ; https://www.nesdev.org/wiki/MMC1#Reset
 .endif

src/main.asm

@@ -55,7 +55,7 @@ mainLoop:
 .include "util/menuthrottle.asm"
 .include "util/modetext.asm"
 .include "util/mapper.asm"
-.if INES_MAPPER = 255
+.if INES_MAPPER = 1000
 .include "util/autodetect.asm"
 .endif
 

src/ram.asm

@@ -210,7 +210,7 @@ dasOnlyShiftDisabled: .res 1 ; $63A
 
 invisibleFlag: .res 1 ; $63B  ; 0 for normal mode, non-zero for Invisible playfield rendering.  Reset on game init and game over.
 
-mapperId: .res 1 ; $63C ; For INES_MAPPER 255 (autodetect).  0 = CNROM.  1 = MMC1.  
+mapperId: .res 1 ; $63C ; For INES_MAPPER 1000 (autodetect).  0 = CNROM.  1 = MMC1.  
 
     .res $38
 

src/reset.asm

@@ -53,7 +53,7 @@ reset:  cld
 
 mapperInit:
 ; autodetect
-.if INES_MAPPER = 255
+.if INES_MAPPER = 1000
         setMMC1PRG ; initialize mmc1 just in case
         ; cnrom can pass one of these tests but not both.
         ; Start with the one it's supposed to fail.

src/util/mapper.asm

@@ -47,7 +47,7 @@ changeCHRBanks:
         sta     generalCounter
 
 ; autodetect
-.if INES_MAPPER = 255
+.if INES_MAPPER = 1000
         ldx     mapperId
         beq     @cnrom
         changeCHRBanksMMC1
@@ -113,7 +113,7 @@ changeCHRBanks:
 
 setHorizontalMirroring:
 ; autodetect
-.if INES_MAPPER = 255
+.if INES_MAPPER = 1000
         lda mapperId
         beq @cnrom
         lda #%10011
@@ -144,10 +144,10 @@ setHorizontalMirroring:
         rts
 
 setVerticalMirroring:
-; Unused except during mapper detect for INES_MAPPER 255
+; Unused except during mapper detect for INES_MAPPER 1000
 
 ; autodetect
-.if INES_MAPPER = 255
+.if INES_MAPPER = 1000
         lda mapperId
         beq @cnrom
         lda #%10010
@@ -177,7 +177,7 @@ setVerticalMirroring:
 .endif
         rts
 
-.if INES_MAPPER = 3 .or INES_MAPPER = 255
+.if INES_MAPPER = 3 .or INES_MAPPER = 1000
 ; bus conflict workaround
 cnromBanks:
         .byte $00,$01