Prefer icinga::String::GetData() over icinga::String::CStr()

Creating the string_view from the std::string (as returned by GetData()) uses
the stored length instead of having to detect it by finding '\0'.
This commit is contained in:
Julian Brost 2025-04-11 12:27:25 +02:00
parent fb2b2e2d5b
commit ccfc72267f
2 changed files with 3 additions and 3 deletions

View File

@ -41,7 +41,7 @@ void Connect(Socket& socket, const String& node, const String& service)
using boost::asio::ip::tcp;
tcp::resolver resolver (IoEngine::Get().GetIoContext());
auto result (resolver.resolve(node.CStr(), service.CStr()));
auto result (resolver.resolve(node.GetData(), service.GetData()));
auto current (result.begin());
for (;;) {
@ -71,7 +71,7 @@ void Connect(Socket& socket, const String& node, const String& service, boost::a
using boost::asio::ip::tcp;
tcp::resolver resolver (IoEngine::Get().GetIoContext());
auto result (resolver.async_resolve(node.CStr(), service.CStr(), yc));
auto result (resolver.async_resolve(node.GetData(), service.GetData(), yc));
auto current (result.begin());
for (;;) {

View File

@ -439,7 +439,7 @@ bool ApiListener::AddListener(const String& node, const String& service)
try {
tcp::resolver resolver (io);
auto result (resolver.resolve(node.CStr(), service.CStr(), tcp::resolver::passive));
auto result (resolver.resolve(node.GetData(), service.GetData(), tcp::resolver::passive));
auto current (result.begin());
for (;;) {