ajrhacker

I agree with @cuavas: `util/palette.h` is, in principle, the wrong header to put phosphor color definitions. I would prefer that they be placed in a new header in `src/devices/video`. There...

> MAME should supply an address space and possibly a base address/mask to the cards and let them map themselves. This is an elegant method, but does not play nicely...

Another pernicious old-MESSism that @angelosa is currently working on removing in the a800_cart branch is installing cartridge memory and control registers in `machine_start` handlers (including old-style overrides), based on mapper...

I don't know whether this has anything to do with this issue, but there appears to be a copy-and-paste error in `hle_msmouse_device_base::input_rts`.

Though my review was requested, I don't really have much to say about this change except to note that one piece of software I tested was mac_flop_orig:macwars. With the broken...

Of the two, I'd much prefer option 1. In principle, I think that interface devices should be decoupled from the peripheral connector slots that they normally control as much as...

Some initial comments: * Yes, I don't particularly like the `try_read` and `try_write` names, which aren't very suggestive of how they should be used. However, `read_block` and `write_block` are the...

Overloading of simple terminology has been the curse of computing jargon since the 1950s. I might suggest `read/write_chunk` as an alternative naming suggestion to `read/write_block`.

These peripheral emulations should really be done through slot device interfaces instead of creating new clone systems, even if it's the case that they can't use the existing `pc_kbd` bus...

I also took a look at the 65816 datasheet, and this does seems to be a dummy read issue. Interestingly, these dummy reads are not supposed to be inserted for...