tags 650808 +patch
thanks

Thanks to Joachim Breitner for taking some time to look at this with me.
A likely cause was some mistake in the n+k removal patch. Indeed I found
the precise culprit and verified that with the attached fix frown is
able to translate grammers again.

Helmut
diff -Nru frown-0.6.1/debian/changelog frown-0.6.1/debian/changelog
--- frown-0.6.1/debian/changelog        2011-06-02 20:33:57.000000000 +0200
+++ frown-0.6.1/debian/changelog        2011-12-07 17:36:24.000000000 +0100
@@ -1,3 +1,11 @@
+frown (0.6.1-11.1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Fix "OOM on compiling any remotely valid grammer" fix logic error in
+    debian/patches/07_no-n-plus-k-pattern (Closes: #650808)
+
+ -- Helmut Grohne <hel...@subdivi.de>  Wed, 07 Dec 2011 17:35:30 +0100
+
 frown (0.6.1-11) unstable; urgency=low
 
   [ Marco Silva ]
diff -Nru frown-0.6.1/debian/patches/07_no-n-plus-k-pattern 
frown-0.6.1/debian/patches/07_no-n-plus-k-pattern
--- frown-0.6.1/debian/patches/07_no-n-plus-k-pattern   2011-06-02 
20:32:03.000000000 +0200
+++ frown-0.6.1/debian/patches/07_no-n-plus-k-pattern   2011-12-07 
17:34:55.000000000 +0100
@@ -9,7 +9,7 @@
 ->   build (n + 1) x             =  (Node l a v r, z)
 ->     where m                   =  n `div` 2
 +>   build n x                   =  (Node l a v r, z)
-+>     where m                   =  n-1 `div` 2
++>     where m                   =  (n-1) `div` 2
  >           (l, (a, v) : y)     =  build m       x
 ->           (r, z)              =  build (n - m) y
 +>           (r, z)              =  build (n - 1 - m) y

Reply via email to