Remove unused includes, fix warnings

badaix · Dec 19, 2024 · 06f1c7d · 06f1c7d
1 parent a79b3e6
commit 06f1c7d
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -1,5 +1,10 @@
 cmake_minimum_required(VERSION 3.14)
 
+# find_package() uses uppercase <PackageName>_ROOT variables.
+if(POLICY CMP0144)
+  cmake_policy(SET CMP0144 NEW)
+endif()
+
 project(
   snapcast
   LANGUAGES CXX

diff --git a/server/control_session_http.cpp b/server/control_session_http.cpp
@@ -449,11 +449,10 @@ void ControlSessionHttp::on_read(beast::error_code ec, std::size_t bytes_transfe
         return;
     }
 
-
-    std::string_view authheader = req_[beast::http::field::authorization];
+    auto authheader = req_[beast::http::field::authorization];
     if (!authheader.empty())
     {
-        auto ec = authinfo.authenticate(std::string(authheader));
+        auto ec = authinfo.authenticate(std::string{authheader});
         if (ec)
         {
             LOG(ERROR, LOG_TAG) << "Authentication failed: " << ec.detailed_message() << "\n";

diff --git a/server/stream_server.cpp b/server/stream_server.cpp
@@ -21,9 +21,6 @@
 
 // local headers
 #include "common/aixlog.hpp"
-#include "common/message/client_info.hpp"
-#include "common/message/hello.hpp"
-#include "common/message/time.hpp"
 #include "config.hpp"
 #include "stream_session_tcp.hpp"
 

diff --git a/server/stream_session_tcp.cpp b/server/stream_session_tcp.cpp
@@ -21,7 +21,6 @@
 
 // local headers
 #include "common/aixlog.hpp"
-#include "common/message/pcm_chunk.hpp"
 
 // 3rd party headers