fixed potential memory leak when motes are removed during a simulation

This commit is contained in:
fros4943 2009-10-28 15:42:03 +00:00
parent a4226e14c5
commit 281639c779

View file

@ -26,7 +26,7 @@
* OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
* SUCH DAMAGE.
*
* $Id: LogVisualizerSkin.java,v 1.3 2009/08/27 13:59:47 fros4943 Exp $
* $Id: LogVisualizerSkin.java,v 1.4 2009/10/28 15:42:03 fros4943 Exp $
*/
package se.sics.cooja.plugins.skins;
@ -43,6 +43,7 @@ import org.apache.log4j.Logger;
import se.sics.cooja.ClassDescription;
import se.sics.cooja.Mote;
import se.sics.cooja.Simulation;
import se.sics.cooja.SimEventCentral.MoteCountListener;
import se.sics.cooja.interfaces.Log;
import se.sics.cooja.interfaces.Position;
import se.sics.cooja.plugins.Visualizer;
@ -67,17 +68,18 @@ public class LogVisualizerSkin implements VisualizerSkin {
visualizer.repaint();
}
};
private Observer simObserver = new Observer() {
public void update(Observable obs, Object obj) {
/* Observe logs */
for (Mote mote: simulation.getMotes()) {
private MoteCountListener newMotesListener = new MoteCountListener() {
public void moteWasAdded(Mote mote) {
Log log = mote.getInterfaces().getLog();
if (log != null) {
log.addObserver(logObserver);
}
}
visualizer.repaint();
public void moteWasRemoved(Mote mote) {
Log log = mote.getInterfaces().getLog();
if (log != null) {
log.deleteObserver(logObserver);
}
}
};
@ -85,17 +87,16 @@ public class LogVisualizerSkin implements VisualizerSkin {
this.simulation = simulation;
this.visualizer = vis;
simulation.addObserver(simObserver);
simObserver.update(null, null);
simulation.getEventCentral().addMoteCountListener(newMotesListener);
for (Mote m: simulation.getMotes()) {
newMotesListener.moteWasAdded(m);
}
}
public void setInactive() {
simulation.deleteObserver(simObserver);
for (Mote mote: simulation.getMotes()) {
Log log = mote.getInterfaces().getLog();
if (log != null) {
log.deleteObserver(logObserver);
}
simulation.getEventCentral().removeMoteCountListener(newMotesListener);
for (Mote m: simulation.getMotes()) {
newMotesListener.moteWasRemoved(m);
}
}