summaryrefslogtreecommitdiffstatshomepage
path: root/src/emu/emuopts.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'src/emu/emuopts.cpp')
-rw-r--r--src/emu/emuopts.cpp7
1 files changed, 4 insertions, 3 deletions
diff --git a/src/emu/emuopts.cpp b/src/emu/emuopts.cpp
index e3de65b14bd..c42fcef8482 100644
--- a/src/emu/emuopts.cpp
+++ b/src/emu/emuopts.cpp
@@ -237,7 +237,7 @@ namespace
{
}
- virtual const char *value() const override
+ virtual const char *value() const noexcept override
{
// This is returning an empty string instead of nullptr to signify that
// specifying the value is a meaningful operation. The option types that
@@ -287,7 +287,7 @@ namespace
{
}
- virtual const char *value() const override
+ virtual const char *value() const noexcept override
{
const char *result = nullptr;
if (m_host.specified())
@@ -298,6 +298,7 @@ namespace
// happen in practice
//
// In reality, I want to really return std::optional<std::string> here
+ // FIXME: the std::string assignment can throw exceptions, and returning std::optional<std::string> also isn't safe in noexcept
m_temp = m_host.specified_value();
result = m_temp.c_str();
}
@@ -325,7 +326,7 @@ namespace
{
}
- virtual const char *value() const override
+ virtual const char *value() const noexcept override
{
return m_host.value().c_str();
}