Scissor::Chunk#loop is not destructive

youpy · Dec 2, 2011 · 9c86ad8 · 9c86ad8
1 parent 1a5d84c
commit 9c86ad8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/lib/scissor/chunk.rb b/lib/scissor/chunk.rb
@@ -108,12 +108,14 @@ def +(other)
 
     def loop(count)
       orig_fragments = @fragments.clone
+      new_instance = Scissor()
+      new_instance.add_fragments(orig_fragments)
 
       (count - 1).times do
-        add_fragments(orig_fragments)
+        new_instance.add_fragments(orig_fragments)
       end
 
-      self
+      new_instance
     end
 
     alias * loop

diff --git a/spec/scissor_spec.rb b/spec/scissor_spec.rb
@@ -88,8 +88,9 @@
   end
 
   it "should loop" do
-    scissor = @mp3.slice(0, 10).loop(3)
+    scissor = @mp3.slice(0, 10)
-    scissor.duration.should eql(30.0)
+    scissor.loop(3).duration.should eql(30.0)
+    scissor.duration.should eql(10.0)
   end
 
   it "should loop using arithmetic operator" do