about summary refs log tree commit homepage
path: root/lib/dtas/player
diff options
authorEric Wong <normalperson@yhbt.net>2013-08-25 22:40:08 +0000
committerEric Wong <normalperson@yhbt.net>2013-08-25 23:13:56 +0000
commit6ad75d32c2389d3bfb358c60b77715db71c1f413 (patch)
tree34ff8b62e2095e95e12717fb6816d33d648db8e5 /lib/dtas/player
parentdfbefc7cb72b2c60e2cb076b33a5964e1be21f33 (diff)
We do not need a respawn flag, since we already infer expected vs
unexpected sink death by checking the @targets array.

Additionally, next_source must always check @current before
calling, and not clobber the existing @current because that
would cause two source processes writing into the same pipe.
Diffstat (limited to 'lib/dtas/player')
1 files changed, 0 insertions, 2 deletions
diff --git a/lib/dtas/player/client_handler.rb b/lib/dtas/player/client_handler.rb
index 44f0b57..e08d9a3 100644
--- a/lib/dtas/player/client_handler.rb
+++ b/lib/dtas/player/client_handler.rb
@@ -91,7 +91,6 @@ module DTAS::Player::ClientHandler # :nodoc:
         # do not reactivate it until we've reaped it
         if sink.pid
-          sink.respawn = true
@@ -375,7 +374,6 @@ module DTAS::Player::ClientHandler # :nodoc:
   def restart_pipeline
     return if @paused
-    @sinks.each_value { |sink| sink.respawn = sink.active }
     @targets.each { |t| drop_target(t) }.clear