decrement N in FreqDist.pop and FreqDist.popitem, remove arg from Fre…

…qDist.popitem; resolves issue 726
nltk · Dec 1, 2011 · 55c4143 · 55c4143
1 parent 5edf189
commit 55c4143
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/nltk/probability.py b/nltk/probability.py
@@ -467,12 +467,14 @@ def update(self, samples):
             self.inc(sample, count=count)    
 
     def pop(self, other):
+        self._N -= 1
         self._reset_caches()
         return dict.pop(self, other)
 
-    def popitem(self, other):
+    def popitem(self):
+        self._N -= 1
         self._reset_caches()
-        return dict.popitem(self, other)
+        return dict.popitem(self)
 
     def clear(self):
         self._N = 0