summaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
-rw-r--r--src/frontend/mame/clifront.cpp6
-rw-r--r--src/lib/util/options.cpp28
2 files changed, 22 insertions, 12 deletions
diff --git a/src/frontend/mame/clifront.cpp b/src/frontend/mame/clifront.cpp
index d48ab54ffef..2adb4e3ecc1 100644
--- a/src/frontend/mame/clifront.cpp
+++ b/src/frontend/mame/clifront.cpp
@@ -217,7 +217,10 @@ void cli_frontend::start_execution(mame_machine_manager *manager, const std::vec
try
{
m_options.parse_command_line(args, OPTION_PRIORITY_CMDLINE);
- m_osd.set_verbose(m_options.verbose());
+ }
+ catch (options_warning_exception &ex)
+ {
+ osd_printf_error("%s", ex.message());
}
catch (options_exception &ex)
{
@@ -228,6 +231,7 @@ void cli_frontend::start_execution(mame_machine_manager *manager, const std::vec
// otherwise, error on the options
throw emu_fatalerror(EMU_ERR_INVALID_CONFIG, "%s", ex.message());
}
+ m_osd.set_verbose(m_options.verbose());
// determine the base name of the EXE
std::string_view exename = core_filename_extract_base(args[0], true);
diff --git a/src/lib/util/options.cpp b/src/lib/util/options.cpp
index bf1a4e45b20..8b7e8018528 100644
--- a/src/lib/util/options.cpp
+++ b/src/lib/util/options.cpp
@@ -158,6 +158,8 @@ void core_options::entry::set_value(std::string &&newvalue, int priority_value,
// it is invalid to set the value on a header
assert(type() != option_type::HEADER);
+ validate(newvalue);
+
// only set the value if we have priority
if (always_override || priority_value >= priority())
{
@@ -213,21 +215,23 @@ void core_options::entry::validate(const std::string &data)
char const *const strmin(minimum());
char const *const strmax(maximum());
int imin(0), imax(0);
- if (strmin)
+ if (strmin && *strmin)
{
str.str(strmin);
+ str.seekg(0);
str >> imin;
}
- if (strmax)
+ if (strmax && *strmax)
{
str.str(strmax);
+ str.seekg(0);
str >> imax;
}
- if ((strmin && (ival < imin)) || (strmax && (ival > imax)))
+ if ((strmin && *strmin && (ival < imin)) || (strmax && *strmax && (ival > imax)))
{
- if (!strmax)
+ if (!strmax || !*strmax)
throw options_warning_exception("Out-of-range integer value for %s: \"%s\" (must be no less than %d); reverting to %s\n", name(), data, imin, value());
- else if (!strmin)
+ else if (!strmin || !*strmin)
throw options_warning_exception("Out-of-range integer value for %s: \"%s\" (must be no greater than %d); reverting to %s\n", name(), data, imax, value());
else
throw options_warning_exception("Out-of-range integer value for %s: \"%s\" (must be between %d and %d, inclusive); reverting to %s\n", name(), data, imin, imax, value());
@@ -245,21 +249,23 @@ void core_options::entry::validate(const std::string &data)
char const *const strmin(minimum());
char const *const strmax(maximum());
float fmin(0), fmax(0);
- if (strmin)
+ if (strmin && *strmin)
{
str.str(strmin);
+ str.seekg(0);
str >> fmin;
}
- if (strmax)
+ if (strmax && *strmax)
{
str.str(strmax);
+ str.seekg(0);
str >> fmax;
}
- if ((strmin && (fval < fmin)) || (strmax && (fval > fmax)))
+ if ((strmin && *strmin && (fval < fmin)) || (strmax && *strmax && (fval > fmax)))
{
- if (!strmax)
+ if (!strmax || !*strmax)
throw options_warning_exception("Out-of-range float value for %s: \"%s\" (must be no less than %f); reverting to %s\n", name(), data, fmin, value());
- else if (!strmin)
+ else if (!strmin || !*strmin)
throw options_warning_exception("Out-of-range float value for %s: \"%s\" (must be no greater than %f); reverting to %s\n", name(), data, fmax, value());
else
throw options_warning_exception("Out-of-range float value for %s: \"%s\" (must be between %f and %f, inclusive); reverting to %s\n", name(), data, fmin, fmax, value());
@@ -1136,7 +1142,7 @@ bool core_options::header_exists(const char *description) const noexcept
void core_options::revert(int priority_hi, int priority_lo)
{
for (entry::shared_ptr &curentry : m_entries)
- if (curentry->type() != option_type::HEADER)
+ if (curentry->type() != option_type::HEADER && curentry->type() != option_type::COMMAND)
curentry->revert(priority_hi, priority_lo);
}