aboutsummaryrefslogtreecommitdiffstats
path: root/al/eax/exception.cpp
diff options
context:
space:
mode:
authorChris Robinson <[email protected]>2023-08-25 14:52:09 -0700
committerChris Robinson <[email protected]>2023-08-25 14:52:09 -0700
commitf1a67347a6cf4b6d397fe1220b999ed7161c7097 (patch)
treeda48b7cd24ef0622e6c47f4bb368ac21bb71d776 /al/eax/exception.cpp
parentca3bc1bd80fdff511e83d563a4ee94d6cd885473 (diff)
Use string_view in a couple more places
Diffstat (limited to 'al/eax/exception.cpp')
-rw-r--r--al/eax/exception.cpp43
1 files changed, 12 insertions, 31 deletions
diff --git a/al/eax/exception.cpp b/al/eax/exception.cpp
index 435e7442..cd32e11a 100644
--- a/al/eax/exception.cpp
+++ b/al/eax/exception.cpp
@@ -6,54 +6,35 @@
#include <string>
-EaxException::EaxException(const char *context, const char *message)
+EaxException::EaxException(std::string_view context, std::string_view message)
: std::runtime_error{make_message(context, message)}
{
}
EaxException::~EaxException() = default;
-std::string EaxException::make_message(const char *context, const char *message)
+std::string EaxException::make_message(std::string_view context, std::string_view message)
{
- const auto context_size = (context ? std::string::traits_type::length(context) : 0);
- const auto has_contex = (context_size > 0);
-
- const auto message_size = (message ? std::string::traits_type::length(message) : 0);
- const auto has_message = (message_size > 0);
-
- if (!has_contex && !has_message)
- {
- return std::string{};
- }
+ auto what = std::string{};
+ if(context.empty() && message.empty())
+ return what;
static constexpr char left_prefix[] = "[";
- const auto left_prefix_size = std::string::traits_type::length(left_prefix);
+ static constexpr auto left_prefix_size = std::string::traits_type::length(left_prefix);
static constexpr char right_prefix[] = "] ";
- const auto right_prefix_size = std::string::traits_type::length(right_prefix);
+ static constexpr auto right_prefix_size = std::string::traits_type::length(right_prefix);
- const auto what_size =
- (
- has_contex ?
- left_prefix_size + context_size + right_prefix_size :
- 0) +
- message_size +
- 1;
+ what.reserve((!context.empty() ? left_prefix_size + context.size() + right_prefix_size : 0) +
+ message.length() + 1);
- auto what = std::string{};
- what.reserve(what_size);
-
- if (has_contex)
+ if(!context.empty())
{
what.append(left_prefix, left_prefix_size);
- what.append(context, context_size);
+ what += context;
what.append(right_prefix, right_prefix_size);
}
-
- if (has_message)
- {
- what.append(message, message_size);
- }
+ what += message;
return what;
}