Logviewer does not remove it's preference store PropertyChangeListener

[GoogleCodeExporter]

What steps will reproduce the problem?
1.Open the log viewer
2.Close the log viewer
3.Open the log viewer
4.Change Font settings

What is the expected output? What do you see instead?
After changing the settings, i get the attached exception

I suggest to remove the PropertyChangeListener on disposal of a LogViewer 
instance:
@Override
public void dispose() {
   viewer.removeListeners();  // new method
   storeAllCurrentlyOpenFiles();
   super.dispose();
}

The new method in LogFileViewer looks like: 
public void removeListeners() {
   if (propertyChangeListener != null) {
      store.removePropertyChangeListener(propertyChangeListener);
   }
}

Original issue reported on code.google.com by matthias...@gmail.com on 24 Aug 2011 at 12:17

Attachments:

• Stacktrace_propertylistener.txt

[GoogleCodeExporter]

Yes, i see the problem. I will add your patch for next release. Thanks!

Original comment by andre.bo...@gmail.com on 24 Aug 2011 at 5:29

• Changed state: Accepted
• Added labels: Milestone-Release-Candidate-0.9.9, OpSys-All

[GoogleCodeExporter]

Original comment by andre.bo...@anbos.de on 29 Sep 2012 at 7:05

[GoogleCodeExporter]

Original comment by andre.bo...@anbos.de on 16 Jan 2013 at 8:51

• Changed state: Started

[GoogleCodeExporter]

Original comment by andre.bo...@anbos.de on 20 Sep 2013 at 5:18

• Added labels: Milestone-Release1.0
• Removed labels: Milestone-Release-Candidate-0.9.9

[anb0s]

Thanks, fixed!