Step 0425: Spec-Correct JOYP + Address Wrap (Remove Hacks)

Implementación

Archivos Modificados

1. Hardware Core (C++)

• src/core/cpp/Joypad.cpp:
  • Constructor: Inicialización a 0xCF (bits 4-5 = 00, spec-correct)
  • read_p1(): Eliminada inversión de bits 4-5, retornar (p1_register_ & 0x30) directamente
  • Caso especial: Si ninguna fila seleccionada, nibble = 0xF
• src/core/cpp/Joypad.hpp: Comentarios actualizados para reflejar comportamiento spec-correct
• src/core/cpp/MMU.cpp:
  • Línea 935: Eliminado bypass test_mode_allow_rom_writes_
  • Línea 1068: Eliminado bypass ROM_ONLY cuando rom_data_.empty()
  • Línea 3564: Eliminado método set_test_mode_allow_rom_writes()
• src/core/cpp/MMU.hpp:
  • Eliminado flag test_mode_allow_rom_writes_
  • Eliminada declaración de set_test_mode_allow_rom_writes()

2. Wrapper Cython

• src/core/cython/mmu.pyx: Eliminado método set_test_mode_allow_rom_writes()
• src/core/cython/mmu.pxd: Eliminada declaración de set_test_mode_allow_rom_writes()

3. Tests (Python)

• tests/conftest.py: Eliminado fixture mmu_romw
• tests/test_core_joypad.py:
  • 8 tests actualizados con valores spec-correct
  • Dirección con bit4=0: Esperado 0xEE (antes 0xDE)
  • Acción con bit5=0: Esperado 0xDE (antes 0xEE)
  • Justificación: Pan Docs especifica que bits 4-5 NO se invierten
• tests/test_mmu_rom_is_readonly_by_default.py:
  • 4 tests actualizados para usar load_rom_py()
  • Eliminado uso de fixture mmu_romw
  • Test de load_rom: Preparar ROM de 512 bytes, cargar con load_rom_py(bytes(custom_rom))
• tests/test_core_mmu.py:
  • 1 test actualizado: test_mmu_address_wrapping
  • Cambio: Validar wrap con WRAM (0xC000) en lugar de ROM (0x0000)
  • Justificación: ROM es read-only, no se puede validar wrap escribiendo en ROM

Fragmentos de Código Clave

JOYP read_p1() Spec-Correct (Joypad.cpp)

uint8_t Joypad::read_p1() const {
    // Empezar con bits 0-3 a 1 (todos sueltos por defecto)
    uint8_t nibble = 0x0F;
    
    // Selección de fila según Pan Docs: bit=0 selecciona
    bool direction_row_selected = (p1_register_ & 0x10) == 0;
    bool action_row_selected = (p1_register_ & 0x20) == 0;
    
    if (direction_row_selected) {
        nibble &= direction_keys_;
    }
    
    if (action_row_selected) {
        nibble &= action_keys_;
    }
    
    // Caso especial: si ninguna fila está seleccionada
    if (!direction_row_selected && !action_row_selected) {
        nibble = 0x0F;
    }
    
    // Construir resultado spec-correct (SIN inversión bits 4-5)
    uint8_t result = 0xC0 | (p1_register_ & 0x30) | (nibble & 0x0F);
    
    return result;
}

ROM Read-Only Spec-Correct (MMU.cpp)

// En MMU::write() - Switch case para ROM (0x0000-0x7FFF)
case MBCType::ROM_ONLY:
default:
    // Step 0425: ROM es SIEMPRE read-only (spec-correct según Pan Docs).
    // Las escrituras en ROM se ignoran (o se interpretan como MBC).
    // NO permitir escrituras incluso si rom_data_ está vacío.
    return;