Cleanup: Avoid parameter copying overhead

28 files changed, 136 insertions, 136 deletions

diff --git a/src/server/events/Connect.cpp b/src/server/events/Connect.cpp
index d27e1e3b..23a7d4b4 100644
--- a/src/server/events/Connect.cpp
+++ b/src/server/events/Connect.cpp
@@ -38,10 +38,10 @@ namespace ingen {
 namespace server {
 namespace events {
 
-Connect::Connect(Engine&               engine,
-                 SPtr<Interface>       client,
-                 SampleCount           timestamp,
-                 const ingen::Connect& msg)
+Connect::Connect(Engine&                engine,
+                 const SPtr<Interface>& client,
+                 SampleCount            timestamp,
+                 const ingen::Connect&  msg)
 	: Event(engine, client, msg.seq, timestamp)
 	, _msg(msg)
 	, _graph(nullptr)
diff --git a/src/server/events/Connect.hpp b/src/server/events/Connect.hpp
index 625fa39d..adc80afc 100644
--- a/src/server/events/Connect.hpp
+++ b/src/server/events/Connect.hpp
@@ -38,10 +38,10 @@ namespace events {
 class Connect : public Event
 {
 public:
-	Connect(Engine&               engine,
-	        SPtr<Interface>       client,
-	        SampleCount           timestamp,
-	        const ingen::Connect& msg);
+	Connect(Engine&                engine,
+	        const SPtr<Interface>& client,
+	        SampleCount            timestamp,
+	        const ingen::Connect&  msg);
 
 	bool pre_process(PreProcessContext& ctx) override;
 	void execute(RunContext& context) override;
diff --git a/src/server/events/Copy.cpp b/src/server/events/Copy.cpp
index 38cbc6d8..a8e08924 100644
--- a/src/server/events/Copy.cpp
+++ b/src/server/events/Copy.cpp
@@ -36,10 +36,10 @@ namespace ingen {
 namespace server {
 namespace events {
 
-Copy::Copy(Engine&            engine,
-           SPtr<Interface>    client,
-           SampleCount        timestamp,
-           const ingen::Copy& msg)
+Copy::Copy(Engine&                engine,
+           const SPtr<Interface>& client,
+           SampleCount            timestamp,
+           const ingen::Copy&     msg)
 	: Event(engine, client, msg.seq, timestamp)
 	, _msg(msg)
 	, _old_block(nullptr)
diff --git a/src/server/events/Copy.hpp b/src/server/events/Copy.hpp
index 86a95e9c..26673a55 100644
--- a/src/server/events/Copy.hpp
+++ b/src/server/events/Copy.hpp
@@ -35,10 +35,10 @@ namespace events {
 class Copy : public Event
 {
 public:
-	Copy(Engine&            engine,
-	     SPtr<Interface>    client,
-	     SampleCount        timestamp,
-	     const ingen::Copy& msg);
+	Copy(Engine&                engine,
+	     const SPtr<Interface>& client,
+	     SampleCount            timestamp,
+	     const ingen::Copy&     msg);
 
 	bool pre_process(PreProcessContext& ctx) override;
 	void execute(RunContext& context) override;
diff --git a/src/server/events/CreateBlock.cpp b/src/server/events/CreateBlock.cpp
index cfeb0e3d..e8816954 100644
--- a/src/server/events/CreateBlock.cpp
+++ b/src/server/events/CreateBlock.cpp
@@ -38,12 +38,12 @@ namespace ingen {
 namespace server {
 namespace events {
 
-CreateBlock::CreateBlock(Engine&           engine,
-                         SPtr<Interface>   client,
-                         int32_t           id,
-                         SampleCount       timestamp,
-                         const Raul::Path& path,
-                         Properties&       properties)
+CreateBlock::CreateBlock(Engine&                engine,
+                         const SPtr<Interface>& client,
+                         int32_t                id,
+                         SampleCount            timestamp,
+                         const Raul::Path&      path,
+                         Properties&            properties)
 	: Event(engine, client, id, timestamp)
 	, _path(path)
 	, _properties(properties)
diff --git a/src/server/events/CreateBlock.hpp b/src/server/events/CreateBlock.hpp
index bf34f6e2..00f58008 100644
--- a/src/server/events/CreateBlock.hpp
+++ b/src/server/events/CreateBlock.hpp
@@ -38,12 +38,12 @@ namespace events {
 class CreateBlock : public Event
 {
 public:
-	CreateBlock(Engine&           engine,
-	            SPtr<Interface>   client,
-	            int32_t           id,
-	            SampleCount       timestamp,
-	            const Raul::Path& path,
-	            Properties&       properties);
+	CreateBlock(Engine&                engine,
+	            const SPtr<Interface>& client,
+	            int32_t                id,
+	            SampleCount            timestamp,
+	            const Raul::Path&      path,
+	            Properties&            properties);
 
 	bool pre_process(PreProcessContext& ctx) override;
 	void execute(RunContext& context) override;
diff --git a/src/server/events/CreateGraph.cpp b/src/server/events/CreateGraph.cpp
index 5561d147..3e847bdd 100644
--- a/src/server/events/CreateGraph.cpp
+++ b/src/server/events/CreateGraph.cpp
@@ -35,12 +35,12 @@ namespace ingen {
 namespace server {
 namespace events {
 
-CreateGraph::CreateGraph(Engine&           engine,
-                         SPtr<Interface>   client,
-                         int32_t           id,
-                         SampleCount       timestamp,
-                         const Raul::Path& path,
-                         const Properties& properties)
+CreateGraph::CreateGraph(Engine&                engine,
+                         const SPtr<Interface>& client,
+                         int32_t                id,
+                         SampleCount            timestamp,
+                         const Raul::Path&      path,
+                         const Properties&      properties)
 	: Event(engine, client, id, timestamp)
 	, _path(path)
 	, _properties(properties)
diff --git a/src/server/events/CreateGraph.hpp b/src/server/events/CreateGraph.hpp
index 6ea07957..dadb644e 100644
--- a/src/server/events/CreateGraph.hpp
+++ b/src/server/events/CreateGraph.hpp
@@ -40,12 +40,12 @@ namespace events {
 class CreateGraph : public Event
 {
 public:
-	CreateGraph(Engine&           engine,
-	            SPtr<Interface>   client,
-	            int32_t           id,
-	            SampleCount       timestamp,
-	            const Raul::Path& path,
-	            const Properties& properties);
+	CreateGraph(Engine&                engine,
+	            const SPtr<Interface>& client,
+	            int32_t                id,
+	            SampleCount            timestamp,
+	            const Raul::Path&      path,
+	            const Properties&      properties);
 
 	bool pre_process(PreProcessContext& ctx) override;
 	void execute(RunContext& context) override;
diff --git a/src/server/events/CreatePort.cpp b/src/server/events/CreatePort.cpp
index 09eb8cff..d1b4f114 100644
--- a/src/server/events/CreatePort.cpp
+++ b/src/server/events/CreatePort.cpp
@@ -40,12 +40,12 @@ namespace ingen {
 namespace server {
 namespace events {
 
-CreatePort::CreatePort(Engine&           engine,
-                       SPtr<Interface>   client,
-                       int32_t           id,
-                       SampleCount       timestamp,
-                       const Raul::Path& path,
-                       const Properties& properties)
+CreatePort::CreatePort(Engine&                engine,
+                       const SPtr<Interface>& client,
+                       int32_t                id,
+                       SampleCount            timestamp,
+                       const Raul::Path&      path,
+                       const Properties&      properties)
 	: Event(engine, client, id, timestamp)
 	, _path(path)
 	, _port_type(PortType::UNKNOWN)
diff --git a/src/server/events/CreatePort.hpp b/src/server/events/CreatePort.hpp
index 9c5bb396..8137328d 100644
--- a/src/server/events/CreatePort.hpp
+++ b/src/server/events/CreatePort.hpp
@@ -44,12 +44,12 @@ namespace events {
 class CreatePort : public Event
 {
 public:
-	CreatePort(Engine&           engine,
-	           SPtr<Interface>   client,
-	           int32_t           id,
-	           SampleCount       timestamp,
-	           const Raul::Path& path,
-	           const Properties& properties);
+	CreatePort(Engine&                engine,
+	           const SPtr<Interface>& client,
+	           int32_t                id,
+	           SampleCount            timestamp,
+	           const Raul::Path&      path,
+	           const Properties&      properties);
 
 	bool pre_process(PreProcessContext& ctx) override;
 	void execute(RunContext& context) override;
diff --git a/src/server/events/Delete.cpp b/src/server/events/Delete.cpp
index 0b46f3f4..5d605bca 100644
--- a/src/server/events/Delete.cpp
+++ b/src/server/events/Delete.cpp
@@ -43,10 +43,10 @@ namespace ingen {
 namespace server {
 namespace events {
 
-Delete::Delete(Engine&           engine,
-               SPtr<Interface>   client,
-               FrameTime         timestamp,
-               const ingen::Del& msg)
+Delete::Delete(Engine&                engine,
+               const SPtr<Interface>& client,
+               FrameTime              timestamp,
+               const ingen::Del&      msg)
 	: Event(engine, client, msg.seq, timestamp)
 	, _msg(msg)
 	, _engine_port(nullptr)
diff --git a/src/server/events/Delete.hpp b/src/server/events/Delete.hpp
index 14acac83..6b3149e9 100644
--- a/src/server/events/Delete.hpp
+++ b/src/server/events/Delete.hpp
@@ -45,10 +45,10 @@ class DisconnectAll;
 class Delete : public Event
 {
 public:
-	Delete(Engine&           engine,
-	       SPtr<Interface>   client,
-	       FrameTime         timestamp,
-	       const ingen::Del& msg);
+	Delete(Engine&                engine,
+	       const SPtr<Interface>& client,
+	       FrameTime              timestamp,
+	       const ingen::Del&      msg);
 
 	~Delete();
 
diff --git a/src/server/events/Delta.cpp b/src/server/events/Delta.cpp
index f5928c12..19e5bb20 100644
--- a/src/server/events/Delta.cpp
+++ b/src/server/events/Delta.cpp
@@ -48,10 +48,10 @@ class PreProcessContext;
 
 namespace events {
 
-Delta::Delta(Engine&           engine,
-             SPtr<Interface>   client,
-             SampleCount       timestamp,
-             const ingen::Put& msg)
+Delta::Delta(Engine&                engine,
+             const SPtr<Interface>& client,
+             SampleCount            timestamp,
+             const ingen::Put&      msg)
 	: Event(engine, client, msg.seq, timestamp)
 	, _create_event(nullptr)
 	, _subject(msg.uri)
@@ -67,10 +67,10 @@ Delta::Delta(Engine&           engine,
 	init();
 }
 
-Delta::Delta(Engine&             engine,
-             SPtr<Interface>     client,
-             SampleCount         timestamp,
-             const ingen::Delta& msg)
+Delta::Delta(Engine&                engine,
+             const SPtr<Interface>& client,
+             SampleCount            timestamp,
+             const ingen::Delta&    msg)
 	: Event(engine, client, msg.seq, timestamp)
 	, _create_event(nullptr)
 	, _subject(msg.uri)
@@ -88,7 +88,7 @@ Delta::Delta(Engine&             engine,
 }
 
 Delta::Delta(Engine&                   engine,
-             SPtr<Interface>           client,
+             const SPtr<Interface>&    client,
              SampleCount               timestamp,
              const ingen::SetProperty& msg)
 	: Event(engine, client, msg.seq, timestamp)
diff --git a/src/server/events/Delta.hpp b/src/server/events/Delta.hpp
index 4328ad83..2aa2ff90 100644
--- a/src/server/events/Delta.hpp
+++ b/src/server/events/Delta.hpp
@@ -47,18 +47,18 @@ class SetPortValue;
 class Delta : public Event
 {
 public:
-	Delta(Engine&           engine,
-	      SPtr<Interface>   client,
-	      SampleCount       timestamp,
-	      const ingen::Put& msg);
+	Delta(Engine&                engine,
+	      const SPtr<Interface>& client,
+	      SampleCount            timestamp,
+	      const ingen::Put&      msg);
 
-	Delta(Engine&             engine,
-	      SPtr<Interface>     client,
-	      SampleCount         timestamp,
-	      const ingen::Delta& msg);
+	Delta(Engine&                engine,
+	      const SPtr<Interface>& client,
+	      SampleCount            timestamp,
+	      const ingen::Delta&    msg);
 
 	Delta(Engine&                   engine,
-	      SPtr<Interface>           client,
+	      const SPtr<Interface>&    client,
 	      SampleCount               timestamp,
 	      const ingen::SetProperty& msg);
 
diff --git a/src/server/events/Disconnect.cpp b/src/server/events/Disconnect.cpp
index 349fb6ae..531d70af 100644
--- a/src/server/events/Disconnect.cpp
+++ b/src/server/events/Disconnect.cpp
@@ -44,7 +44,7 @@ namespace server {
 namespace events {
 
 Disconnect::Disconnect(Engine&                  engine,
-                       SPtr<Interface>          client,
+                       const SPtr<Interface>&   client,
                        SampleCount              timestamp,
                        const ingen::Disconnect& msg)
 	: Event(engine, client, msg.seq, timestamp)
diff --git a/src/server/events/Disconnect.hpp b/src/server/events/Disconnect.hpp
index 15872ba5..ec638c73 100644
--- a/src/server/events/Disconnect.hpp
+++ b/src/server/events/Disconnect.hpp
@@ -38,7 +38,7 @@ class Disconnect : public Event
 {
 public:
 	Disconnect(Engine&                  engine,
-	           SPtr<Interface>          client,
+	           const SPtr<Interface>&   client,
 	           SampleCount              timestamp,
 	           const ingen::Disconnect& msg);
 
diff --git a/src/server/events/DisconnectAll.cpp b/src/server/events/DisconnectAll.cpp
index 9c949480..ada51302 100644
--- a/src/server/events/DisconnectAll.cpp
+++ b/src/server/events/DisconnectAll.cpp
@@ -41,7 +41,7 @@ namespace server {
 namespace events {
 
 DisconnectAll::DisconnectAll(Engine&                     engine,
-                             SPtr<Interface>             client,
+                             const SPtr<Interface>&      client,
                              SampleCount                 timestamp,
                              const ingen::DisconnectAll& msg)
 	: Event(engine, client, msg.seq, timestamp)
diff --git a/src/server/events/DisconnectAll.hpp b/src/server/events/DisconnectAll.hpp
index 3edfedd6..9ab908c1 100644
--- a/src/server/events/DisconnectAll.hpp
+++ b/src/server/events/DisconnectAll.hpp
@@ -44,7 +44,7 @@ class DisconnectAll : public Event
 {
 public:
 	DisconnectAll(Engine&                     engine,
-	              SPtr<Interface>             client,
+	              const SPtr<Interface>&      client,
 	              SampleCount                 timestamp,
 	              const ingen::DisconnectAll& msg);
 
diff --git a/src/server/events/Get.cpp b/src/server/events/Get.cpp
index 167f5f91..7bb6fd2f 100644
--- a/src/server/events/Get.cpp
+++ b/src/server/events/Get.cpp
@@ -37,10 +37,10 @@ namespace ingen {
 namespace server {
 namespace events {
 
-Get::Get(Engine&           engine,
-         SPtr<Interface>   client,
-         SampleCount       timestamp,
-         const ingen::Get& msg)
+Get::Get(Engine&                engine,
+         const SPtr<Interface>& client,
+         SampleCount            timestamp,
+         const ingen::Get&      msg)
 	: Event(engine, client, msg.seq, timestamp)
 	, _msg(msg)
 	, _object(nullptr)
diff --git a/src/server/events/Get.hpp b/src/server/events/Get.hpp
index b23c7294..e24c9998 100644
--- a/src/server/events/Get.hpp
+++ b/src/server/events/Get.hpp
@@ -39,10 +39,10 @@ namespace events {
 class Get : public Event
 {
 public:
-	Get(Engine&           engine,
-	    SPtr<Interface>   client,
-	    SampleCount       timestamp,
-	    const ingen::Get& msg);
+	Get(Engine&                engine,
+	    const SPtr<Interface>& client,
+	    SampleCount            timestamp,
+	    const ingen::Get&      msg);
 
 	bool pre_process(PreProcessContext& ctx) override;
 	void execute(RunContext& context) override {}
diff --git a/src/server/events/Mark.cpp b/src/server/events/Mark.cpp
index 4f9b1261..314805f5 100644
--- a/src/server/events/Mark.cpp
+++ b/src/server/events/Mark.cpp
@@ -27,7 +27,7 @@ namespace server {
 namespace events {
 
 Mark::Mark(Engine&                   engine,
-           SPtr<Interface>           client,
+           const SPtr<Interface>&    client,
            SampleCount               timestamp,
            const ingen::BundleBegin& msg)
 	: Event(engine, client, msg.seq, timestamp)
@@ -36,7 +36,7 @@ Mark::Mark(Engine&                   engine,
 {}
 
 Mark::Mark(Engine&                 engine,
-           SPtr<Interface>         client,
+           const SPtr<Interface>&  client,
            SampleCount             timestamp,
            const ingen::BundleEnd& msg)
 	: Event(engine, client, msg.seq, timestamp)
diff --git a/src/server/events/Mark.hpp b/src/server/events/Mark.hpp
index ad8cb66b..c32c2588 100644
--- a/src/server/events/Mark.hpp
+++ b/src/server/events/Mark.hpp
@@ -41,12 +41,12 @@ class Mark : public Event
 {
 public:
 	Mark(Engine&                   engine,
-	     SPtr<Interface>           client,
+	     const SPtr<Interface>&    client,
 	     SampleCount               timestamp,
 	     const ingen::BundleBegin& msg);
 
 	Mark(Engine&                 engine,
-	     SPtr<Interface>         client,
+	     const SPtr<Interface>&  client,
 	     SampleCount             timestamp,
 	     const ingen::BundleEnd& msg);
 
diff --git a/src/server/events/Move.cpp b/src/server/events/Move.cpp
index c7478898..857be189 100644
--- a/src/server/events/Move.cpp
+++ b/src/server/events/Move.cpp
@@ -31,10 +31,10 @@ namespace ingen {
 namespace server {
 namespace events {
 
-Move::Move(Engine&           engine,
-           SPtr<Interface>   client,
-           SampleCount       timestamp,
-           const ingen::Move& msg)
+Move::Move(Engine&                engine,
+           const SPtr<Interface>& client,
+           SampleCount            timestamp,
+           const ingen::Move&     msg)
 	: Event(engine, client, msg.seq, timestamp)
 	, _msg(msg)
 {
diff --git a/src/server/events/Move.hpp b/src/server/events/Move.hpp
index 327b31b9..b4487b78 100644
--- a/src/server/events/Move.hpp
+++ b/src/server/events/Move.hpp
@@ -36,10 +36,10 @@ namespace events {
 class Move : public Event
 {
 public:
-	Move(Engine&            engine,
-	     SPtr<Interface>    client,
-	     SampleCount        timestamp,
-	     const ingen::Move& msg);
+	Move(Engine&                engine,
+	     const SPtr<Interface>& client,
+	     SampleCount            timestamp,
+	     const ingen::Move&     msg);
 
 	bool pre_process(PreProcessContext& ctx) override;
 	void execute(RunContext& context) override;
diff --git a/src/server/events/SetPortValue.cpp b/src/server/events/SetPortValue.cpp
index 9193f876..02f3f66b 100644
--- a/src/server/events/SetPortValue.cpp
+++ b/src/server/events/SetPortValue.cpp
@@ -37,14 +37,14 @@ namespace server {
 namespace events {
 
 /** Internal */
-SetPortValue::SetPortValue(Engine&         engine,
-                           SPtr<Interface> client,
-                           int32_t         id,
-                           SampleCount     timestamp,
-                           PortImpl*       port,
-                           const Atom&     value,
-                           bool            activity,
-                           bool            synthetic)
+SetPortValue::SetPortValue(Engine&                engine,
+                           const SPtr<Interface>& client,
+                           int32_t                id,
+                           SampleCount            timestamp,
+                           PortImpl*              port,
+                           const Atom&            value,
+                           bool                   activity,
+                           bool                   synthetic)
 	: Event(engine, client, id, timestamp)
 	, _port(port)
 	, _value(value)
diff --git a/src/server/events/SetPortValue.hpp b/src/server/events/SetPortValue.hpp
index 1e7c7250..99d55af3 100644
--- a/src/server/events/SetPortValue.hpp
+++ b/src/server/events/SetPortValue.hpp
@@ -40,14 +40,14 @@ namespace events {
 class SetPortValue : public Event
 {
 public:
-	SetPortValue(Engine&         engine,
-	             SPtr<Interface> client,
-	             int32_t         id,
-	             SampleCount     timestamp,
-	             PortImpl*       port,
-	             const Atom&     value,
-	             bool            activity,
-	             bool            synthetic = false);
+	SetPortValue(Engine&                engine,
+	             const SPtr<Interface>& client,
+	             int32_t                id,
+	             SampleCount            timestamp,
+	             PortImpl*              port,
+	             const Atom&            value,
+	             bool                   activity,
+	             bool                   synthetic = false);
 
 	bool pre_process(PreProcessContext& ctx) override;
 	void execute(RunContext& context) override;
diff --git a/src/server/events/Undo.cpp b/src/server/events/Undo.cpp
index 3a3bdbe2..0741e60d 100644
--- a/src/server/events/Undo.cpp
+++ b/src/server/events/Undo.cpp
@@ -27,24 +27,24 @@ namespace ingen {
 namespace server {
 namespace events {
 
-Undo::Undo(Engine&            engine,
-           SPtr<Interface>    client,
-           SampleCount        timestamp,
-           const ingen::Undo& msg)
+Undo::Undo(Engine&                engine,
+           const SPtr<Interface>& client,
+           SampleCount            timestamp,
+           const ingen::Undo&     msg)
 	: Event(engine, client, msg.seq, timestamp)
 	, _is_redo(false)
 {}
 
-Undo::Undo(Engine&            engine,
-           SPtr<Interface>    client,
-           SampleCount        timestamp,
-           const ingen::Redo& msg)
+Undo::Undo(Engine&                engine,
+           const SPtr<Interface>& client,
+           SampleCount            timestamp,
+           const ingen::Redo&     msg)
 	: Event(engine, client, msg.seq, timestamp)
 	, _is_redo(true)
 {}
 
 bool
-Undo::pre_process(PreProcessContext& ctx)
+Undo::pre_process(PreProcessContext&)
 {
 	const UPtr<UndoStack>& stack = _is_redo ? _engine.redo_stack() : _engine.undo_stack();
 	const Event::Mode      mode  = _is_redo ? Event::Mode::REDO    : Event::Mode::UNDO;
diff --git a/src/server/events/Undo.hpp b/src/server/events/Undo.hpp
index f8469960..e36ebaad 100644
--- a/src/server/events/Undo.hpp
+++ b/src/server/events/Undo.hpp
@@ -32,15 +32,15 @@ namespace events {
 class Undo : public Event
 {
 public:
-	Undo(Engine&            engine,
-	     SPtr<Interface>    client,
-	     SampleCount        timestamp,
-	     const ingen::Undo& msg);
-
-	Undo(Engine&            engine,
-	     SPtr<Interface>    client,
-	     SampleCount        timestamp,
-	     const ingen::Redo& msg);
+	Undo(Engine&                engine,
+	     const SPtr<Interface>& client,
+	     SampleCount            timestamp,
+	     const ingen::Undo&     msg);
+
+	Undo(Engine&                engine,
+	     const SPtr<Interface>& client,
+	     SampleCount            timestamp,
+	     const ingen::Redo&     msg);
 
 	bool pre_process(PreProcessContext& ctx) override;
 	void execute(RunContext& context) override;