# HG changeset patch
# User wenzelm
# Date 1349789505 -7200
# Node ID 5073cb632b6c9a80df531c28a75742631f0485fe
# Parent  083accbfa77d52ab3b9ac2b5418f1fc419038159
minor tuning;

diff -r 083accbfa77d -r 5073cb632b6c src/Tools/Graphview/src/layout_pendulum.scala
--- a/src/Tools/Graphview/src/layout_pendulum.scala	Tue Oct 09 13:57:51 2012 +0200
+++ b/src/Tools/Graphview/src/layout_pendulum.scala	Tue Oct 09 15:31:45 2012 +0200
@@ -103,27 +103,27 @@
     buckets.iterator.map(_.sorted).toList
   }
 
-  def count_crossings(graph: Model.Graph, levels: Levels): Int = {
+  def count_crossings(graph: Model.Graph, levels: Levels): Int =
+  {
     def in_level(ls: Levels): Int = ls match {
       case List(top, bot) =>
-        top.zipWithIndex.map{
-          case (outer_parent, outer_parent_index) => 
-            graph.imm_succs(outer_parent).map(bot.indexOf(_))  // FIXME iterator
-            .map(outer_child => {
+        top.iterator.zipWithIndex.map {
+          case (outer_parent, outer_parent_index) =>
+            graph.imm_succs(outer_parent).iterator.map(bot.indexOf(_))
+            .map(outer_child =>
               (0 until outer_parent_index)
-              .map(inner_parent => 
-                graph.imm_succs(top(inner_parent))
-                .map(bot.indexOf(_))
+              .map(inner_parent =>
+                graph.imm_succs(top(inner_parent)).iterator.map(bot.indexOf(_))
                 .filter(inner_child => outer_child < inner_child)
                 .size
               ).sum
-            }).sum
+            ).sum
         }.sum
-      
+
       case _ => 0
     }
-    
-    levels.sliding(2).map(in_level).sum
+
+    levels.iterator.sliding(2).map(ls => in_level(ls.toList)).sum
   }
   
   def minimize_crossings(graph: Model.Graph, levels: Levels): Levels =
@@ -247,7 +247,7 @@
     val regions = levels.map(level => level.map(new Region(graph, _)))
     
     ((regions, coords, true, true) /: (1 to pendulum_iterations)) {
-      case ((regions, coords, top_down, moved), i) =>
+      case ((regions, coords, top_down, moved), _) =>
         if (moved) {
           val (nextr, nextc, m) = iteration(regions, coords, top_down)
           (nextr, nextc, !top_down, m)