Четыре разбора с прогонов пользователя; три бага закрыты, четвёртый (BUG-SPIKE-1, пики) заведён с замером и гипотезой. BUG-JUMPWALL-1 (Critical) — недолетевший прыжок проходил СКВОЗЬ стену. check_collisions держал флаги перекрытия ОДНОГО ряда, а оригинал (seg004:0004) — трёх, и move_coll_to_prev (seg004:00DF) берёт «прошлые» флаги из нужного. Наш prev=3 («уже перекрывал») подавлял бамп ровно на кадре смены ряда, а в падении ряд меняется почти каждый кадр — переход 0→1 на стене приходился как раз на него. Порт трёх рядов дословно. Воспроизведено и закрыто на харнессе (новый набор t_wall: свип по 20 стартовым X, 4 давали проход сквозь кладку); 1723 трассы t_phys НЕ изменились — правка поведение-сохраняющая. Живьём подтвердил пользователь. BUG-GUARD-DEAF-1 (Major) — is_guard_notice не взводился НИГДЕ, поэтому неактивный страж не оборачивался на Кида за спиной никогда. Портированы все пять мест оригинала: опкод SOUND в play_seq (звуки 0..2), bumped_sound, мягкое/среднее приземление, обрушенная плита, щелчок кнопки. Ждёт игровой проверки боем в комнате 11 уровня 2. BUG-SEAM-WEDGE-1 — клин кладки в пустом (2,0). Сосед угла снизу-слева лежит в комнате по диагонали (room_BL); мы безусловно считали его стеной, оригинал (load_rowbelow, seg008:368) — только когда такой комнаты нет. Ряд «снизу» стал 11-байтным: [10] = тайл (0,9) диагональной комнаты. BUG-LOOSE-3 — чёрный бар под упавшей плитой-потолком. pop_ceil_bake_empty стирал полосу и восстанавливал только два тайла ряда −1, а в полосу лезет графика соседа слева и верхушки ряда 0. Теперь перерисовываются ряды −1 и 0, колонки col−1..col+1. Проверено попиксельной сверкой с эталонной перерисовкой: 0 различий. Плюс карта связности комнат уровней 1–3 (TASKS.md): на ур. 2 недостижимых нет, на ур. 3 это 23 и 24 — те же односторонние ссылки, что дали 13/18/24 на уровне 1, только комнаты полностью пустые. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
tests-host — модульные тесты движка roomtest
Прогоняются под ucsim_z80, обвязка общая — testkit/ (там же объяснение,
почему под z80, а не хостовым gcc, и грабли).
make host-tests # из корня, вместе с остальными
make -C applications/PoP/roomtest/tests-host # только эти
make -C applications/PoP/roomtest/tests-host t_geom
Что покрыто
| набор | модуль | что проверяет |
|---|---|---|
t_geom |
pop_geom.c |
геометрия комнаты (x_bump, y_land, y_to_row) и PRNG оригинала |
t_phys |
pop_map.c + pop_kid.c + pop_ctrl.c |
покадровые трассы физики Кида: стойка, бег, шаг с Shift, разворот, присед, прыжок вверх, падение с кромки, loose-плита, разбег-прыжок через провал |
Главный тест в t_geom — geom_lcg_matches_reference. LCG оригинала
(s = s*214013 + 2531011) написан в pop_geom.c на ассемблере по схеме
Горнера ради обхода __mullong, и заявка «бит-в-бит как в SDLPoP» до
появления теста держалась только на комментарии. Тест сверяет рукописный
asm с наивной 32-битной формулой на 128 шагах — и по возвращаемому
значению, и по обеим половинам сида.
t_phys: характеризация, а не проверка правильности
Набор существует ради страховки под рефакторинг. Эталоны сняты с
текущего билда, то есть консервируют и текущие баги — правильность
по-прежнему проверяется сверкой с SDLPoP/src/. Ценность в другом:
ближайший крупный шаг (перенос физики с Kid. на Char., чтобы её мог
переиспользовать страж) обязан быть поведение-сохраняющим, и «трасса до
== трасса после» ловит ровно тот класс ошибок, который там возможен.
Поэтому эталон переснимается только осознанно, и в коммите должно быть написано, что и почему поменялось. Молча обновлённый эталон обесценивает весь набор.
python3 record_golden.py # переснять все трассы
python3 record_golden.py 7 9 # только сценарии 7 и 9
Трасса — по одной записи на кадр: frame, x, y, dir, col, row, action и
смещение curr_seq от SEQTBL_BASE. Кадр гоняется ровно в том порядке,
что и в главном цикле roomtest.c:
pop_ctrl_tick(); // ввод -> control(): смена последовательности
kid_tick(); // play_seq: следующий кадр
pop_phys_tick(); // падение/приземление/стена
pop_loose_tick(); // досчёт тряски и снятие провалившейся плиты
Покадровость здесь принципиальна: порт seg005/seg006 — конечный автомат, и
ошибка почти всегда проявляется не в одной функции, а в расхождении
траектории через несколько кадров. Плюс главный риск переноса на Char —
не арифметика, а то, кто владеет окном Char внутри кадра; поймать это
можно только прогоном полного кадра.
Проверка, что харнесс воспроизводит устройство: сценарий stand даёт
frame=15, x=114, y=118, curr_seq=0x19A3 — те же значения, что читаются из
_Kid в живом MAME.
Как это заработало под ucsim
Три шва, каждый закрыт без правок продуктового кода:
- Таблицы анимации.
pop_kid.cходит по ним абсолютными адресами отKD_DATA_OFF(0x100) — там, куда на устройстве их приводит маппинг W0. В тестеkid_data.binпросто кладётся по 0x100 (gen_kid_blob.py), аgfx_w0_mapзаглушен. Следствие: код набора линкуется с--code-loc 0x1000(CODE_LOCв Makefile), чтобы не налезть на блоб. - Окружение.
stubs.c— libc/libbgi (графика, память, файлы), соседние модули (pop_bg/pop_trob/pop_level/pop_redraw) и состояние стража. Заглушки не пустые там, где это меняет смысл: файловое чтение реально отдаётkid_data.bin(иначеpop_kid_data_loadтихо сдаётся,kdat_okостаётся нулём иplay_seqне делает ничего — трасса выходит из одного застывшего кадра), клавиатура отвечает по набору «нажатых» скан-кодов (так под тест попадаютread_inputиread_user_control, а не только диспетчер), а обращения к соседям журналируются (tk_log) — можно проверять «плита отвалилась и попросила перерисовку», а не только координаты. __banked. Модули физики помечены как банковые, SDCC генерирует на них трамплин___sdcc_bcall_ehl, которому нужныset_bank/get_bank. В тесте память плоская —bank_stub.sотдаёт нулевой банк и пустое переключение, соблюдая контракт по регистрам.
Что харнесс НЕ покрывает и остаётся за MAME: отрисовка и окклюзия, реальные переключения банков, тайминги кадра, настоящая клавиатура.
Что нужно, чтобы двинуться дальше
- Доступ к странице уровня.
pop_level.cходит по абсолютным адресам ((uint8_t *)(LVL_DATA_OFF + …)послеgfx_w0_map). Дляpop_mapэто не потребовалось — карту комнаты в него ИНЪЕКТИРУЮТ черезpop_map_set, — но набор на самpop_level/pop_trobв это упрётся. Решение то же, что дляkid_data: положить страницу по нужному адресу и поднятьCODE_LOC. - Сценарии из
bug_closed.md. У каждой закрытой записи есть симптом и ожидаемое поведение — готовый список регрессионных кейсов, который стоит переносить вt_physпо мере надобности. - BUG-LOOSE-2 (гонка «уйти из комнаты раньше, чем долетит плита») — через мост MAME воспроизвести не удалось, а на уровне логики это несколько строк: заспавнить кусок, сменить комнату, тикать до приземления, проверить щебень.