X-Git-Url: https://gerrit.simantics.org/r/gitweb?a=blobdiff_plain;f=org.simantics.g3d.vtk%2Fsrc%2Forg%2Fsimantics%2Fg3d%2Fvtk%2Fcommon%2FAbstractVTKNodeMap.java;h=bc7c35528fde4a33f07cc0c5755ecf27e781f1de;hb=e80f56a645f98570ead038096a8963c0c2af4b05;hp=ba62ddbfdb64ca14745420ec7d17b145a1e0ee06;hpb=156f859a01694337da2013f263bed0d9e60df8c9;p=simantics%2F3d.git

diff --git a/org.simantics.g3d.vtk/src/org/simantics/g3d/vtk/common/AbstractVTKNodeMap.java b/org.simantics.g3d.vtk/src/org/simantics/g3d/vtk/common/AbstractVTKNodeMap.java
index ba62ddbf..bc7c3552 100644
--- a/org.simantics.g3d.vtk/src/org/simantics/g3d/vtk/common/AbstractVTKNodeMap.java
+++ b/org.simantics.g3d.vtk/src/org/simantics/g3d/vtk/common/AbstractVTKNodeMap.java
@@ -372,12 +372,14 @@ public abstract class AbstractVTKNodeMap<DBObject,E extends INode> implements VT
 	private boolean useFullSyncWithUndo = false;
 	
 	protected void update(ReadGraph graph) throws DatabaseException {
-		synchronized (syncMutex) {
-			if (DEBUG) System.out.println("Graph update start");
-			
-			if (runUndo && useFullSyncWithUndo) {
+		if (DEBUG) System.out.println("Graph update start");
+		
+		if (runUndo && useFullSyncWithUndo) {
+			synchronized (syncMutex) {
 				reset(graph);
-			} else {
+			}
+		} else {
+			synchronized (syncMutex) {
 				graphUpdates = true;
 				for (DBObject domainObject : mapping.getDomainModified()) {
 					@SuppressWarnings("unchecked")
@@ -385,18 +387,24 @@ public abstract class AbstractVTKNodeMap<DBObject,E extends INode> implements VT
 					if (rangeObject != null)
 						graphModified.add(rangeObject);
 				}
-				mapping.updateRange(graph);
+			
+			}
+			
+			mapping.updateRange(graph);
+			
+			synchronized (syncMutex) {
 				graphModified.clear();
 				syncDeletes();
-				clearDeletes();
-				graphUpdates = false;
-    		}
-			
-			if (mapping.isRangeModified() && !runUndo && !runRedo)
-				commit((String)null);
+			}
 			
-			if (DEBUG) System.out.println("Graph update done");
+			clearDeletes();
+			graphUpdates = false;
 		}
+		
+		if (mapping.isRangeModified() && !runUndo && !runRedo)
+			commit((String)null);
+		
+		if (DEBUG) System.out.println("Graph update done");
 	}
 	
 	@Override
@@ -438,7 +446,6 @@ public abstract class AbstractVTKNodeMap<DBObject,E extends INode> implements VT
 	 * This code here synchronizes removed and added objects to collect deletable objects. (a deletable object is one which is removed but not added).  
 	 * 
 	 */
-	@SuppressWarnings("unused")
 	protected void syncDeletes() {
 		deleteUC.clear();
 		for (Pair<E, String> n : removed) {
@@ -460,7 +467,6 @@ public abstract class AbstractVTKNodeMap<DBObject,E extends INode> implements VT
 	/**
 	 * Clears deletable objects from mapping cache.
 	 */
-	@SuppressWarnings("unused")
 	protected void clearDeletes() {
 		if (DEBUG && delete.size() > 0) System.out.println("Delete");
 		for (E n : delete) {