Emu80 v.4

[nzeemin]

Pyk, компилятор -- g++ из MinGW.
Вообще было бы здорово и MSVC поддерживать, VS2013 или VS2015.

Не, не, см. 2-ю строчку, индекс-то меняется!

Ну давай раскроем шаблон AF и вот так запишем:

    case 0x08:          /* EX AF,AF' */
        af[af_sel] = af[af_sel];
        af_sel = 1 - af_sel;
        af[af_sel] = af[af_sel];
        break;

Получается что меняется только af_sel, может оно и правильно, но зачем тогда лишние присваивания.

- - - Updated - - -

7. Ut88.cpp:65, Micro80.cpp:69
memset два раза, странный размер m_bufSize, а выделяется до этого 512 * 512, совсем не факт что эти две величины равны.

8. CpuZ80.cpp:281

            cbshflg1:
                AF = (AF & ~0xff) | (temp & 0xa8) |
                    (((temp & 0xff) == 0) << 6) |
                    parity(temp) | !!cbits;

Тут что-то странное, два оператора логического NOT дают тип bool, с которым делают побитовую операцию -- в общем, тут надо копать.

[Advertiser]

[Pyk]

Вообще было бы здорово и MSVC поддерживать, VS2013 или VS2015

Так MSVC поддерживается, хотя я нигде явно это не афишировал. По крайней мере, я периодически собираю в MSVC 2015 x32 и x64. Для того, чтобы официально объявить о поддержке, не хватает некоторых мелочей: есть проблемы с отображением статусной строки в последних версиях (что-то с интерпретацией unicode-констант в исходниках) и нечитаемые варнинги при сборке из-за русской локали установленного у меня MSVC. Думаю, что и с тем, и с другим вполне можно разобраться, но опять-таки - руки не дошли...

Ну давай раскроем шаблон AF и вот так запишем:

Так в данном контексте AF вроде бы переменная и ее нельзя так раскрыть!

Ut88.cpp:65, Micro80.cpp:69
memset два раза, странный размер m_bufSize, а выделяется до этого 512 * 512, совсем не факт что эти две величины равны

Да, похоже, что лишняя память выделяется, а в memset размер правильный. Завтра внимательно проверю.

Тут что-то странное, два оператора логического NOT дают тип bool, с которым делают побитовую операцию

Опять-таки не мой код, сильно подозреваю, что !!cbits эквивалентно чему-то вроде (cbsts == 0 ? 0 : 1);

[nzeemin]

> Так MSVC поддерживается
Тогда нужно файл проекта положить в исходники.
Я попробовал сделать проект VS2013 со сборкой через nmake, были проблемы с типами данных, такими как int32_t.

В целом мои рекомендации:
1. Использовать бьютифаер кода, который по заданным правилам форматирует код автоматически. Применять сразу после изменений. Я использую AStyle, ещё есть clang-format.
2. Настроить CI для автоматической сборки всех вариантов по коммиту, под всеми системами. На это нужно несколько часов, окупается это время быстро.
3. Сделать тестовый стенд -- консольное приложение на основе той же самой кодовой базы, выполняющее загрузку и запуск наиболее часто используемых программ, с контролем совпадения "скриншотов" в контрольных точках. Прогонять их после существенных изменений. Это даёт проверку на то что нет регрессии -- что не упало то что раньше уже работало.
4. Найти программные тесты (в частности, процессора), включить их в набор тестового стенда.

[Pyk]

nzeemin, я имел в виду, что успешно собирается из Qt Creator и использованием компилятора командной строки MSVC. Собирать из IDE Visual Studio я никогда не пробовал. Согласен, что неплохо бы поэкспериментировать с этим - для многих будет актуально... Кстати, избавился от имевшейся проблемы с MSVC - см. последний коммит. Сейчас все работает после сборки с помощью MSVC, хотя при сборке и выдается большое количество варнингов.

За рекомендации спасибо: настройка CI есть в ближайших планах, остальное по возможности.

Думаю еще, что делать с установкой в linux-системах для тех, кто не хочет либо не может заниматься сборкой. Надо бы наверное сделать deb/rpm-пакеты. Как быть только с библиотеками Qt? Пакеты с актуальной версией Qt есть только в репозиториях самых последних версияй дистрибутивов. Наверное статически слинковать? Нет опыта в подобных вещах?

[nzeemin]

nzeemin, я имел в виду, что успешно собирается из Qt Creator и использованием компилятора командной строки MSVC.

Под Qt Creator в конфигурации Desktop Qt 5.6.3 MSVC2013 32bit -- НЕ собирается, ошибки на типы, такие как uint32_t.

Под Qt Creator в конфигурации Desktop Qt 5.6.3 MSVC2015 32bit -- собралось, 87 варнингов, работает.

- - - Updated - - -

В Qt версии команда сохранения скриншота предлагает только .BMP, но Qt позволяет много других форматов, хотя бы .PNG стоит добавить.

- - - Updated - - -

Нет опыта в подобных вещах?

Нет.

[Pyk]

nzeemin, да, все так. От варнингов MSVC 2015 постараюсь со временем избавиться, не уверен, есть ли смысл добиваться сборки с помощью MSVC 2013?

Кстати, MSCV 2013 и 2015 одновременно установлены? Qt без проблем их различает? У меня не получилось так просто завести одновременно MSVC 2015 и 2010: Qt не видит установленный MSVC 2010 (Express).

Насчет форматов сохранения в курсе, но отложил пока этот вопрос, поскольку требуется небольшой рефакторинг в SDL-части, чтобы не городить условную компиляцию... В общем, в ближайших планах...

- - - Добавлено - - -

P.S. Да, в конфигурации MSVC2015 64bit также собирается и работает.

[nzeemin]

Под macOS в Qt Creator 4.6.0 собралось, конфигурация Desktop Qt 5.9.4 clang 64bit, 3 warnings.
Особенность в том что конфиг и прочее нужно положить в "папку" /Users/nzeemin/MyProjects/emu80v4/src/Emu80qt.app/Contents/MacOS, то есть по сути внутрь "приложения".
Радио-86РК запустился, отладчик тоже, кстати это очень забавно видеть такой терминал на маке.

- - - Updated - - -

/Users/nzeemin/MyProjects/emu80v4/src/Debugger.cpp:24: In file included from Debugger.cpp:24:
/Users/nzeemin/MyProjects/emu80v4/src/Debugger.h:99: warning: 'init' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]
        void init() {EmuWindow::init();};
             ^
/Users/nzeemin/MyProjects/emu80v4/src/EmuWindow.h:68: overridden virtual function is here
        void init() override;
             ^
/Users/nzeemin/MyProjects/emu80v4/src/Platform.cpp:34: In file included from Platform.cpp:34:
/Users/nzeemin/MyProjects/emu80v4/src/Debugger.h:99: warning: 'init' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]
        void init() {EmuWindow::init();};
             ^
/Users/nzeemin/MyProjects/emu80v4/src/EmuWindow.h:68: overridden virtual function is here
        void init() override;
             ^
/Users/nzeemin/MyProjects/emu80v4/src/qt/qtPaintWidget.cpp:180: warning: unused parameter 'event' [-Wunused-parameter]
void PaintWidget::mouseMoveEvent(QMouseEvent *event)
                                              ^

Скриншот: https://yadi.sk/i/pphicPgx3VhJCz

[Pyk]

nzeemin, спасибо за хорошие новости.
Куда на маке обычно нужно класть конфиг и прочее?
На unix-like это обычно usr/share или usr/local/share, но насчет MacOS совершенно не уверен...
Кстати, если из "прочего" убрать папку "_settings", то текущие настройки должны сохраняться в $HOME/.config/Emu80, при этом директория установки может быть закрыта для записи.

А насчет отладчика - сам не знаю, почему мне захотелось сделать такой old-school-отладчик Может быть, когда-нибудь переделаю... Ну или как минимум управление мышкой добавлю...

[nzeemin]

Pyk, я по маку мало что знаю, почти не пишу там, проще нагуглить.

Про отладчик -- по опыту скажу что хороший отладчик это очень важно, в свой я вложил очень много, и это окупается.
И ещё один вариант с отладчиком -- сделать стаб для gdb, и отлаживать снаружи через gdb или любой другой отладчик, работающий по этому протоколу.

[ZEman]

больше месяца затишье, продолжение следует ?