diff options
author | Vas Crabb <vas@vastheman.com> | 2020-11-13 15:26:31 +1100 |
---|---|---|
committer | Vas Crabb <vas@vastheman.com> | 2020-11-13 15:26:31 +1100 |
commit | fe9ac15ca9f027d0eb5e21aa5496e3d6fa674e45 (patch) | |
tree | b48da30d7948f1e251b3f468de3fa7f3405d2a29 /src/mame/drivers/metlfrzr.cpp | |
parent | 52285c255363d1f431eee95f0e8d01b546c33d48 (diff) |
-emu/devfind: More cleanup/consistency changes.
* Removed .mask(), as it’s not reliable in the general case.
* Added asserts to things that assume power-of-two sizes.
* Got rid of virtual qualifier on pointer-to-member operator.
* Made helpers a bit more assertive about logging warnings.
-emu/rendlay.cpp: Use delegates to avoid hot conditional branches.
-docs: Finished off description of object finders and output finders.
Diffstat (limited to 'src/mame/drivers/metlfrzr.cpp')
-rw-r--r-- | src/mame/drivers/metlfrzr.cpp | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/mame/drivers/metlfrzr.cpp b/src/mame/drivers/metlfrzr.cpp index 7e7183db1bf..e092dc58fb1 100644 --- a/src/mame/drivers/metlfrzr.cpp +++ b/src/mame/drivers/metlfrzr.cpp @@ -79,6 +79,9 @@ private: void metlfrzr_state::video_start() { + // assumes it can make an address mask with m_vram.length() - 1 + assert(!(m_vram.length() & (m_vram.length() - 1))); + m_fg_tilebank = 0; m_rowscroll_enable = false; @@ -98,7 +101,7 @@ void metlfrzr_state::video_start() void metlfrzr_state::legacy_bg_draw(bitmap_ind16 &bitmap,const rectangle &cliprect) { gfx_element *gfx = m_gfxdecode->gfx(m_fg_tilebank); - const uint16_t vram_mask = m_vram.mask() >> 1; + const uint16_t vram_mask = (m_vram.length() - 1) >> 1; int count; int x_scroll_base; int x_scroll_shift; @@ -195,7 +198,7 @@ void metlfrzr_state::metlfrzr_map(address_map &map) { map(0x0000, 0x7fff).rom(); map(0x8000, 0xbfff).bankr("bank1"); - map(0xc000, 0xcfff).ram().share("vram"); + map(0xc000, 0xcfff).ram().share(m_vram); map(0xd000, 0xd1ff).ram().w(m_palette, FUNC(palette_device::write_indirect)).share("palette"); map(0xd200, 0xd3ff).ram().w(m_palette, FUNC(palette_device::write_indirect_ext)).share("palette_ext"); |