From 1c77d0d7c414d43f751eeb308a1a47d87de6cc1e Mon Sep 17 00:00:00 2001
From: Eric Wong <normalperson@yhbt.net>
Date: Tue, 15 Oct 2013 07:36:37 +0000
Subject: trimfx: flesh out to_sox_arg implementation

This will be dependent upon sox for the trim effect,
at least.  Other bits will be exposed as needed.
---
 lib/dtas/trimfx.rb | 22 ++++++++++++++++------
 1 file changed, 16 insertions(+), 6 deletions(-)

(limited to 'lib')

diff --git a/lib/dtas/trimfx.rb b/lib/dtas/trimfx.rb
index 5cfac26..cbb2fa6 100644
--- a/lib/dtas/trimfx.rb
+++ b/lib/dtas/trimfx.rb
@@ -22,7 +22,13 @@ class DTAS::TrimFX
 
   def to_sox_arg(format)
     if @tbeg && @tlen
-      %W(trim #{@tbeg * format.rate}s #{@tlen * format.rate}s)
+      beg = @tbeg * format.rate
+      len = @tlen * format.rate
+      %W(trim #{beg.round}s #{len.round}s)
+    elsif @tbeg
+      return [] if @tbeg == 0
+      beg = @tbeg * format.rate
+      %W(trim #{beg.round}s)
     else
       []
     end
@@ -55,11 +61,15 @@ class DTAS::TrimFX
 
   def parse_trim!(args)
     tbeg = parse_time(args.shift)
-    tlen = args.shift
-    is_stop_time = tlen.sub!(/\A=/, "") ? true : false
-    tlen = parse_time(tlen)
-    if is_stop_time
-      tlen = tlen - tbeg
+    if args[0] =~ /\A=?[\d\.]+\z/
+      tlen = args.shift
+      is_stop_time = tlen.sub!(/\A=/, "") ? true : false
+      tlen = parse_time(tlen)
+      if is_stop_time
+        tlen = tlen - tbeg
+      end
+    else
+      tlen = nil
     end
     @tbeg = tbeg
     @tlen = tlen
-- 
cgit v1.2.3-24-ge0c7