X-Git-Url: https://mograsim.net/gitweb/?a=blobdiff_plain;f=net.mograsim.logic.model%2Fsrc%2Fnet%2Fmograsim%2Flogic%2Fmodel%2Fmodel%2Fcomponents%2FGUIComponent.java;h=60e8fb2a782e2dcd2d0f2c01f69094cc932e83fd;hb=0a0877d1d0fca51260e66d65da3636752ded4242;hp=85b1e5a081b7d2598d722cb89b1790c258b90047;hpb=a47c9b1f38bbb6d2f5a3c482f3b09d1f42270401;p=Mograsim.git

diff --git a/net.mograsim.logic.model/src/net/mograsim/logic/model/model/components/GUIComponent.java b/net.mograsim.logic.model/src/net/mograsim/logic/model/model/components/GUIComponent.java
index 85b1e5a0..60e8fb2a 100644
--- a/net.mograsim.logic.model/src/net/mograsim/logic/model/model/components/GUIComponent.java
+++ b/net.mograsim.logic.model/src/net/mograsim/logic/model/model/components/GUIComponent.java
@@ -50,9 +50,6 @@ public abstract class GUIComponent implements JSONSerializable
 	private final List<Consumer<? super GUIComponent>> componentResizedListeners;
 	private final List<Consumer<? super Pin>> pinAddedListeners;
 	private final List<Consumer<? super Pin>> pinRemovedListeners;
-	private final List<Runnable> redrawListeners;
-
-	private final Runnable redrawListenerForSubcomponents;
 
 	private HighLevelStateHandler highLevelStateHandler;
 
@@ -70,10 +67,9 @@ public abstract class GUIComponent implements JSONSerializable
 		this.componentResizedListeners = new ArrayList<>();
 		this.pinAddedListeners = new ArrayList<>();
 		this.pinRemovedListeners = new ArrayList<>();
-		this.redrawListeners = new ArrayList<>();
-
-		redrawListenerForSubcomponents = this::requestRedraw;
 
+		// TODO this will crash the high level state debug shell because submodel is not yet set.
+		// The same problem exists in ViewModelModifiable.getDefaultComponentName; see there
 		model.componentCreated(this);
 	}
 
@@ -107,8 +103,7 @@ public abstract class GUIComponent implements JSONSerializable
 			throw new IllegalArgumentException("Duplicate pin name: " + pin.name);
 		pinsByName.put(pin.name, pin);
 		callPinAddedListeners(pin);
-		pin.addRedrawListener(redrawListenerForSubcomponents);
-		requestRedraw();
+		model.requestRedraw();
 	}
 
 	/**
@@ -122,8 +117,7 @@ public abstract class GUIComponent implements JSONSerializable
 	{
 		Pin pin = pinsByName.remove(name);
 		callPinRemovedListeners(pin);
-		pin.removeRedrawListener(redrawListenerForSubcomponents);
-		requestRedraw();
+		model.requestRedraw();
 	}
 
 	/**
@@ -206,7 +200,7 @@ public abstract class GUIComponent implements JSONSerializable
 		bounds.x = x;
 		bounds.y = y;
 		callComponentMovedListeners();
-		requestRedraw();
+		model.requestRedraw();
 	}
 
 	/**
@@ -219,7 +213,7 @@ public abstract class GUIComponent implements JSONSerializable
 		bounds.width = width;
 		bounds.height = height;
 		callComponentResizedListener();
-		requestRedraw();
+		model.requestRedraw();
 	}
 
 	/**
@@ -302,33 +296,20 @@ public abstract class GUIComponent implements JSONSerializable
 
 	// listeners
 
-	/**
-	 * Calls redraw listeners.
-	 * 
-	 * @author Daniel Kirschten
-	 */
-	protected void requestRedraw()
-	{
-		callRedrawListeners();
-	}
-
 	// @formatter:off
 	public void addComponentMovedListener      (Consumer<? super GUIComponent> listener) {componentMovedListeners  .add   (listener);}
 	public void addComponentResizedListener    (Consumer<? super GUIComponent> listener) {componentResizedListeners.add   (listener);}
 	public void addPinAddedListener            (Consumer<? super Pin         > listener) {pinAddedListeners        .add   (listener);}
 	public void addPinRemovedListener          (Consumer<? super Pin         > listener) {pinRemovedListeners      .add   (listener);}
-	public void addRedrawListener              (Runnable                       listener) {redrawListeners          .add   (listener);}
 
 	public void removeComponentMovedListener   (Consumer<? super GUIComponent> listener) {componentMovedListeners  .remove(listener);}
 	public void removeComponentResizedListener (Consumer<? super GUIComponent> listener) {componentResizedListeners.remove(listener);}
 	public void removePinAddedListener         (Consumer<? super Pin         > listener) {pinAddedListeners        .remove(listener);}
 	public void removePinRemovedListener       (Consumer<? super Pin         > listener) {pinRemovedListeners      .remove(listener);}
-	public void removeRedrawListener           (Runnable                       listener) {redrawListeners          .remove(listener);}
 
 	private void callComponentMovedListeners (     ) {componentMovedListeners  .forEach(l -> l.accept(this));}
 	private void callComponentResizedListener(     ) {componentResizedListeners.forEach(l -> l.accept(this));}
 	private void callPinAddedListeners       (Pin p) {pinAddedListeners        .forEach(l -> l.accept(p   ));}
 	private void callPinRemovedListeners     (Pin p) {pinRemovedListeners      .forEach(l -> l.accept(p   ));}
-	private void callRedrawListeners         (     ) {redrawListeners          .forEach(l -> l.run(       ));}
 	// @formatter:on
 }
\ No newline at end of file