pqueue 2.0.1 → 2.0.2

data/.ruby

@@ -44,7 +44,7 @@ revision: 0
 created: '2001-03-10'
 summary: Queue of Prioritized Elements
 title: PQueue
-version: 2.0.1
+version: 2.0.2
 name: pqueue
 description: ! 'A priority queue is like a standard queue, except that each inserted
   elements

data/HISTORY.rdoc

@@ -1,5 +1,21 @@
 = CHANGE HISTORY
 
+== 2.0.2 / 2011-10-29
+
+It's been one of those days. I went to to get a wash cloth for the shower,
+on my way through the kitchen realized the chilling cookie dough needed to
+be put in the oven, but I forgot to put flower in the batter, so they burnt
+up in minutes and made a mess that took a half-hour to clean-up. Then I discover
+the shower still running, hot water was all but gone, and my bedroom felt like
+a suana. To top it all off, it was at this moment I realized PQueue's binary
+search algrithm didn't work. Sigh. And fuck if my grandmother won't stop making
+me food I don't want to eat. Yea, one of those days!
+
+Changes:
+
+* Fixed #reheap search algorithm.
+
+
 == 2.0.1 / 2011-10-29
 
 Quick fix to remove old legacy library that was supposed to be 

data/lib/pqueue.rb

@@ -261,31 +261,9 @@ class PQueue
     que = @que.dup
 
     v = que.delete_at(k)
+    i = binary_index(que, v)
 
-    i = que.size.div(2)
-    q = i
-    r = nil
-
-    loop do
-      case @cmp.call(v, que[i])
-      when 0, nil
-        r = i
-        break
-      when 1, true
-        i = (que.size + i).div(2)
-        i += 1 if i == q  # don't repeat yourself
-      when -1, false
-        i = (i).div(2)
-        i -= 1 if i == q  # don't repeat yourself
-      else
-        warn "bad comparison procedure in #{self.inspect}"
-        r = i
-        break
-      end
-      q = i
-    end
-
-    que.insert(r, v)
+    que.insert(i, v)
 
     @que = que
 
@@ -314,4 +292,26 @@ class PQueue
   #
   alias heapify sort!
 
+  #
+  def binary_index(que, target)
+    upper = que.size - 1
+    lower = 0
+
+    while(upper >= lower) do
+      idx  = lower + (upper - lower) / 2
+      comp = @cmp.call(target, que[idx])
+
+      case comp
+      when 0, nil
+        return idx
+      when 1, true
+        lower = idx + 1
+      when -1, false
+        upper = idx - 1
+      else
+      end
+    end
+    lower
+  end
+
 end # class PQueue

data/test/test_pqueue.rb

@@ -1,9 +1,10 @@
 require 'microtest'
+require 'ae'
 require 'ae/legacy'
 
 require 'pqueue'
 
-class TC_PQueue < MicroTest::TestCase
+class PQueueTest < MicroTest::TestCase
   include AE::Legacy::Assertions
 
   ARY_TEST   = [2,6,1,3,8,15,0,-4,7,8,10]
@@ -152,5 +153,26 @@ class TC_PQueue < MicroTest::TestCase
     q.pop
     assert_not_equal(q, r)
   end
-end
 
+  def test_reheap
+    q = PQueue.new([2,4,5])
+    q << 6
+    q.to_a.assert == [2,4,5,6]
+    q << 1
+    q.to_a.assert == [1,2,4,5,6]
+    q << 3
+    q.to_a.assert == [1,2,3,4,5,6]
+
+    q = PQueue.new([100,5,25])
+    q.to_a.assert == [5,25,100]
+    q << 17
+    q.to_a.assert == [5,17,25,100]
+    q << 0
+    q.to_a.assert == [0,5,17,25,100]
+    q << -5
+    q.to_a.assert == [-5,0,5,17,25,100]
+    q << 100
+    q.to_a.assert == [-5,0,5,17,25,100,100]
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pqueue
 version: !ruby/object:Gem::Version
-  version: 2.0.1
+  version: 2.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -17,7 +17,7 @@ date: 2011-10-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: detroit
-  requirement: &22996340 !ruby/object:Gem::Requirement
+  requirement: &18261020 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -25,10 +25,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *22996340
+  version_requirements: *18261020
 - !ruby/object:Gem::Dependency
   name: microtest
-  requirement: &22995600 !ruby/object:Gem::Requirement
+  requirement: &18260240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -36,10 +36,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *22995600
+  version_requirements: *18260240
 - !ruby/object:Gem::Dependency
   name: ae
-  requirement: &22994880 !ruby/object:Gem::Requirement
+  requirement: &18259560 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -47,7 +47,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *22994880
+  version_requirements: *18259560
 description: ! 'A priority queue is like a standard queue, except that each inserted
   elements