From 73161988bb80813ae68e4ec9ad676d54169bcb4c Mon Sep 17 00:00:00 2001
From: Paul Davis <paul@linuxaudiosystems.com>
Date: Wed, 9 May 2012 16:25:50 +0000
Subject: add some explanatory comments to pbd/signals.h

git-svn-id: svn://localhost/ardour2/branches/3.0@12226 d708f5d6-7413-0410-9779-e7cbd77b26cf
---
 libs/pbd/pbd/signals.h | 57 +++++++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 56 insertions(+), 1 deletion(-)

(limited to 'libs/pbd/pbd/signals.h')

diff --git a/libs/pbd/pbd/signals.h b/libs/pbd/pbd/signals.h
index 1acc6022b0..7e51ff2a27 100644
--- a/libs/pbd/pbd/signals.h
+++ b/libs/pbd/pbd/signals.h
@@ -76,16 +76,57 @@ public:
     Signal0 () {}
     typedef boost::signals2::signal<R()> SignalType;
 
+    /** Arrange for @a slot to be executed in the context of @a event_loop
+	whenever this signal is emitted. Store the connection that represents
+	this arrangement to @a c.
+
+	NOTE: @a slot will be executed in the same thread that the signal is
+	emitted in.
+    */
+
     void connect_same_thread (ScopedConnection& c, 
 		  const typename SignalType::slot_function_type& slot) {
 	    c = _signal.connect (slot);
     }
 
+    /** Arrange for @a slot to be executed in the context of @a event_loop
+	whenever this signal is emitted. Add the connection that represents
+	this arrangement to @a clist.
+
+	NOTE: @a slot will be executed in the same thread that the signal is
+	emitted in.
+    */
+	
     void connect_same_thread (ScopedConnectionList& clist, 
 		  const typename SignalType::slot_function_type& slot) {
 	    clist.add_connection (_signal.connect (slot));
     }
 
+    /** Arrange for @a slot to be executed in the context of @a event_loop
+	whenever this signal is emitted. Add the connection that represents
+	this arrangement to @a clist.
+	
+	If the event loop/thread in which @a slot will be executed will
+	outlive the lifetime of any object referenced in @a slot,
+	then an InvalidationRecord should be passed, allowing
+	any request sent to the @a event_loop and not executed
+	before the object is destroyed to be marked invalid.
+	
+	"outliving the lifetime" doesn't have a specific, detailed meaning,
+	but is best illustrated by two contrasting examples:
+	
+	1) the main GUI event loop/thread - this will outlive more or 
+	less all objects in the application, and thus when arranging for
+	@a slot to be called in that context, an invalidation record is 
+	highly advisable.
+	
+	2) a secondary event loop/thread which will be destroyed along
+	with the objects that are typically referenced by @a slot.
+	Assuming that the event loop is stopped before the objects are
+	destroyed, there is no reason to pass in an invalidation record,
+	and MISSING_INVALIDATOR may be used.
+    */
+
     void connect (ScopedConnectionList& clist, 
                   PBD::EventLoop::InvalidationRecord* ir, 
 		  const typename SignalType::slot_function_type& slot,
@@ -95,7 +136,12 @@ public:
 	    }
 	    clist.add_connection (_signal.connect (boost::bind (&EventLoop::call_slot, event_loop, ir, slot)));
     }
-    
+
+    /** See notes for the ScopedConnectionList variant of this function. This
+     * differs in that it stores the connection to the signal in a single
+     * ScopedConnection rather than a ScopedConnectionList.
+     */
+
     void connect (ScopedConnection& c, 
                   PBD::EventLoop::InvalidationRecord* ir, 
 		  const typename SignalType::slot_function_type& slot,
@@ -105,11 +151,20 @@ public:
 	    }
 	    c = _signal.connect (boost::bind (&EventLoop::call_slot, event_loop, ir, slot));
     }
+
+    /** Emit this signal. This will cause all slots connected to it be executed
+	in the order that they were connected (cross-thread issues may alter
+	the precise execution time of cross-thread slots).
+    */
     
     typename SignalType::result_type operator()() {
 	    return _signal ();
     }
 
+    /** Return true if there is nothing connected to this signal, false
+     *  otherwise.
+     */
+
     bool empty() const { return _signal.empty(); }
     
 private:
-- 
cgit v1.2.3