summaryrefslogtreecommitdiffstats
path: root/src/emu
diff options
context:
space:
mode:
author AJR <ajrhacker@users.noreply.github.com>2019-06-27 10:38:29 -0400
committer AJR <ajrhacker@users.noreply.github.com>2019-06-27 10:38:29 -0400
commitf9745cdafce3120eab8c82e63499865f8a4ed18c (patch)
tree6891a9a93aa1555707f9350688103955595efb3f /src/emu
parenta620f24ffc3b8bb9abea7bd3b337856a1ee466cd (diff)
devcb.h: Revert unnecessary change from ed0202220a5243537f5ba7dbc0a38e52a8390d79 (nw)
isnull() checks the list of unresolved functions along with the list of resolved ones, so resolve_safe() doesn't really need a return value as long as it is called second.
Diffstat (limited to 'src/emu')
-rw-r--r--src/emu/devcb.h16
1 files changed, 6 insertions, 10 deletions
diff --git a/src/emu/devcb.h b/src/emu/devcb.h
index 1ac9cd1e37f..6b16c37b33a 100644
--- a/src/emu/devcb.h
+++ b/src/emu/devcb.h
@@ -899,7 +899,7 @@ public:
virtual void validity_check(validity_checker &valid) const override;
void resolve();
- bool resolve_safe(Result dflt);
+ void resolve_safe(Result dflt);
Result operator()(address_space &space, offs_t offset = 0, std::make_unsigned_t<Result> mem_mask = DefaultMask);
Result operator()(offs_t offset, std::make_unsigned_t<Result> mem_mask = DefaultMask);
@@ -958,13 +958,11 @@ void devcb_read<Result, DefaultMask>::resolve()
}
template <typename Result, std::make_unsigned_t<Result> DefaultMask>
-bool devcb_read<Result, DefaultMask>::resolve_safe(Result dflt)
+void devcb_read<Result, DefaultMask>::resolve_safe(Result dflt)
{
resolve();
- bool resolved = !m_functions.empty();
- if (!resolved)
+ if (m_functions.empty())
m_functions.emplace_back([dflt] (address_space &space, offs_t offset, std::make_unsigned_t<Result> mem_mask) { return dflt; });
- return resolved;
}
template <typename Result, std::make_unsigned_t<Result> DefaultMask>
@@ -2371,7 +2369,7 @@ public:
virtual void validity_check(validity_checker &valid) const override;
void resolve();
- bool resolve_safe();
+ void resolve_safe();
void operator()(address_space &space, offs_t offset, Input data, std::make_unsigned_t<Input> mem_mask = DefaultMask);
void operator()(address_space &space, Input data);
@@ -2422,13 +2420,11 @@ void devcb_write<Input, DefaultMask>::resolve()
}
template <typename Input, std::make_unsigned_t<Input> DefaultMask>
-bool devcb_write<Input, DefaultMask>::resolve_safe()
+void devcb_write<Input, DefaultMask>::resolve_safe()
{
resolve();
- bool resolved = !m_functions.empty();
- if (!resolved)
+ if (m_functions.empty())
m_functions.emplace_back([] (address_space &space, offs_t offset, Input data, std::make_unsigned_t<Input> mem_mask) { });
- return resolved;
}
template <typename Input, std::make_unsigned_t<Input> DefaultMask>