UDQDefine: Suppress 'Unused Variable' Warning

Assign 'stod' result to a 'std::ignore' object to maintain the
'token_size' side effect without creating an otherwise unused value
of type 'double'.  While here, also put the 'next_token' helper into
an anonymous namespace to avoid generating warnings of the form

    no previous declaration for ...
This commit is contained in:
Bård Skaflestad
2020-07-01 22:24:02 +02:00
parent d6a3cdb986
commit adf90e3410

View File

@@ -18,7 +18,7 @@
*/
#include <iostream>
#include <cstring>
#include <tuple>
#include <opm/parser/eclipse/Parser/ParseContext.hpp>
@@ -82,7 +82,7 @@ UDQDefine::UDQDefine(const UDQParams& udq_params_arg,
UDQDefine(udq_params_arg, keyword, deck_data, ParseContext(), ErrorGuard())
{}
namespace {
std::optional<std::string> next_token(const std::string& item, std::size_t offset, const std::vector<std::string>& splitters) {
if (offset == item.size())
return {};
@@ -91,7 +91,7 @@ std::optional<std::string> next_token(const std::string& item, std::size_t offse
std::size_t token_size = 0;
try {
auto substring = item.substr(offset);
double value = std::stod(substring, &token_size);
std::ignore = std::stod(substring, &token_size);
} catch (const std::invalid_argument &) {}
if (token_size > 0)
@@ -112,7 +112,7 @@ std::optional<std::string> next_token(const std::string& item, std::size_t offse
}
return token;
}
} // Anonymous namespace
UDQDefine::UDQDefine(const UDQParams& udq_params,
const std::string& keyword,