From ddb1f774839b7935968fe86a937dd305a9233d89 Mon Sep 17 00:00:00 2001
From: Eric Wong <normalperson@yhbt.net>
Date: Mon, 9 Sep 2013 08:05:18 +0000
Subject: player: "tl goto" takes optional offset arg in HHMMSS.SUBSEC

This should make implementing SetPosition in the MPRIS 2.0 spec
possible.
---
 lib/dtas/player.rb                |  4 ++--
 lib/dtas/player/client_handler.rb |  3 ++-
 lib/dtas/tracklist.rb             | 12 +++++++-----
 3 files changed, 11 insertions(+), 8 deletions(-)

(limited to 'lib')

diff --git a/lib/dtas/player.rb b/lib/dtas/player.rb
index 5e78d8e..9ab45fc 100644
--- a/lib/dtas/player.rb
+++ b/lib/dtas/player.rb
@@ -353,9 +353,9 @@ class DTAS::Player # :nodoc:
 
     # don't get stuck in an infinite loop if @tl.repeat==true and we can't
     # decode anything (FS errors, sox uninstalled, etc...)
-    while path = @tl.advance_track(false)
+    while path_off = @tl.advance_track(false)
       @sources.each do |src|
-        rv = src.try(path) and return rv
+        rv = src.try(*path_off) and return rv
       end
     end
 
diff --git a/lib/dtas/player/client_handler.rb b/lib/dtas/player/client_handler.rb
index 33cae05..c7dd31d 100644
--- a/lib/dtas/player/client_handler.rb
+++ b/lib/dtas/player/client_handler.rb
@@ -571,7 +571,8 @@ module DTAS::Player::ClientHandler # :nodoc:
       io.emit(@tl.tracks.map! { |i| i.to_s }.join(' '))
     when "goto"
       track_id = msg.shift or return io.emit("ERR track_id not specified")
-      if @tl.go_to(track_id.to_i)
+      offset = msg.shift # may be nil
+      if @tl.go_to(track_id.to_i, offset)
         _tl_skip
         io.emit("OK")
       else
diff --git a/lib/dtas/tracklist.rb b/lib/dtas/tracklist.rb
index c3cd3ff..094c22d 100644
--- a/lib/dtas/tracklist.rb
+++ b/lib/dtas/tracklist.rb
@@ -32,11 +32,11 @@ class DTAS::Tracklist
   def initialize
     TL_DEFAULTS.each { |k,v| instance_variable_set("@#{k}", v) }
     @list = []
-    @goto_pos = nil
+    @goto_off = @goto_pos = nil
   end
 
   def reset
-    @goto_pos = nil
+    @goto_off = @goto_pos = nil
     @pos = TL_DEFAULTS["pos"]
   end
 
@@ -68,7 +68,8 @@ class DTAS::Tracklist
   def advance_track(repeat_ok = true)
     return if @list.empty?
     next_pos = @goto_pos || @pos + 1
-    @goto_pos = nil
+    next_off = @goto_off # nil by default
+    @goto_pos = @goto_off = nil
     if @list[next_pos]
       @pos = next_pos
     elsif @repeat && repeat_ok
@@ -76,7 +77,7 @@ class DTAS::Tracklist
     else
       return
     end
-    @list[next_pos]
+    [ @list[next_pos], next_off ]
   end
 
   def cur_track
@@ -106,9 +107,10 @@ class DTAS::Tracklist
     end
   end
 
-  def go_to(track_id)
+  def go_to(track_id, offset_hhmmss = nil)
     by_track_id = _track_id_map
     if idx = by_track_id[track_id]
+      @goto_off = offset_hhmmss
       return @list[@goto_pos = idx]
     end
     @goto_pos = nil
-- 
cgit v1.2.3-24-ge0c7