oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

output/ao: use std::string_view

Browse Source
This commit is contained in:
Max Kellermann 2022-06-30 20:58:42 +02:00
parent 6440df60aa
commit c953ed48b7
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/output/plugins/AoOutputPlugin.cxx
View File

@ -25,6 +25,8 @@
#include "util/RuntimeError.hxx"
#include "util/Domain.hxx"
#include "util/StringAPI.hxx"
#include "util/StringSplit.hxx"
#include "util/StringStrip.hxx"
#include "Log.hxx"
#include <ao/ao.h>
@ -127,16 +129,14 @@ AoOutput::AoOutput(const ConfigBlock &block)
value = block.GetBlockValue("options", nullptr);
if (value != nullptr) {
for (StringView i : IterableSplitString(value, ';')) {
i.Strip();
auto s = i.Split('=');
if (s.first.empty() || s.second.IsNull())
for (const std::string_view i : IterableSplitString(value, ';')) {
const auto [n, v] = Split(Strip(i), '=');
if (n.empty() || v.data() == nullptr)
throw FormatRuntimeError("problems parsing option \"%.*s\"",
int(i.size), i.data);
int(i.size()), i.data());
const std::string n(s.first), v(s.second);
ao_append_option(&options, n.c_str(), v.c_str());
ao_append_option(&options, std::string{n}.c_str(),
std::string{v}.c_str());
}
}
}