05 июля 2015 |
|
Баги: практика Теперь перейдём собственно к "типичным ошибкам". Без всяких высасываний из пальца прямо разберём все ошибки,которые возника─ ли при написании и серии оптимизаций эффе─ кта"ball record"в New View 48K. Это, ко─ нечно,не большая программа со сложным раз─ ветвлённым функционалом, но я хотел бы по─ смотреть на того, кто сможет так же подро─ бно документировать процесс разработки че─ го-то более крупного :) Начато03.10.2014. Этапы - это внесён─ ные изменения, не имеющие отношения к баг─ фиксам.Баги пронумерованы,для каждого бага указано, как он проявился и как найден (иногда это одно и то же). Разработка про─ изводилась в SjAsmPlus с автосборкой (из─ начально в TRD) с помощью упаковщика mhmt и утилиты trdtool. Я всегда использую ав─ тосборку,чтобы не отлаживать отдельно сна─ пшот, а потом писать лоадер и заново отла─ живать. Первая пустая сборка сделана из исходников Chaos Zoomer 1K. - (этап 1)Нарисовал спрайт шарика. - (этап 2)Графика вручную перенесена в нолики и единички. Написаны простые проце─ дуры вывода спрайтов. Протестированы на координатах от балды(y+=9,x++). 1. В середине маски была точка (случайно поставилась цифра уже после рисования???)- заметил только при перемещении спрайта. - (этап 3)Процедура вывода заменена на набор из 8x8 генерируемых выводилок (в #c000..#ffff - для каждой комбинации x mod 8, y mod 8), генератор выводилок - genprspr. 2. Забыл инициализировать hl перед циклом вызова genprspr - нашёл трассировкой. 3. jr z вместо jr nz в genprbyte (часть genprspr) - нашёл просмотром сгенерённого кода. 4. Забыл случай, когда после маски ничего не накладывается - нашёл просмотром сгене─ рённого кода. Оптимизировал. 5. Забыл случай, когда маска перекрывает весь байт - нашёл просмотром сгенерённого кода (и первый раз поставил переход непра─ вильно, второй раз баг из-за того, что пе─ ренёс ветку из другого места). 6. Не работал вывод по зигзагу - нашёл первым запуском сгенерённого кода (и пер─ вый раз неправильно написал метку, второй раз не сохранил,третий раз зигзаг не в той фазе - торопился исправить и исправлял раз за разом неправильно,но эти циклы шли один за другим и очень быстро). 7. Не работал down hl в сгенерённом коде (ld a,l вместо ld l,a, ошибка copy-paste)- нашёл первым запуском сгенерённого кода. - (этап 4)В генераторе выводилок сдела─ ны ветки не только дляand:or(реальноor: xor),но и дляor,xor,and.Спрайт инверти─ рован для удобства написания всех веток. 8. Ошибочно поставил единицы вне маски (потому что инвертировал маску и спрайт автозаменой) - нашёл печатью спрайтов без фона (до этого печатал поверх копии ПЗУ). - (этап 5) В генераторе выводилок до─ бавлен оптимизатор вывода одного бита (genprsprres). 9. В genprsprres был пропущен inc hl (скопировал не целиком из другой ветки - ошибка copy-paste) - нашёл просмотром сге─ нерённого кода. - (этап 6)В генераторе выводилок добав─ лен оптимизатор конструкций типаinc l: down hl:dec l(removeЗinc). 10. В removeЗinc не обслужена одна ветка выхода - нашёл запуском выводилки с пара─ метрами. (С этого момента правильно заработала вы─ водилка.) - (этап 7) Написан генератор восстанав─ ливалок (стиралок) genrespr (на основе генератора выводилок) - стиралки по форме спрайта,черезldi/lddиз копии экрана, 8x8 штук в#8000..#bfff.В связи с этим актив─ ная часть программы отделена от инициали─ зации и помещена в#7800,а в#6000поме─ щена копия экрана. При этом пришлось ис─ правлять адреса в депакере (depkmain.asm). 11. В genprsprxor (ветка genprspr, реаль─ но не используется) пропущено inc c - на─ шёл просмотром кода, когда делал из него другую процедуру. 12. Обвязку распаковки depkmain.asm ис─ правил не в том каталоге (другая панель TotalCmd) - программа не запустилась. 13. Забыл скопировать экран для восстана─ вливалки - нашёл запуском восстанавливал─ ки. 14. Забыл присвоить de для восстанавлива─ лки - нашёл просмотром исходника. (Прошёл час и 40 минут после того, как заработала выводилка; с этого момента за─ работала стиралка, уже 858 строк.) 15. Вместо ld a,(hl):ld (de),a стояло ld a,(de):ld (hl),a - увидел после фикса пре─ дыдущего бага (баг был и в трёх комментах, хотя рядом было написано ldi,ldd). - (этап 8) Написана обвязка вывода по синусу (mainprO) и обвязка стирания (mainreO) через использование значений, рассчитанных обвязкой вывода (черезix= COORDS+n). Число шариков занесено в конс─ танту BALLS.Проверяю работоспособность и число тактов на тесткейзах приnomove=1 (т.е. не двигать спрайты). 16. Вместо ld ix,COORDS+128 (работа через -128,+127) стояло ld ix,COORDS - программа вывалилась. 17. Пропущена метка BALLS (хотя откомпи─ лировалось) - программа сбросилась. 18. jp nz,mainreO вместо jp nz,mainprO (ошибка copy-paste) - программа сброси─ лась. 19. А надо было jp p и там, и там (т.к. использовалось ix+128) - глючило. Переде─ лал на ix, ix-BALLS, ix-(BALLS*2). (Через 50 минут после того,как заработала стиралка; один баг на >45 строк.) - (этап 9)Оптимизировал выводилку через пересчёт таблицы синусов дляXв значения, нужные для вывода:x/8иf(x&7)=8*(x&7)+ #c0(циклrexsinO), а таблицы синусов для Y в таблицу экранных адресов этихY(цикл reysinO). До этого использовалась про─ цедура расчёта экранных координат в ПЗУ (8881). 20. Было COORDS=#8Off-BALLS вместо #8100- BALLS - глючило. 21. ld (ix+),a после оптимизации было не там - глючило. (Прошёл ещё час, на этот момент написано 1100 строк - один баг на 50 строк, включая №24.) - (этап 10)Смотрел скорость при разных рисунках шариков. Выбрал самый быстрый ва─ риант. Второй день: - (этап 11)Добавлял оптимизатор стирал─ ки -genreopt(запускается послеgenrespr, исправляет уже сгенерированную процедуру - ld a,(hl):ld (de),a:inc l:inc e заменяется наldi,и ld a,(hl):ld (de),a:dec l:dec e заменяется наldd). 22. В genreopt не был сделан пропуск inc l:inc e - программа висла (и первый раз неправильно исправил, оставил ld a,#ed наряду с ld (hl),#ed ). 23. В genreoptn (ветка genreopt) был про─ пущен ld a,(hl) перед сравнением (прежде─ временная оптимизация) - программа сбрасы─ валась (и первый раз скомпилил исправле─ ние, не сохранив). - (этап 12)Добавил компиляцию в tap. Через 2 дня: - (этап 13) Оптимизировал обвязку рисо─ валки и стиралки через хранение параметров цикла в регистрах, которые не портятся при вызове. 24. В rexsinO использовался l вместо (hl) - шарики дёргались (первый раз понял при─ чину неправильно, потом исправил неправи─ льно). 25. Добавил ld c,0 после ld ($+3+2),a - висло. Если бы было сделано через метку, глюка бы не было. - (этап 14) Оптимизировал рисовалку и стиралку через константы в регистрах. 26. Изменил prdhlsize, но забыл исправить в removeЗreinc (в removeЗinc исправил) - артефакты на экране (и первый раз исправил неправильно, т.к. надо в respr свой dhl - портится de). [Ложный баг:сначала оптимизация dhl через константы казалась проигрышной (забыл за─ ранее посчитать, нашёл просмотром тактов в отладчике), в итоге потратил время на №26; но впоследствии в новом коде этот новый вариант dhl выиграл.] 27. Добавил jr z,genprsprorxorconst_c вместо genprsprorxor2const_c (ошибка copy- paste) - висло при const_b=0. 28. Написал 'xsin вместо xsin/256 - SjAsm не скомпилил. Через несколько дней: - (этап 15)Написал вывод фона (до этого фон был из ПЗУ). 29. Была опечатка: pgbg2, pgbgЗ вместо prbg2,prbgЗ (причём prbgO,prbg1 было напи─ сано правильно) - ошибка компиляции, но запустилось с неправильной фоновой картин─ кой. Ещё через несколько дней: - (этап 16) Оптимизировал обвязку рисо─ валки и стиралки через сохранение парамет─ ров в стек, а вызов через ret:pop hl(это экономит регистры на параметры цикла и константы отрисовки). 30. В genprspror ошибочно был переход на genprsprorxorconst_c - вместо нужного genprsprorconst_c (только один из трёх ря─ дом стоящих неверный!) - нашёл по ошибке компилятора (длинный jr), но всё работало! 31. Был ещё один длинный jr на genprspror - нашёл по ошибке компилятора,но всё рабо─ тало! При оптимизации: - (этап 17) Написал другой вариант сти─ ралок, которые генерируются прямо из текс─ туры фона (настройкаbgtype=2),им не нуж─ на копия экрана в#6000.Процедура называ─ етсяgenrebg. 32. Комментарий (e&7)*8+#c0 вместо (x&7)* 8+#c0 - нашёл при изменении исходника.Этот баг в комментарии был бы очень чреватым при последующих изменениях. 33. Удалил вызов genreopt, но забыл уда─ лить вызов genrespr - сбрасывалось. 34. Перед genrebgsO написал ld hl,PROGS вместо ld hl,PROGS-#4000 - висло с артефа─ ктами на экране. 35. Было jr nz,genrebgs1 вместо genrebgsO (ошибка copy-paste) - висло с артефактами на экране. 36. ld a,b:dec a:call nz,dde вместо call dde:ld a,b:dec a:call nz,genrebgdhl (пер─ вый раз исправил наоборот). 37. IMVEC и IMER затирались новыми длин─ ными процедурами rebg - сбрасывалось. Пе─ ренёс в другие адреса (где выводилки). 38. Забыл в genrebg в начале сгенериро─ вать pop hl - сбрасывалось. 39. Забыл push hl в новой ветке mainreO (запуск стиралок из главного цикла)- сбра─ сывалось. 40. Размеры процедур rebg могут превышать #80 байт (на 1 байт) - нашёл спецтестом при генерации, исправил фоновую графику (#fe на #ff в углу). - (этап 18) Пробовал сделать восстанав─ ливалку черезpush(bgtype=0)- невыгодно (нужно маленькое окно), но пригодилось для следующих изменений. 41. Когда сделал три варианта bgtype, по─ ставил if..else без endif, не компилирова─ лось (unexpected end of file) - нашёл вы─ кидыванием частей исходника (оборачивание if..endif не помогает). (На тот момент написано 1436 строк - один баг на 35 строк.) Ещё через время: - (этап 19) Новый скоростной расчёт па─ раметров для вывода (pop hl:ld bc:add hl, bc:ld (),hl- причём в стеке для стирания патчится не слово адреса процедуры или слово адреса экрана, а по одному байту от того и того). Для этого всеYдля шариков зациклены,для каждой из 8 фаз входа в таб─ лицу Yгенерируются "скрипты" (в#6000), где Y сортируются сверху вниз. ТаблицаX копируется в стек с циклическим сдвигом. Настройкаrex=1. 42. При генерации таблиц для Y брались не синусы, а пересчитанные в адреса (поверх того же места в памяти) - сбрасывалось (нашёл трассировкой и просмотром памяти). 43. Забыл присвоить sp при входе в новый расчёт параметров вывода - сбрасывалось (нашёл трассировкой и просмотром памяти). 44. Вместо PROGS&255 в новом расчёте сто─ яло 0 - висло (нашёл трассировкой). 45. После нового расчёта не присваивались регистры перед вызовом шариков - сбрасыва─ лось (нашёл трассировкой). [Ложный баг: для нового расчёта после со─ ртировки шариков по Y надо по идее читать X-координаты не подряд, а в порядке сорти─ ровки (нашёл, когда начал добавлять сор─ тировку). Но потом отсортировал запись (не чтение) по порядку X и вернул чтение под─ ряд.] 46. Когда добавлял номер шарика в порядке сортировки (т.е. 2 байта вместо 1 в списке y), забыл пропустить этот байт при состав─ лении стека - артефакты на экране, нашёл постепенным возвратом изменений,первый раз исправил неправильно (пропустил не тот байт). 47. При добавлении сортировки поставил jr nc,fillreysortskip ;skip if (hl)>=min вме─ сто jr c,fillreysortskip ;skip if (hl)>min - сбрасывалось,нашёл трассировкой. (С этого момента на Скорпионе всё выводи─ тся без резания лучом. 32 шарика для круг─ лого счёта, чтобы y=ysin[i],i+=8. Пока что используется стиралка через push.) - (этап 20)Оптимизировал стиралку через push- на1,2,7строках чётных знакомест всё заливаем одним регистром,т.к.в графике там нули или ff-ки. 48. При оптимизации стиралки один jr z не компилировался, но всё запускалось - нашёл трассировкой сгенерированной стиралки и просмотром исходника. - (этап 21)Придумал, как быстро генери─ ровать параметры для старой стиралки (ге─ нерируемой из фона).Чтобы вернуться к ней, стал тестировать условную компиляцию с разными комбинациямиrexиbgtype- убеди─ ться,что всё ещё работает перед написанием нового расчёта параметров для стиралки. 49. В условной компиляции в двух местах стояло if trex==0 вместо if rex==0 - выва─ ливалось, нашёл при возврате к старому ме─ тоду стирания. 50. reysin (перекодирование таблицы Y) была выброшена за пределы процедур (возв─ ратил на место) - сбрасывалось,нашёл трас─ сировкой и просмотром содержимого таблицы. - (этап 22) Приблизил адресацию восста─ навливалок к адресации рисовалок переста─ новкой битов адреса (настройкиREYYYYXXX, SWAP2310- сначала отлаживал одну, потом добавил вторую). 51. При добавлении второго варианта адре─ сации в одном из двух if REYYYYXXX были перепутаны ветки - артефакты на экране, нашёл переключением REYYYYXXX (артефакты остались) и просмотром исходника. - (этап 23)Теперь вход в восстанавлива─ лки всегда по#nnOO,без#nn80.Для этого в#c000..#ffOOставитсяjp nzна восстана─ вливалку#nn80,в конце восстанавливалки всегдаNZ,а в конце выводилки добавилxor a.НастройкаPRJP. - (этап 24)Вместо генератора параметров восстанавливалки сделал патч стека выводи─ лки (прямо в выводилке), чтобы превратить его в стек стиралки - настройкаPRPATCHRE. С новым вариантом адресации меняется всего один байт из 4 на каждый шарик. 52. При добавлении патча стека в выводил─ ку пропустил inc sp:inc sp - сброс, нашёл трассировкой. 53. При добавлении стиралки по патченному стеку поставил ld a,l вместо ld a,(ysinst) - нашёл трассировкой во время поиска сле─ дующего бага. 54. В стеке патчился адрес не текущего, а следующего шарика (ошибка проектирования)- артефакты на экране, нашёл трассировкой. 55. В генераторе рисовалки координата Y для патча бралась не из того регистра - артефакты на экране,нашёл просмотром исхо─ дника. 56. Координата Y бралась в конце шарика, когда она уже не соответствовала - артефа─ кты на экране, нашёл просмотром исходника. 57. В mkscript (генератор скрипта по таб─ лице Y) inc e:ld (hl),e вместо ld (hl),e: inc (hl) и из-за копипасты ещё одно inc e дальше,в результате в стеке не было адреса выхода - сброс, нашёл трассировкой и прос─ мотром памяти. (С этого момента вернулся к стиралкам, сгенерированным из фона.) - (этап 25) Переделал патч стека для стиралок через (hl')вместо ex (sp),hl, теперь патчится адрес нужного шарика, а не следующего. 58. Когда возвращал pop hl ;scr в начало re (раньше я переместил его в конец для исправления фикса адреса следующего шари─ ка), перепутал ветки условной компиляции, так что оно осталось в конце - сброс,нашёл трассировкой. 59. В новом вызове стиралки не присваива─ лись константы в регистры - артефакты на экране, нашёл просмотром исходника. 60. В tap не чистился экран - висло на генераторе стиралки,нашёл трассировкой. (С этого момента 32 шарика работают без проблем на 48K - в эмуляторе Fuse.) - (этап 26)Поскольку в эмуляторе Fuse не увидеть число тактов в процедуре, доба─ влен настраиваемый пустойldir,чтобы ви─ деть, сколько осталось свободного времени. Потом он сделан автонастраиваемым, находит по прерыванию худший случай - настройка SPEEDMETER. - (этап 27) Переделал патч стека для стиралок через(de)вместо(hl'). 61. При убирании констант const_d и const_e написал ifdef вместо if - заметил по неизменению числа тактов. 62. В mkscript (генератор скрипта по таб─ лице Y) при замене hl' на de пропустил inc hl в патче - артефакты на экране, нашёл трассировкой. 63. В genprspr ret оказался в ветке усло─ вной компиляции после добавления xor a - заметил просмотром исходника при убирании xor a, когда менял hl' на de. 64. Убирать xor a было нельзя, т.к. она давала Z - артефакты на экране,нашёл трас─ сировкой, исправил добавлением DRAWUP и сменой Z на C. - (этап 28)НастройкаDRAWUP- рисование спрайтов снизу вверх, чтобы в восстанавли─ валках всегда было CY=0 (раньше всегда былоNZ,но дляZв выводилках надо лишнюю команду xor a,а дляCY=1не надо). Потом сделал, чтобы спрайт переворачивался в со─ ответсвии с этой настройкой. 65. После вставки DRAWUP не оптимизирова─ лись inc l:dhl:dec l - нашёл подсчётом та─ ктов. - (этап 29)Оптимизировал генерациюX- координат пересортировкой таблицы синусов через 7 значений и сразу по 2 байта -x/8 иf(x&7)(настройкаSORTX7). 66. После добавления SORTX7 вызывал fillrexO, хотя добавил перед ней код (fillrex) - сбрасывалось, нашёл трассиров─ кой. - (этап 30)ОптимизировалfillrexO (пе─ ренос данных для X из таблицы синусов в линейную таблицу, с одновременной цикличе─ ской перестановкой шариков) - копирование не черезld,а черезldi. 67. При замене в fillrexO копирования из ld на ldi забыл сохранить регистр c - сбрасывалось, первый раз исправил неправи─ льно (ld c,b, хотя ldi были парами). - (этап 31)Убрал fillrexO,заменил на ldir.Исходную таблицу синусов зациклил методом добавления копии начала в конец. 68. Делал это копирование не к +512, а к +256 - примерно каждый 3-й кадр искажался, нашёл просмотром исходника. - (этап 32) Пересортировал расчёт пара─ метров вывода по порядку ball number,чтобы читать данные дляXиз стека,как и задумы─ валось. 69. При добавлении пересортировки по ball number в поиске был цикл без выхода при нахождении - сбрасывалось,нашёл просмотром исходника. (На этот момент написано 2696 строк в 8 модулях - один баг на 39 строк.) - (этап 33) Оптимизировал в стиралках down hlна нечётных знакоместах черезset/ res 5,l. 70. При оптимизации dhl на нечётных зна─ коместах в ветке DRAWUP написал set вместо res - артефакты на экране, нашёл трассиро─ вкой. (На этот момент написано 2774 строки - один баг на 40 строк; около 7000 тактов свободно при 32 шариках.) - (этап 34) 28.11.2014.Искал самые уда─ чные палитры для шариков и фона, учитывая мнение населения на работе. - (этап 35) 02.12.2014.Добавлял шарики сверх 32 - для этого таблица синусовY увеличена сверх 256 байт (8*BALLSбайт). 71. Когда добавлял шарики сверх 32, напи─ сал ysinst вместо ysin в двух местах - ви─ сло, нашёл просмотром исходника. 72. Тогда же запортил hl в сортировке X (на глаз не было видно, что он использует─ ся, преждевременная оптимизация) - висло, нашёл просмотром исходника. 73. Пересчитывал ysinst (фаза синуса по Y) до того,как с ним была сделана половина действий (преждевременная оптимизация) - каждый восьмой кадр дёргался, нашёл прос─ мотром исходника. 74. Было неправильное зацикливание при BALLS>32 - нашёл случайно просмотром исхо─ дника при поиске предыдущего бага. 75. Проверял if ysind - forward reference, не компилировалось, переписал математикой, первый раз ошибочно ld hl, ysinst вместо ld hl,(ysinst). - (этап 36)Сразу получилось 36 шариков. Оптимизировал подбором траекторий поXи Y,чтобы успел 37-й. - (этап 37)Оптимизировал начало стира─ лок методом устраненияjp #8080+#nnOOна месте выводилок, которые не используются в траекториях - туда кладётся сама стиралка, на которую раньше переходили (настройка MOVERE). - (этап 38)Когда шариков стало 37, сде─ лал шаг по таблице синусовX не 7, а 9, иначе получалось кольцо из шариков. 76. Когда делал шаг по X не 7, а 9, неп─ равильно посчитал 0,9,...,252, (29 шт) - кадры мелькали,нашёл тестовыми изменениями и просмотром исходника. - (этап 39)Оптимизировал доступ к стеку при вайтах методом переноса стека на место выводилок,которые не используются в траек─ ториях (настройкаMOVESCRIPT)- пока несо─ вместимо сMOVERE. - (этап 40)Оптимизировал вывод (настро─ йкаPRFORWAIT)- вместо..ld (hl),a:inc l: ld a,(hl):or reg:xor n:ld (hl),a...теперь будет...ld (hl),a:inc l:ld a,reg:or (hl): xor n:ld (hl),a...,так лучше укладывается в вайты. (На этот момент написано 2932 строки - один баг на 39 строк.) - (этап 41)Оптимизировал вход в восста─ навливалки - не вычислять зановоSP(наст─ ройкаPRPATCHRESP). 77. Когда добавлял PRPATCHRESP, не заме─ тил, что стек уже переприсвоен в mainprret - сбрасывалось, нашёл трассировкой. 78. Тогда же забыл сбросить флаг CY - ар─ тефакты на экране, нашёл просмотром исход─ ника. - (этап 42)Оптимизировал выход из вос─ станавливалок - без использования адреса выхода вix,просто через условный пере─ ход, как при объединении выводилок со сти─ ралками. (На этот момент по оптимистичной оценке свободно порядка 500 тактов при 37 шариках - не считая выбросов на отдельных кадрах.) Ещё через время(04.12.2014): - (этап 43)Реализовал возможность одно─ временно включить оптимизацииMOVESCRIPTи MOVERE(хотя последняя всего на 46 тактов - при совмещении ещё меньше), заодно доба─ влена настройкаEVENY- использовать толь─ ко чётные номераY,чтобы осталось больше места дляMOVERE. - (этап 44)Попробовал ещё более простой рисунок шарика. Теперь почти 38 шариков (на некоторых кадрах не успевает). - (этап 45)Небольшие оптимизации глав─ ного цикла - перенёс xsinstв освободив─ шийся lx,убрал сохранение spиз цикла, перенёс ysinst в используемыйld hl.Всё ещё не успевает.Сколько тактов не успевает - понять невозможно (SPEEDMETERвычисляет только оставшееся время, причём сам тратит много тактов). - (этап 46)Стал выкидывать неиспользуе─ мые настройки (перемещать куски кода в main_nu.asm),чтобы было лучше видно акту─ альный код.Убраны веткиrex=0,PRPATCHRE=0, PRPATCHRESP=0,PRJP=0,SORTX7=0. - (этап 47)Убрал jp mainloop, ускорил ветвление наmainprret,убрал переприсваи─ вание bcпри входе в стиралки. Всё ещё не успевает. - (этап 48)В mainprret вместо ld hl, -(BALLS*4+2):add hl,sp(что, к слову, даёт неправильный CY) теперь поставил ld e, tscriptstkshift&255:ex de,hl. - (этап 49)Убрал mainprret в быструю память, т.к. он выполняется в поле экрана (хотя там всего 7 команд). - (этап 50)Убрал две парыpush...popв подготовке данных дляXперед выводом. 79. На промежуточной фазе этой оптимиза─ ции запортил de (преждевременная оптимиза─ ция) - висло, нашёл просмотром исходника. - (этап 51)Использовал стекtrexвместо стандартного на время прерывания - сэконо─ мил одно переприсваиваниеsp. 80. Забыл присвоить sp на первой итерации - глючило и сбрасывалось, нашёл трассиров─ кой. - (этап 52)Ускорил работу сysinstдля случаяEVENY=1(использовать только чётные номераY в таблице синусов, самYлюбой - старое поведение этой настройки ошибочно осталось): поскольку реально данные из та─ блицыYне читаются, нужен только номер, и он в этом случае делится на 2 и становится однобайтным.Следующий номер теперь берётся без ветвлений по таблице черезld l,(hl). Уже почти успевает 38 шариков с новой гра─ фикой - мигает ОЧЕНЬ редко. - (этап 53)Вместоei:haltпоставил ei: <код, терпимый к прерываниям>:ld a,r:jp pe (po=di,т.е. прерывание было захвачено), или дажеld a,r:ret po:jp.Всё это для то─ го, чтобы одиночный медленный кадр не дал пропуска следующего кадра поhalt.Теперь с новой графикой не мигает на 38 шариках и почти успевает 39. 81. Когда убирал команду ld b,0: условие if reconst_b!=0 не компилилось SjAsm'ом, заменил на if !reconst_b, но на самом деле надо было наоборот: if reconst_b - нашёл просмотром кода. - (этап 54)Оптимизировал ld l иld bc на основании bc=0 послеldirпри входе в скрипты. 82. Написал условие if !tscripts вместо if !(tscripts&255) - нашёл просмотром ко─ да. - (этап 55)Оптимизировалld bcна входе в рисовалки. (На этот момент написано 3154 строки, в среднем один баг на 38 строк; все оптими─ зации кода за этот день дали выигрыш всего около 400 тактов (есть задел на ещё неско─ лько тактов в трёх местах), но изменение графики и алгоритма захвата прерываний да─ ли существенный выигрыш.) Через большое время(30.01.2015): 83. Из-за убирания ld b,0 нельзя было вы─ ключить EVENY и bgtype - висло,нашёл трас─ сировкой, временно вернул ld b,0, оставил TODO на сложное условие. 84. Одновременно обнаружил, что две наст─ ройки для sinx??opt были включены, а для bgtype=0 инклюдилась таблица без opt. 85. Фаза X лежала в lx, а он был занят в bgtype=0 - заметил по поведению шариков, нашёл просмотром исходника, исправил ix на iy в repush.asm. - (этап 56)Сделал в окошке и убрал патч стека дляbgtype=0- теперь с новой графи─ кой успевает 40 шариков. 03.03.2015: 86. Заметил, что верхнюю и нижнюю строчки экрана не надо чистить из-за рисунка шари─ ков - нашёл просмотром экрана, выиграл 332 такта. - (этап 57) Заменил "оптимизированную" таблицу Y на простой синус - скорость не упала, зато стало видно одновременно все шарики (раньше они перекрывали друг друга, и не было такого скриншота, где было видно все). Подгонкой окошка добился 42 шариков с небольшим запасом на музыку. А дальше всё завертелось - до пати ос─ тавалось всё меньше и меньше времени, и я перестал документировать. Добавлял плеер музыки, появление/стирание фона, фикс под музыку, но основной код и таблицы остались неизменными. * * * Статистика (по ball record и цветным чанкам): - в среднем один баг у меня приходится на каждые 40 добавленных строк. Поэтому надо быть осторожным при добавлении кусков по 100 строк и более - лучше этого избе─ гать, чтобы не ловить несколько багов од─ новременно. - из 105 зафиксированных багов 20 отно─ сятся к упомянутым выше типам: 9 являются ошибками copy-paste, 7 - ошибками прежде─ временной оптимизации, 4 - ошибками проек─ тирования. А из оставшихся, кажется, самые популярные: неполные изменения (исправил в одном месте и не исправил в другом), про─ пущенные строки, ошибочные выражения. - Если я не ошибся в подсчётах, 9 багов найдены компилятором, 19 - просмотром ис─ ходника (в том числе при изменениях), 23 - запуском (в том числе подсчётом тактов), 3 - просмотром скомпилированного кода, 8 - изменением тесткейза, 31 - трассировкой (в том числе 6 - просмотром памяти, 4 - прос─ мотром сгенерённого кода), 1 - выкидывани─ ем частей исходника, 1 - постепенным отка─ том изменений. * * * Я записал на видео один пример, как я пишу и отлаживаю код - в реальном времени, со всеми ошибками. Это мультиколорный скроллер картинки для фирменного 48K: http://www.youtube.com/watch?v=eQacAM9VRmЧ Сама программа с графикой от Trixs/ Conscience лежит в приложении. * * * Кажется, всё самое главное я рассказал. Но тема интересная и перспективная. Ведь если умеешь отлаживать - можешь рискнуть писать большой код. Если не умеешь - не можешь. Конечно, надо ещё иметь фантазию, усидчивость и определённый кругозор,но эти качества присущи практически любой творче─ ской специальности. Осталось только пожелать вам приятных глюков и успешной разработки кода с боль─ шой буквы!
Other articles:
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Similar articles:
В этот день... 23 November