Fix linter warnings

client/player/player.hpp

@@ -1,6 +1,6 @@
 /***
     This file is part of snapcast
-    Copyright (C) 2014-2024  Johannes Pohl
+    Copyright (C) 2014-2025  Johannes Pohl
 
     This program is free software: you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -51,7 +51,7 @@ public:
     struct Volume
     {
         double volume{1.0}; ///< volume [0..1]
-        bool mute{false}; ///< muted?
+        bool mute{false};   ///< muted?
     };
 
     using volume_callback = std::function<void(const Volume& volume)>;

control/scripts.md

@@ -17,6 +17,7 @@ curl --header "Content-Type: application/json" --request POST --data '{"id":7,"j
 ```
 
 ### Remove all disconnected clients
+
 ```bash
 curl -s  -H "Content-Type: application/json" -d '{"id":1, "jsonrpc":"2.0", "method": "Server.GetStatus"}' http://127.0.0.1:1780/jsonrpc | jq '.result.server.groups[].clients[] | select(.connected==false) .id' | while read ln; do curl -s -H  "Content-Type: application/json" -d '{"id":1, "jsonrpc":"2.0", "method": "Server.DeleteClient", "params": {"id":'$ln'}}' http://127.0.0.1:1780/jsonrpc; done
 ```

server/control_requests.hpp

@@ -43,6 +43,7 @@ public:
     struct Description
     {
         /// c'tor
+        // NOLINTNEXTLINE
         Description(std::string description, std::vector<std::pair<std::string, std::string>> parameters = {}, std::string result = "")
             : description(std::move(description)), parameters(std::move(parameters)), result(std::move(result))
         {

server/stream_server.cpp

@@ -57,14 +57,14 @@ void StreamServer::cleanup()
 }
 
 
-void StreamServer::addSession(const std::shared_ptr<StreamSession>& session)
+void StreamServer::addSession(std::shared_ptr<StreamSession> session)
 {
     session->setMessageReceiver(this);
     session->setBufferMs(settings_.stream.bufferMs);
     session->start();
 
     std::lock_guard<std::recursive_mutex> mlock(sessionsMutex_);
-    sessions_.emplace_back(session);
+    sessions_.emplace_back(std::move(session));
     cleanup();
 }
 
@@ -210,7 +210,7 @@ void StreamServer::handleAccept(tcp::socket socket)
 
         LOG(NOTICE, LOG_TAG) << "StreamServer::NewConnection: " << socket.remote_endpoint().address().to_string() << "\n";
         shared_ptr<StreamSession> session = make_shared<StreamSessionTcp>(this, settings_, std::move(socket));
-        addSession(session);
+        addSession(std::move(session));
     }
     catch (const std::exception& e)
     {

server/stream_server.hpp

@@ -67,7 +67,7 @@ public:
     //	void send(const msg::BaseMessage* message);
 
     /// Add a new stream session
-    void addSession(const std::shared_ptr<StreamSession>& session);
+    void addSession(std::shared_ptr<StreamSession> session);
     /// Callback for chunks that are ready to be sent
     void onChunkEncoded(const PcmStream* pcmStream, bool isDefaultStream, const std::shared_ptr<msg::PcmChunk>& chunk, double duration);