112. Ror página cero

añadir el opcode ROR de página cero ``0x66``.

Lección 112 de 356 · tests/chapter_01_cpu/test_112_ROR_zero_page.py

En este paso, tras la lección 111, añade `ror a las importaciones de instrucciones en emulator/cpu/opcodes.py` y añade el handler de página cero y la entrada en la tabla.

Por qué existe este paso

Esto conecta la operación de memoria ROR compartida con la primera forma compacta de direccionamiento, demostrando que una dirección de página cero codificada selecciona el byte que se va a rotar.

Implementación sugerida

def ror_zero_page(cpu: CPU):
    addr = zero_page(cpu)
    ror(cpu, addr)

OPCODE_TABLE = {
    ...
    0x66: ror_zero_page,
}

`zero_page ya existe en emulator/cpu/addressing_modes.py y captura un byte. ror ya existe en emulator/cpu/instructions.py; realiza la lectura/modificación/escritura en el bus, rota a través del Carry antiguo, fija Carry a partir del bit 0 antiguo y fija Zero/Negative a partir del resultado. Así, 66 10 modifica cpu.bus[$0010]`, no A, y PC avanza dos bytes en total.

Concepto erróneo: el `0x10 capturado es una dirección, no el valor que se pasa a ror`. Fuera del alcance: los modos ROR indexado y absoluto son las lecciones 113-115; no dupliques la lógica de direccionamiento o de rotación en este handler de opcode.

Ejecutar esta lección

uv run pytest tests/chapter_01_cpu/test_112_ROR_zero_page.py -v

Implementación de ejemplo

Implementación sugerida

def ror_zero_page(cpu: CPU):
    addr = zero_page(cpu)
    ror(cpu, addr)

OPCODE_TABLE = {
    ...
    0x66: ror_zero_page,
}

Ver código del testtest_112_ROR_zero_page.py

import inspect

from emulator.bus.cpu_bus import CpuBus
from emulator.cpu import opcodes
from emulator.cpu.cpu import CPU
from emulator.memory.fake_rom import FakeROM


CARRY_FLAG = 1 << 0


def make_cpu_with_rom():
    rom = FakeROM()
    rom.write(0x7FFC, 0x00)
    rom.write(0x7FFD, 0x80)
    bus = CpuBus(program_rom=rom)
    return CPU(bus), bus, rom


def test_ror_zero_page_handler_exists_and_is_in_opcode_table():
    """Objective: create ror_zero_page(cpu) and add 0x66 to OPCODE_TABLE."""
    assert hasattr(opcodes, "ror_zero_page")
    assert callable(opcodes.ror_zero_page)
    assert list(inspect.signature(opcodes.ror_zero_page).parameters) == ["cpu"]
    assert opcodes.OPCODE_TABLE[0x66] is opcodes.ror_zero_page


def test_opcode_66_ror_zero_page_rotates_memory_value():
    """Objective: 66 10 means ROR $10, so RAM[$0010] is rotated right."""
    cpu, bus, rom = make_cpu_with_rom()
    rom.write(0x0000, 0x66)
    rom.write(0x0001, 0x10)
    bus.write(0x0010, 0b0000_0110)

    cpu.reset()
    cpu.step()

    assert bus.read(0x0010) == 0b0000_0011
    assert cpu.pc == 0x8002


def test_opcode_66_ror_zero_page_uses_old_carry_as_new_bit_7():
    """Objective: old Carry=1 is inserted into memory result bit 7."""
    cpu, bus, rom = make_cpu_with_rom()
    rom.write(0x0000, 0x66)
    rom.write(0x0001, 0x10)
    bus.write(0x0010, 0b0000_0010)

    cpu.reset()
    cpu.p |= CARRY_FLAG
    cpu.step()

    assert bus.read(0x0010) == 0b1000_0001
    assert (cpu.p & CARRY_FLAG) == 0


def test_opcode_66_ror_zero_page_sets_carry_from_old_bit_0():
    """Objective: old memory bit 0 becomes Carry."""
    cpu, bus, rom = make_cpu_with_rom()
    rom.write(0x0000, 0x66)
    rom.write(0x0001, 0x10)
    bus.write(0x0010, 0b0000_0011)

    cpu.reset()
    cpu.step()

    assert bus.read(0x0010) == 0b0000_0001
    assert (cpu.p & CARRY_FLAG) != 0

Funciones de test (4)

test_ror_zero_page_handler_exists_and_is_in_opcode_table
test_opcode_66_ror_zero_page_rotates_memory_value
test_opcode_66_ror_zero_page_uses_old_carry_as_new_bit_7
test_opcode_66_ror_zero_page_sets_carry_from_old_bit_0