summaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
author Miodrag Milanovic <mmicko@gmail.com>2016-06-18 10:18:30 +0200
committer Miodrag Milanovic <mmicko@gmail.com>2016-06-18 15:32:15 +0200
commita07b82ac11122eeed45c9e6a78f34c3bee8ed543 (patch)
tree9ecdb3e8db34e2ad4e394565657409b58cc1a689
parent62805f7cac1b007828b6aabaa660b82afc00e0bc (diff)
tagged_list to unordered_map for slots (nw)
-rw-r--r--src/emu/dislot.cpp4
-rw-r--r--src/emu/dislot.h6
-rw-r--r--src/emu/rendutil.cpp7
-rw-r--r--src/emu/validity.cpp6
-rw-r--r--src/frontend/mame/clifront.cpp16
-rw-r--r--src/frontend/mame/info.cpp16
-rw-r--r--src/frontend/mame/ui/slotopt.cpp16
7 files changed, 35 insertions, 36 deletions
diff --git a/src/emu/dislot.cpp b/src/emu/dislot.cpp
index 051dc33bdb2..0b6c26199b2 100644
--- a/src/emu/dislot.cpp
+++ b/src/emu/dislot.cpp
@@ -36,7 +36,7 @@ void device_slot_interface::static_option_reset(device_t &device)
{
device_slot_interface &intf = dynamic_cast<device_slot_interface &>(device);
- intf.m_options.reset();
+ intf.m_options.clear();
}
void device_slot_interface::static_option_add(device_t &device, const char *name, const device_type &devtype)
@@ -47,7 +47,7 @@ void device_slot_interface::static_option_add(device_t &device, const char *name
if (option != nullptr)
throw emu_fatalerror("slot '%s' duplicate option '%s\n", device.tag(), name);
- intf.m_options.append(name, *global_alloc(device_slot_option(name, devtype)));
+ intf.m_options.insert(std::make_pair(name, std::make_unique<device_slot_option>(name, devtype)));
}
device_slot_option *device_slot_interface::static_option(device_t &device, const char *name)
diff --git a/src/emu/dislot.h b/src/emu/dislot.h
index 7aaf28d004d..e81652738ae 100644
--- a/src/emu/dislot.h
+++ b/src/emu/dislot.h
@@ -114,15 +114,15 @@ public:
static void static_set_option_clock(device_t &device, const char *option, UINT32 default_clock) { static_option(device, option)->m_clock = default_clock; }
bool fixed() const { return m_fixed; }
const char *default_option() const { return m_default_option; }
- const tagged_list<device_slot_option> &option_list() const { return m_options; }
- device_slot_option *option(const char *name) const { if (name) return m_options.find(name); return nullptr; }
+ const std::unordered_map<std::string, std::unique_ptr<device_slot_option>> &option_list() const { return m_options; }
+ device_slot_option *option(const char *name) const { if (name) { auto search = m_options.find(name); if (search != m_options.end()) return search->second.get(); else return nullptr; } else return nullptr; }
virtual std::string get_default_card_software() { return std::string(); }
device_t *get_card_device();
private:
// internal state
static device_slot_option *static_option(device_t &device, const char *option);
- tagged_list<device_slot_option> m_options;
+ std::unordered_map<std::string,std::unique_ptr<device_slot_option>> m_options;
const char *m_default_option;
bool m_fixed;
};
diff --git a/src/emu/rendutil.cpp b/src/emu/rendutil.cpp
index c4ec425dad9..c356a1d0abd 100644
--- a/src/emu/rendutil.cpp
+++ b/src/emu/rendutil.cpp
@@ -570,11 +570,11 @@ void render_load_jpeg(bitmap_argb32 &bitmap, emu_file &file, const char *dirname
// allocates a buffer for the image
UINT32 jpg_size = file.size();
- unsigned char *jpg_buffer = global_alloc_array(unsigned char, jpg_size);
+ std::unique_ptr<unsigned char[]> jpg_buffer = std::make_unique<unsigned char[]>(jpg_size);
// read data from the file and set them in the buffer
- file.read(jpg_buffer, jpg_size);
- jpeg_mem_src(&cinfo, jpg_buffer, jpg_size);
+ file.read(jpg_buffer.get(), jpg_size);
+ jpeg_mem_src(&cinfo, jpg_buffer.get(), jpg_size);
// read JPEG header and start decompression
jpeg_read_header(&cinfo, TRUE);
@@ -617,7 +617,6 @@ void render_load_jpeg(bitmap_argb32 &bitmap, emu_file &file, const char *dirname
file.close();
free(buffer[0]);
free(buffer);
- global_free_array(jpg_buffer);
}
diff --git a/src/emu/validity.cpp b/src/emu/validity.cpp
index f31cd20a8cd..27bff60cf56 100644
--- a/src/emu/validity.cpp
+++ b/src/emu/validity.cpp
@@ -971,11 +971,11 @@ void validity_checker::validate_devices()
std::unordered_set<std::string> slot_device_map;
for (const device_slot_interface &slot : slot_interface_iterator(m_current_config->root_device()))
{
- for (const device_slot_option &option : slot.option_list())
+ for (auto &option : slot.option_list())
{
std::string temptag("_");
- temptag.append(option.name());
- device_t *dev = const_cast<machine_config &>(*m_current_config).device_add(&m_current_config->root_device(), temptag.c_str(), option.devtype(), 0);
+ temptag.append(option.second->name());
+ device_t *dev = const_cast<machine_config &>(*m_current_config).device_add(&m_current_config->root_device(), temptag.c_str(), option.second->devtype(), 0);
// notify this device and all its subdevices that they are now configured
for (device_t &device : device_iterator(*dev))
diff --git a/src/frontend/mame/clifront.cpp b/src/frontend/mame/clifront.cpp
index a03479cde61..0ee056ec6c7 100644
--- a/src/frontend/mame/clifront.cpp
+++ b/src/frontend/mame/clifront.cpp
@@ -721,16 +721,16 @@ void cli_frontend::listslots(const char *gamename)
bool first_option = true;
// get the options and print them
- for (const device_slot_option &option : slot.option_list())
+ for (auto &option : slot.option_list())
{
- if (option.selectable())
+ if (option.second->selectable())
{
- device_t *dev = (*option.devtype())(drivlist.config(), "dummy", &drivlist.config().root_device(), 0);
+ device_t *dev = (*option.second->devtype())(drivlist.config(), "dummy", &drivlist.config().root_device(), 0);
dev->config_complete();
if (first_option) {
- printf("%-15s %s\n", option.name(),dev->name());
+ printf("%-15s %s\n", option.second->name(),dev->name());
} else {
- printf("%-23s %-15s %s\n", "",option.name(),dev->name());
+ printf("%-23s %-15s %s\n", "", option.second->name(),dev->name());
}
global_free(dev);
@@ -929,11 +929,11 @@ void cli_frontend::verifyroms(const char *gamename)
for (const device_slot_interface &slot : slot_interface_iterator(config.root_device()))
{
- for (const device_slot_option &option : slot.option_list())
+ for (auto &option : slot.option_list())
{
std::string temptag("_");
- temptag.append(option.name());
- device_t *dev = const_cast<machine_config &>(config).device_add(&config.root_device(), temptag.c_str(), option.devtype(), 0);
+ temptag.append(option.second->name());
+ device_t *dev = const_cast<machine_config &>(config).device_add(&config.root_device(), temptag.c_str(), option.second->devtype(), 0);
// notify this device and all its subdevices that they are now configured
for (device_t &device : device_iterator(*dev))
diff --git a/src/frontend/mame/info.cpp b/src/frontend/mame/info.cpp
index e5b1677370e..423eda91680 100644
--- a/src/frontend/mame/info.cpp
+++ b/src/frontend/mame/info.cpp
@@ -438,11 +438,11 @@ void info_xml_creator::output_devices()
// then, run through slot devices
for (const device_slot_interface &slot : slot_interface_iterator(m_drivlist.config().root_device()))
{
- for (const device_slot_option &option : slot.option_list())
+ for (auto &option : slot.option_list())
{
std::string temptag("_");
- temptag.append(option.name());
- device_t *dev = const_cast<machine_config &>(m_drivlist.config()).device_add(&m_drivlist.config().root_device(), temptag.c_str(), option.devtype(), 0);
+ temptag.append(option.second->name());
+ device_t *dev = const_cast<machine_config &>(m_drivlist.config()).device_add(&m_drivlist.config().root_device(), temptag.c_str(), option.second->devtype(), 0);
// notify this device and all its subdevices that they are now configured
for (device_t &device : device_iterator(*dev))
@@ -1528,18 +1528,18 @@ void info_xml_creator::output_slots(device_t &device, const char *root_tag)
fprintf(m_output, " interface=\"%s\"", xml_normalize_string(slot.slot_interface()));
*/
- for (const device_slot_option &option : slot.option_list())
+ for (auto &option : slot.option_list())
{
- if (option.selectable())
+ if (option.second->selectable())
{
- device_t *dev = const_cast<machine_config &>(m_drivlist.config()).device_add(&m_drivlist.config().root_device(), "dummy", option.devtype(), 0);
+ device_t *dev = const_cast<machine_config &>(m_drivlist.config()).device_add(&m_drivlist.config().root_device(), "dummy", option.second->devtype(), 0);
if (!dev->configured())
dev->config_complete();
fprintf(m_output, "\t\t\t<slotoption");
- fprintf(m_output, " name=\"%s\"", xml_normalize_string(option.name()));
+ fprintf(m_output, " name=\"%s\"", xml_normalize_string(option.second->name()));
fprintf(m_output, " devname=\"%s\"", xml_normalize_string(dev->shortname()));
- if (slot.default_option() != nullptr && strcmp(slot.default_option(),option.name())==0)
+ if (slot.default_option() != nullptr && strcmp(slot.default_option(), option.second->name())==0)
fprintf(m_output, " default=\"yes\"");
fprintf(m_output, "/>\n");
const_cast<machine_config &>(m_drivlist.config()).device_remove(&m_drivlist.config().root_device(), "dummy");
diff --git a/src/frontend/mame/ui/slotopt.cpp b/src/frontend/mame/ui/slotopt.cpp
index 2da3a411599..5c97dc594a0 100644
--- a/src/frontend/mame/ui/slotopt.cpp
+++ b/src/frontend/mame/ui/slotopt.cpp
@@ -49,12 +49,12 @@ int menu_slot_devices::slot_get_current_index(device_slot_interface &slot)
if (current != nullptr)
{
int val = 0;
- for (const device_slot_option &option : slot.option_list())
+ for (auto &option : slot.option_list())
{
- if (&option == current)
+ if (option.second.get() == current)
return val;
- if (option.selectable())
+ if (option.second->selectable())
val++;
}
}
@@ -68,8 +68,8 @@ int menu_slot_devices::slot_get_current_index(device_slot_interface &slot)
int menu_slot_devices::slot_get_length(device_slot_interface &slot)
{
int val = 0;
- for (const device_slot_option &option : slot.option_list())
- if (option.selectable())
+ for (auto &option : slot.option_list())
+ if (option.second->selectable())
val++;
return val;
@@ -117,12 +117,12 @@ const char *menu_slot_devices::slot_get_option(device_slot_interface &slot, int
if (index >= 0)
{
int val = 0;
- for (const device_slot_option &option : slot.option_list())
+ for (auto &option : slot.option_list())
{
if (val == index)
- return option.name();
+ return option.second->name();
- if (option.selectable())
+ if (option.second->selectable())
val++;
}
}