Performance and resource consumption optimization for G2D picking

[simantics/platform.git] / bundles / org.simantics.g2d / src / org / simantics / g2d / diagram / DiagramUtils.java

diff --git a/bundles/org.simantics.g2d/src/org/simantics/g2d/diagram/DiagramUtils.java b/bundles/org.simantics.g2d/src/org/simantics/g2d/diagram/DiagramUtils.java
index ed812c21a556aa7517a38bd5778e474decbb9bdf..7f83cf9bc58ee0131ed79c3e04e93963a9c966f9 100644 (file)
--- a/bundles/org.simantics.g2d/src/org/simantics/g2d/diagram/DiagramUtils.java
+++ b/bundles/org.simantics.g2d/src/org/simantics/g2d/diagram/DiagramUtils.java
@@ -58,6 +58,8 @@ import org.simantics.g2d.routing.IConnection;
 import org.simantics.g2d.routing.IRouter2;
 import org.simantics.g2d.routing.TrivialRouter2;
 import org.simantics.scenegraph.utils.GeometryUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import gnu.trove.map.hash.THashMap;
 
@@ -68,6 +70,7 @@ import gnu.trove.map.hash.THashMap;
  */
 public class DiagramUtils {
 
+    private static final Logger LOGGER = LoggerFactory.getLogger(DiagramUtils.class);
     /**
      * Get rectangle that contains all elements or null if there are no elements.
      * @param d
@@ -157,7 +160,7 @@ public class DiagramUtils {
 
         for (final IElement element : elementsToFix) {
             if (!d.containsElement(element)) {
-                System.err.println("Fixing element not contained by diagram " + d + ": " + element);
+                LOGGER.warn("Fixing element not contained by diagram " + d + ": " + element);
                 continue;
             }
 
@@ -329,7 +332,7 @@ public class DiagramUtils {
     }
 
     /**
-     * Execute the specified {@link Callback} within a diagram write transaction
+     * Execute the specified <code>callback</code> within a diagram write transaction
      * using the {@link TransactionContext} handler available in the
      * {@link DiagramClass} of the specified {@link Diagram}. The diagram must
      * contain a valid value for the {@link DiagramHints#KEY_MUTATOR} hint which