[news.eclipse.tools] For Randy (the patch maker....:))

["Johan Compagner" <jcompagner@xxxxxxxx>]

Randy (i don't have youre mail at hand)

there is a bug in the implementation of your CTabFolder

in the dispose you do this:
 notSelectedBG.dispose();
 notSelectedFG.dispose();


but i think one of those is sometimes nul at my place because i get this:

java.lang.NullPointerException
 at org.eclipse.swt.custom.CTabFolder.dispose(CTabFolder.java:529)
 at org.eclipse.ui.internal.PartTabFolder.dispose(PartTabFolder.java:325)
 at org.eclipse.ui.internal.PartSashContainer.dispose(PartSashContainer.java:228)
 at org.eclipse.ui.internal.PerspectivePresentation.deactivate(PerspectivePresentation.java:446)
 at org.eclipse.ui.internal.Perspective.onDeactivate(Perspective.java:740)
 at org.eclipse.ui.internal.WorkbenchPage.setPerspective(WorkbenchPage.java:2572)
 at org.eclipse.ui.internal.WorkbenchPage.busySetPerspective(WorkbenchPage.java:624)
 at org.eclipse.ui.internal.WorkbenchPage.access$8(WorkbenchPage.java:609)
 at org.eclipse.ui.internal.WorkbenchPage$12.run(WorkbenchPage.java:2705)


so i think null checks must be before those 2 lines..

johan