summaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
author Nathan Woods <npwoods@mess.org>2017-06-18 11:28:56 -0400
committer Vas Crabb <cuavas@users.noreply.github.com>2017-06-19 10:37:03 +1000
commitc192588f61fd8d3654d1fb2be11dda52da44fed0 (patch)
tree079468843b05c253f00b06705bf0440711e4143b
parente14c43ae7e8c11d0fd729d0da267959264d798f4 (diff)
Changed 'enum read_or_write' to be 'enum class'
-rw-r--r--src/emu/debug/debugcmd.cpp2
-rw-r--r--src/emu/emumem.cpp52
-rw-r--r--src/emu/emumem.h28
-rw-r--r--src/mame/machine/fddebug.cpp2
4 files changed, 42 insertions, 42 deletions
diff --git a/src/emu/debug/debugcmd.cpp b/src/emu/debug/debugcmd.cpp
index 843b8e383dc..145aaa0ac3f 100644
--- a/src/emu/debug/debugcmd.cpp
+++ b/src/emu/debug/debugcmd.cpp
@@ -2877,7 +2877,7 @@ void debugger_commands::execute_map(int ref, const std::vector<std::string> &par
taddress = space->address_to_byte(address) & space->bytemask();
if (space->device().memory().translate(space->spacenum(), intention, taddress))
{
- const char *mapname = space->get_handler_string((intention == TRANSLATE_WRITE_DEBUG) ? ROW_WRITE : ROW_READ, taddress);
+ const char *mapname = space->get_handler_string((intention == TRANSLATE_WRITE_DEBUG) ? read_or_write::WRITE : read_or_write::READ, taddress);
m_console.printf(
"%7s: %0*X logical == %0*X physical -> %s\n",
intnames[intention & 3],
diff --git a/src/emu/emumem.cpp b/src/emu/emumem.cpp
index 012d7901bef..a6aaa1d86cb 100644
--- a/src/emu/emumem.cpp
+++ b/src/emu/emumem.cpp
@@ -906,7 +906,7 @@ public:
u8 buffer[16];
for (int index = 0; index < 16; index++)
buffer[index ^ ((_Endian == ENDIANNESS_NATIVE) ? 0 : (data_width()/8 - 1))] = index * 0x11;
- install_ram_generic(0x00, 0x0f, 0x0f, 0, ROW_READWRITE, buffer);
+ install_ram_generic(0x00, 0x0f, 0x0f, 0, read_or_write::READWRITE, buffer);
printf("\n\naddress_space(%d, %s, %s)\n", NATIVE_BITS, (_Endian == ENDIANNESS_LITTLE) ? "little" : "big", _Large ? "large" : "small");
// walk through the first 8 addresses
@@ -1633,13 +1633,13 @@ void memory_manager::dump(FILE *file)
"====================================================\n"
"Device '%s' %s address space read handler dump\n"
"====================================================\n", space->device().tag(), space->name());
- space->dump_map(file, ROW_READ);
+ space->dump_map(file, read_or_write::READ);
fprintf(file, "\n\n"
"====================================================\n"
"Device '%s' %s address space write handler dump\n"
"====================================================\n", space->device().tag(), space->name());
- space->dump_map(file, ROW_WRITE);
+ space->dump_map(file, read_or_write::WRITE);
}
}
@@ -2109,8 +2109,8 @@ void address_space::populate_from_map(address_map *map)
last_entry = entry;
// map both read and write halves
- populate_map_entry(*entry, ROW_READ);
- populate_map_entry(*entry, ROW_WRITE);
+ populate_map_entry(*entry, read_or_write::READ);
+ populate_map_entry(*entry, read_or_write::WRITE);
populate_map_entry_setoffset(*entry);
}
}
@@ -2124,7 +2124,7 @@ void address_space::populate_from_map(address_map *map)
void address_space::populate_map_entry(const address_map_entry &entry, read_or_write readorwrite)
{
- const map_handler_data &data = (readorwrite == ROW_READ) ? entry.m_read : entry.m_write;
+ const map_handler_data &data = (readorwrite == read_or_write::READ) ? entry.m_read : entry.m_write;
// based on the handler type, alter the bits, name, funcptr, and object
switch (data.m_type)
{
@@ -2133,7 +2133,7 @@ void address_space::populate_map_entry(const address_map_entry &entry, read_or_w
case AMH_ROM:
// writes to ROM are no-ops
- if (readorwrite == ROW_WRITE)
+ if (readorwrite == read_or_write::WRITE)
return;
// fall through to the RAM case otherwise
@@ -2150,7 +2150,7 @@ void address_space::populate_map_entry(const address_map_entry &entry, read_or_w
break;
case AMH_DEVICE_DELEGATE:
- if (readorwrite == ROW_READ)
+ if (readorwrite == read_or_write::READ)
switch (data.m_bits)
{
case 8: install_read_handler(entry.m_addrstart, entry.m_addrend, entry.m_addrmask, entry.m_addrmirror, entry.m_addrselect, read8_delegate(entry.m_rproto8, entry.m_devbase), data.m_mask); break;
@@ -2170,14 +2170,14 @@ void address_space::populate_map_entry(const address_map_entry &entry, read_or_w
case AMH_PORT:
install_readwrite_port(entry.m_addrstart, entry.m_addrend, entry.m_addrmirror,
- (readorwrite == ROW_READ) ? data.m_tag : nullptr,
- (readorwrite == ROW_WRITE) ? data.m_tag : nullptr);
+ (readorwrite == read_or_write::READ) ? data.m_tag : nullptr,
+ (readorwrite == read_or_write::WRITE) ? data.m_tag : nullptr);
break;
case AMH_BANK:
install_bank_generic(entry.m_addrstart, entry.m_addrend, entry.m_addrmirror,
- (readorwrite == ROW_READ) ? data.m_tag : nullptr,
- (readorwrite == ROW_WRITE) ? data.m_tag : nullptr);
+ (readorwrite == read_or_write::READ) ? data.m_tag : nullptr,
+ (readorwrite == read_or_write::WRITE) ? data.m_tag : nullptr);
break;
case AMH_DEVICE_SUBMAP:
@@ -2275,7 +2275,7 @@ void address_space::locate_memory()
{
// once this is done, find the starting bases for the banks
for (auto &bank : manager().banks())
- if (bank.second->base() == nullptr && bank.second->references_space(*this, ROW_READWRITE))
+ if (bank.second->base() == nullptr && bank.second->references_space(*this, read_or_write::READWRITE))
{
// set the initial bank pointer
for (address_map_entry &entry : m_map->m_entrylist)
@@ -2358,7 +2358,7 @@ address_map_entry *address_space::block_assign_intersecting(offs_t bytestart, of
const char *address_space::get_handler_string(read_or_write readorwrite, offs_t byteaddress)
{
- if (readorwrite == ROW_READ)
+ if (readorwrite == read_or_write::READ)
return read().handler_name(read().lookup(byteaddress));
else
return write().handler_name(write().lookup(byteaddress));
@@ -2372,7 +2372,7 @@ const char *address_space::get_handler_string(read_or_write readorwrite, offs_t
void address_space::dump_map(FILE *file, read_or_write readorwrite)
{
- const address_table &table = (readorwrite == ROW_READ) ? static_cast<address_table &>(read()) : static_cast<address_table &>(write());
+ const address_table &table = (readorwrite == read_or_write::READ) ? static_cast<address_table &>(read()) : static_cast<address_table &>(write());
// dump generic information
fprintf(file, " Address bits = %d\n", m_config.m_addrbus_width);
@@ -2406,18 +2406,18 @@ void address_space::unmap_generic(offs_t addrstart, offs_t addrend, offs_t addrm
VPRINTF(("address_space::unmap(%s-%s mirror=%s, %s, %s)\n",
core_i64_hex_format(addrstart, m_addrchars), core_i64_hex_format(addrend, m_addrchars),
core_i64_hex_format(addrmirror, m_addrchars),
- (readorwrite == ROW_READ) ? "read" : (readorwrite == ROW_WRITE) ? "write" : (readorwrite == ROW_READWRITE) ? "read/write" : "??",
+ (readorwrite == read_or_write::READ) ? "read" : (readorwrite == read_or_write::WRITE) ? "write" : (readorwrite == read_or_write::READWRITE) ? "read/write" : "??",
quiet ? "quiet" : "normal"));
offs_t nstart, nend, nmask, nmirror;
check_optimize_mirror("unmap_generic", addrstart, addrend, addrmirror, nstart, nend, nmask, nmirror);
// read space
- if (readorwrite == ROW_READ || readorwrite == ROW_READWRITE)
+ if (readorwrite == read_or_write::READ || readorwrite == read_or_write::READWRITE)
read().map_range(nstart, nend, nmask, nmirror, quiet ? STATIC_NOP : STATIC_UNMAP);
// write space
- if (readorwrite == ROW_WRITE || readorwrite == ROW_READWRITE)
+ if (readorwrite == read_or_write::WRITE || readorwrite == read_or_write::READWRITE)
write().map_range(nstart, nend, nmask, nmirror, quiet ? STATIC_NOP : STATIC_UNMAP);
}
@@ -2499,7 +2499,7 @@ void address_space::install_bank_generic(offs_t addrstart, offs_t addrend, offs_
if (rtag != nullptr)
{
std::string fulltag = device().siblingtag(rtag);
- memory_bank &bank = bank_find_or_allocate(fulltag.c_str(), addrstart, addrend, addrmirror, ROW_READ);
+ memory_bank &bank = bank_find_or_allocate(fulltag.c_str(), addrstart, addrend, addrmirror, read_or_write::READ);
read().map_range(nstart, nend, nmask, nmirror, bank.index());
}
@@ -2507,7 +2507,7 @@ void address_space::install_bank_generic(offs_t addrstart, offs_t addrend, offs_
if (wtag != nullptr)
{
std::string fulltag = device().siblingtag(wtag);
- memory_bank &bank = bank_find_or_allocate(fulltag.c_str(), addrstart, addrend, addrmirror, ROW_WRITE);
+ memory_bank &bank = bank_find_or_allocate(fulltag.c_str(), addrstart, addrend, addrmirror, read_or_write::WRITE);
write().map_range(nstart, nend, nmask, nmirror, bank.index());
}
@@ -2553,17 +2553,17 @@ void address_space::install_ram_generic(offs_t addrstart, offs_t addrend, offs_t
VPRINTF(("address_space::install_ram_generic(%s-%s mirror=%s, %s, %p)\n",
core_i64_hex_format(addrstart, m_addrchars), core_i64_hex_format(addrend, m_addrchars),
core_i64_hex_format(addrmirror, m_addrchars),
- (readorwrite == ROW_READ) ? "read" : (readorwrite == ROW_WRITE) ? "write" : (readorwrite == ROW_READWRITE) ? "read/write" : "??",
+ (readorwrite == read_or_write::READ) ? "read" : (readorwrite == read_or_write::WRITE) ? "write" : (readorwrite == read_or_write::READWRITE) ? "read/write" : "??",
baseptr));
offs_t nstart, nend, nmask, nmirror;
check_optimize_mirror("install_ram_generic", addrstart, addrend, addrmirror, nstart, nend, nmask, nmirror);
// map for read
- if (readorwrite == ROW_READ || readorwrite == ROW_READWRITE)
+ if (readorwrite == read_or_write::READ || readorwrite == read_or_write::READWRITE)
{
// find a bank and map it
- memory_bank &bank = bank_find_or_allocate(nullptr, addrstart, addrend, addrmirror, ROW_READ);
+ memory_bank &bank = bank_find_or_allocate(nullptr, addrstart, addrend, addrmirror, read_or_write::READ);
read().map_range(nstart, nend, nmask, nmirror, bank.index());
// if we are provided a pointer, set it
@@ -2590,10 +2590,10 @@ void address_space::install_ram_generic(offs_t addrstart, offs_t addrend, offs_t
}
// map for write
- if (readorwrite == ROW_WRITE || readorwrite == ROW_READWRITE)
+ if (readorwrite == read_or_write::WRITE || readorwrite == read_or_write::READWRITE)
{
// find a bank and map it
- memory_bank &bank = bank_find_or_allocate(nullptr, addrstart, addrend, addrmirror, ROW_WRITE);
+ memory_bank &bank = bank_find_or_allocate(nullptr, addrstart, addrend, addrmirror, read_or_write::WRITE);
write().map_range(nstart, nend, nmask, nmirror, bank.index());
// if we are provided a pointer, set it
@@ -2907,7 +2907,7 @@ memory_bank *address_space::bank_find_anonymous(offs_t bytestart, offs_t byteend
{
// try to find an exact match
for (auto &bank : manager().banks())
- if (bank.second->anonymous() && bank.second->references_space(*this, ROW_READWRITE) && bank.second->matches_exactly(bytestart, byteend))
+ if (bank.second->anonymous() && bank.second->references_space(*this, read_or_write::READWRITE) && bank.second->matches_exactly(bytestart, byteend))
return bank.second.get();
// not found
diff --git a/src/emu/emumem.h b/src/emu/emumem.h
index 12b72d40101..752e1287e30 100644
--- a/src/emu/emumem.h
+++ b/src/emu/emumem.h
@@ -43,11 +43,11 @@ enum address_spacenum
DECLARE_ENUM_OPERATORS(address_spacenum)
// read or write constants
-enum read_or_write
+enum class read_or_write
{
- ROW_READ = 1,
- ROW_WRITE = 2,
- ROW_READWRITE = 3
+ READ = 1,
+ WRITE = 2,
+ READWRITE = 3
};
@@ -318,12 +318,12 @@ public:
offs_t byte_to_address_end(offs_t address) const { return m_config.byte2addr_end(address); }
// umap ranges (short form)
- void unmap_read(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, ROW_READ, false); }
- void unmap_write(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, ROW_WRITE, false); }
- void unmap_readwrite(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, ROW_READWRITE, false); }
- void nop_read(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, ROW_READ, true); }
- void nop_write(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, ROW_WRITE, true); }
- void nop_readwrite(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, ROW_READWRITE, true); }
+ void unmap_read(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, read_or_write::READ, false); }
+ void unmap_write(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, read_or_write::WRITE, false); }
+ void unmap_readwrite(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, read_or_write::READWRITE, false); }
+ void nop_read(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, read_or_write::READ, true); }
+ void nop_write(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, read_or_write::WRITE, true); }
+ void nop_readwrite(offs_t addrstart, offs_t addrend, offs_t addrmirror = 0) { unmap_generic(addrstart, addrend, addrmirror, read_or_write::READWRITE, true); }
// install ports, banks, RAM (short form)
void install_read_port(offs_t addrstart, offs_t addrend, const char *rtag) { install_read_port(addrstart, addrend, 0, rtag); }
@@ -349,9 +349,9 @@ public:
void install_read_bank(offs_t addrstart, offs_t addrend, offs_t addrmirror, memory_bank *bank) { install_bank_generic(addrstart, addrend, addrmirror, bank, nullptr); }
void install_write_bank(offs_t addrstart, offs_t addrend, offs_t addrmirror, memory_bank *bank) { install_bank_generic(addrstart, addrend, addrmirror, nullptr, bank); }
void install_readwrite_bank(offs_t addrstart, offs_t addrend, offs_t addrmirror, memory_bank *bank) { install_bank_generic(addrstart, addrend, addrmirror, bank, bank); }
- void install_rom(offs_t addrstart, offs_t addrend, offs_t addrmirror, void *baseptr = nullptr) { install_ram_generic(addrstart, addrend, addrmirror, ROW_READ, baseptr); }
- void install_writeonly(offs_t addrstart, offs_t addrend, offs_t addrmirror, void *baseptr = nullptr) { install_ram_generic(addrstart, addrend, addrmirror, ROW_WRITE, baseptr); }
- void install_ram(offs_t addrstart, offs_t addrend, offs_t addrmirror, void *baseptr = nullptr) { install_ram_generic(addrstart, addrend, addrmirror, ROW_READWRITE, baseptr); }
+ void install_rom(offs_t addrstart, offs_t addrend, offs_t addrmirror, void *baseptr = nullptr) { install_ram_generic(addrstart, addrend, addrmirror, read_or_write::READ, baseptr); }
+ void install_writeonly(offs_t addrstart, offs_t addrend, offs_t addrmirror, void *baseptr = nullptr) { install_ram_generic(addrstart, addrend, addrmirror, read_or_write::WRITE, baseptr); }
+ void install_ram(offs_t addrstart, offs_t addrend, offs_t addrmirror, void *baseptr = nullptr) { install_ram_generic(addrstart, addrend, addrmirror, read_or_write::READWRITE, baseptr); }
// install device memory maps
template <typename T> void install_device(offs_t addrstart, offs_t addrend, T &device, void (T::*map)(address_map &map), int bits = 0, u64 unitmask = 0) {
@@ -498,7 +498,7 @@ class memory_bank
// does this reference match the space+read/write combination?
bool matches(const address_space &space, read_or_write readorwrite) const
{
- return (&space == &m_space && (readorwrite == ROW_READWRITE || readorwrite == m_readorwrite));
+ return (&space == &m_space && (readorwrite == read_or_write::READWRITE || readorwrite == m_readorwrite));
}
private:
diff --git a/src/mame/machine/fddebug.cpp b/src/mame/machine/fddebug.cpp
index 0c908535926..6495af49f52 100644
--- a/src/mame/machine/fddebug.cpp
+++ b/src/mame/machine/fddebug.cpp
@@ -467,7 +467,7 @@ static inline int addr_is_valid(address_space &space, uint32_t addr, uint32_t fl
return 0;
/* if we're invalid, fail */
- if (strcmp(const_cast<address_space &>(space)->get_handler_string(ROW_READ, addr), "segaic16_memory_mapper_lsb_r") == 0)
+ if (strcmp(const_cast<address_space &>(space)->get_handler_string(read_or_write::READ, addr), "segaic16_memory_mapper_lsb_r") == 0)
return 2;
return 1;