diff options
author | Aaron Giles <aaron@aarongiles.com> | 2010-06-08 06:09:57 +0000 |
---|---|---|
committer | Aaron Giles <aaron@aarongiles.com> | 2010-06-08 06:09:57 +0000 |
commit | 100564d41225bfd2fa163c0174557ed979285b4e (patch) | |
tree | 0be83fbde1ddb95633293c9faee60d0f2bc6ace1 /src/emu/inptport.c | |
parent | 962dca9f57d61cdea65ee32c6362bf3d0d1e2dae (diff) |
WARNING: There are likely to be regressions in both functionality and
performance as a result of this change. Do not panic; report issues to the
list in the short term and I will look into them. There are probably also
some details I forgot to mention. Please ask questions if anything is not
clear.
NOTE: This is a major internal change to the way devices are handled in
MAME. There is a small impact on drivers, but the bulk of the changes are
to the devices themselves. Full documentation on the new device handling
is in progress at http://mamedev.org/devwiki/index.php/MAME_Device_Basics
Defined two new casting helpers: [Aaron Giles]
downcast<type>(value) should be used for safe and efficient downcasting
from a base class to a derived class. It wraps static_cast<> by adding
an assert that a matching dynamic_cast<> returns the same result in
debug builds.
crosscast<type>(value) should be used for safe casting from one type to
another in multiple inheritance scenarios. It compiles to a
dynamic_cast<> plus an assert on the result. Since it does not optimize
down to static_cast<>, you should prefer downcast<> over crosscast<>
when you can.
Redefined running_device to be a proper C++ class (now called device_t).
Same for device_config (still called device_config). All devices and
device_configs must now be derived from these base classes. This means
each device type now has a pair of its own unique classes that describe
the device. Drivers are encouraged to use the specific device types
instead of the generic running_device or device_t classes. Drivers that
have a state class defined in their header file are encouraged to use
initializers off the constructor to locate devices. [Aaron Giles]
Removed the following fields from the device and device configuration
classes as they never were necessary or provided any use: device class,
device family, source file, version, credits. [Aaron Giles]
Added templatized variant of machine->device() which performs a downcast
as part of the device fetch. Thus machine->device<timer_device>("timer")
will locate a device named "timer", downcast it to a timer_device, and
assert if the downcast fails. [Aaron Giles]
Removed most publically accessible members of running_device/device_t in
favor of inline accessor functions. The only remaining public member is
machine. Thus all references to device->type are now device->type(), etc.
[Aaron Giles]
Created a number of device interface classes which are designed to be mix-
ins for the device classes, providing specific extended functionality and
information. There are standard interface classes for sound, execution,
state, nvram, memory, and disassembly. Devices can opt into 0 or more of
these classes. [Aaron Giles]
Converted the classic CPU device to a standard device that uses the
execution, state, memory, and disassembly interfaces. Used this new class
(cpu_device) to implement the existing CPU device interface. In the future
it will be possible to convert each CPU core to its own device type, but
for now they are still all CPU devices with a cpu_type() that specifies
exactly which kind of CPU. [Aaron Giles]
Created a new header devlegcy.h which wraps the old device interface using
some special template classes. To use these with an existing device,
simply remove from the device header the DEVICE_GET_INFO() declaration and
the #define mapping the ALL_CAPS name to the DEVICE_GET_INFO. In their
place #include "devlegcy.h" and use the DECLARE_LEGACY_DEVICE() macro.
In addition, there is a DECLARE_LEGACY_SOUND_DEVICE() macro for wrapping
existing sound devices into new-style devices, and a
DECLARE_LEGACY_NVRAM_DEVICE() for wrapping NVRAM devices. Also moved the
token and inline_config members to the legacy device class, as these are
not used in modern devices. [Aaron Giles]
Converted the standard base devices (VIDEO_SCREEN, SPEAKER, and TIMER)
from legacy devices to the new C++ style. Also renamed VIDEO_SCREEN to
simply SCREEN. The various global functions that were previously used to
access information or modify the state of these devices are now replaced
by methods on the device classes. Specifically:
video_screen_configure() == screen->configure()
video_screen_set_visarea() == screen->set_visible_area()
video_screen_update_partial() == screen->update_partial()
video_screen_update_now() == screen->update_now()
video_screen_get_vpos() == screen->vpos()
video_screen_get_hpos() == screen->hpos()
video_screen_get_vblank() == screen->vblank()
video_screen_get_hblank() == screen->hblank()
video_screen_get_width() == screen->width()
video_screen_get_height() == screen->height()
video_screen_get_visible_area() == screen->visible_area()
video_screen_get_time_until_pos() == screen->time_until_pos()
video_screen_get_time_until_vblank_start() ==
screen->time_until_vblank_start()
video_screen_get_time_until_vblank_end() ==
screen->time_until_vblank_end()
video_screen_get_time_until_update() == screen->time_until_update()
video_screen_get_scan_period() == screen->scan_period()
video_screen_get_frame_period() == screen->frame_period()
video_screen_get_frame_number() == screen->frame_number()
timer_device_adjust_oneshot() == timer->adjust()
timer_device_adjust_periodic() == timer->adjust()
timer_device_reset() == timer->reset()
timer_device_enable() == timer->enable()
timer_device_enabled() == timer->enabled()
timer_device_get_param() == timer->param()
timer_device_set_param() == timer->set_param()
timer_device_get_ptr() == timer->get_ptr()
timer_device_set_ptr() == timer->set_ptr()
timer_device_timeelapsed() == timer->time_elapsed()
timer_device_timeleft() == timer->time_left()
timer_device_starttime() == timer->start_time()
timer_device_firetime() == timer->fire_time()
Updated all drivers that use the above functions to fetch the specific
device type (timer_device or screen_device) and call the appropriate
method. [Aaron Giles]
Changed machine->primary_screen and the 'screen' parameter to VIDEO_UPDATE
to specifically pass in a screen_device object. [Aaron Giles]
Defined a new custom interface for the Z80 daisy chain. This interface
behaves like the standard interfaces, and can be added to any device that
implements the Z80 daisy chain behavior. Converted all existing Z80 daisy
chain devices to new-style devices that inherit this interface.
[Aaron Giles]
Changed the way CPU state tables are built up. Previously, these were data
structures defined by a CPU core which described all the registers and how
to output them. This functionality is now part of the state interface and
is implemented via the device_state_entry class. Updated all CPU cores
which were using the old data structure to use the new form. The syntax is
currently awkward, but will be cleaner for CPUs that are native new
devices. [Aaron Giles]
Converted the okim6295 and eeprom devices to the new model. These were
necessary because they both require multiple interfaces to operate and it
didn't make sense to create legacy device templates for these single cases.
(okim6295 needs the sound interface and the memory interface, while eeprom
requires both the nvram and memory interfaces). [Aaron Giles]
Changed parameters in a few callback functions from pointers to references
in situations where they are guaranteed to never be NULL. [Aaron Giles]
Removed MDRV_CPU_FLAGS() which was only used for disabling a CPU. Changed
it to MDRV_DEVICE_DISABLE() instead. Updated drivers. [Aaron Giles]
Reorganized the token parsing for machine configurations. The core parsing
code knows how to create/replace/remove devices, but all device token
parsing is now handled in the device_config class, which in turn will make
use of any interface classes or device-specific token handling for custom
token processing. [Aaron Giles]
Moved many validity checks out of validity.c and into the device interface
classes. For example, address space validation is now part of the memory
interface class. [Aaron Giles]
Consolidated address space parameters (bus width, endianness, etc.) into
a single address_space_config class. Updated all code that queried for
address space parameters to use the new mechanism. [Aaron Giles]
Diffstat (limited to 'src/emu/inptport.c')
-rw-r--r-- | src/emu/inptport.c | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/src/emu/inptport.c b/src/emu/inptport.c index 028ed3551b5..ac1975cb35b 100644 --- a/src/emu/inptport.c +++ b/src/emu/inptport.c @@ -202,7 +202,7 @@ struct _device_field_info { device_field_info * next; /* linked list of info for this port */ const input_field_config * field; /* pointer to the input field referenced */ - running_device * device; /* device */ + device_t * device; /* device */ UINT8 shift; /* shift to apply to the final result */ input_port_value oldval; /* last value */ }; @@ -888,7 +888,7 @@ INLINE const char *get_port_tag(const input_port_config *port, char *tempbuffer) if (port->tag != NULL) return port->tag; - for (curport = port->machine->portlist.first(); curport != NULL; curport = curport->next) + for (curport = port->machine->portlist.first(); curport != NULL; curport = curport->next()) { if (curport == port) break; @@ -1515,7 +1515,7 @@ input_port_value input_port_read_direct(const input_port_config *port) /* update VBLANK bits */ if (port->state->vblank != 0) { - if (video_screen_get_vblank(port->machine->primary_screen)) + if (port->machine->primary_screen->vblank()) result |= port->state->vblank; else result &= ~port->state->vblank; @@ -1597,7 +1597,7 @@ int input_port_get_crosshair_position(running_machine *machine, int player, floa int gotx = FALSE, goty = FALSE; /* read all the lightgun values */ - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) for (field = port->fieldlist; field != NULL; field = field->next) if (field->player == player && field->crossaxis != CROSSHAIR_AXIS_NONE) if (input_condition_true(machine, &field->condition)) @@ -1666,7 +1666,7 @@ void input_port_update_defaults(running_machine *machine) const input_port_config *port; /* loop over all input ports */ - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) { const input_field_config *field; @@ -2016,7 +2016,7 @@ static void init_port_state(running_machine *machine) const input_port_config *port; /* allocate live structures to mirror the configuration */ - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) { analog_field_state **analogstatetail; device_field_info **readdevicetail; @@ -2103,7 +2103,7 @@ static void init_port_state(running_machine *machine) /* look for 4-way joysticks and change the default map if we find any */ if (joystick_map_default[0] == 0 || strcmp(joystick_map_default, "auto") == 0) - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) for (field = port->fieldlist; field != NULL; field = field->next) if (field->state->joystick != NULL && field->way == 4) { @@ -2157,7 +2157,7 @@ static void init_autoselect_devices(const ioport_list &portlist, int type1, int /* only scan the list if we haven't already enabled this class of control */ if (portlist.first() != NULL && !input_device_class_enabled(portlist.first()->machine, autoenable)) - for (port = portlist.first(); port != NULL; port = port->next) + for (port = portlist.first(); port != NULL; port = port->next()) for (field = port->fieldlist; field != NULL; field = field->next) /* if this port type is in use, apply the autoselect criteria */ @@ -2193,7 +2193,7 @@ static device_field_info *init_field_device_info(const input_field_config *field if (device_name != NULL) info->device = field->port->machine->device(device_name); else - info->device = (running_device *) info; + info->device = (device_t *) info; info->oldval = field->defvalue >> info->shift; return info; @@ -2495,7 +2495,7 @@ profiler_mark_start(PROFILER_INPUT); } /* loop over all input ports */ - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) { const input_field_config *field; device_field_info *device_field; @@ -3515,7 +3515,7 @@ static void port_config_detokenize(ioport_list &portlist, const input_port_token -------------------------------------------------*/ input_port_config::input_port_config(const char *_tag) - : next(NULL), + : m_next(NULL), tag(_tag), fieldlist(NULL), state(NULL), @@ -4068,7 +4068,7 @@ static int load_game_config(running_machine *machine, xml_data_node *portnode, i defvalue = xml_get_attribute_int(portnode, "defvalue", 0); /* find the port we want; if no tag, search them all */ - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) if (tag == NULL || strcmp(get_port_tag(port, tempbuffer), tag) == 0) for (field = port->fieldlist; field != NULL; field = field->next) @@ -4230,7 +4230,7 @@ static void save_game_inputs(running_machine *machine, xml_data_node *parentnode const input_port_config *port; /* iterate over ports */ - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) for (field = port->fieldlist; field != NULL; field = field->next) if (save_this_input_field_type(field->type)) { @@ -4708,7 +4708,7 @@ int input_machine_has_keyboard(running_machine *machine) #ifdef MESS const input_field_config *field; const input_port_config *port; - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) { for (field = port->fieldlist; field != NULL; field = field->next) { @@ -4789,7 +4789,7 @@ static int scan_keys(running_machine *machine, const input_port_config *portconf assert(keys < NUM_SIMUL_KEYS); - for (port = portconfig; port != NULL; port = port->next) + for (port = portconfig; port != NULL; port = port->next()) { for (field = port->fieldlist; field != NULL; field = field->next) { @@ -5408,7 +5408,7 @@ int input_has_input_class(running_machine *machine, int inputclass) const input_port_config *port; const input_field_config *field; - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) { for (field = port->fieldlist; field != NULL; field = field->next) { @@ -5433,7 +5433,7 @@ int input_count_players(running_machine *machine) int joystick_count; joystick_count = 0; - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) { for (field = port->fieldlist; field != NULL; field = field->next) { @@ -5464,7 +5464,7 @@ int input_category_active(running_machine *machine, int category) assert(category >= 1); /* loop through the input ports */ - for (port = machine->portlist.first(); port != NULL; port = port->next) + for (port = machine->portlist.first(); port != NULL; port = port->next()) { for (field = port->fieldlist; field != NULL; field = field->next) { |