This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
trunk as of 20030310 Disabled and enabled all java modules (form, java, ...) at org.netbeans.core.windows.toolbars.ToolbarConstraints.updateNextBars(ToolbarConstraints.java:382) at org.netbeans.core.windows.toolbars.ToolbarConstraints.updatePosition(ToolbarConstraints.java:347)
Adding Marek to cc. Looks like it could be fixed with a simple boolean flag somewhere, but I don't know this code.
Changing line 383 of ToolbarConstraints to test if it is calling this method on itself seems to work: if (tc != this) tc.updatePosition(); ^^^^^^^^^^^^^^^ I have no idea yet if it breaks something else. Disabling all of the java modules produced such an incredible number of exceptions it is difficult to tell.
Created attachment 9329 [details] Patch, appears to work - would someone closer to the code review?
I checked and fix seems ok. But I do not know this code either. It is not clear to me how nextBars can contain reference to itself.
David, now I can't reproduce this on my machine. The code definitely looks like it could get into an endless loop if it found itself in the set of toolbars it's iterating (that would be the real bug). Could you try the patch and see if it fixes it on your machine?
Committing this fix to the trunk & release35 branch - it can do no harm, and it definitely makes it physically impossible for an instance of ToolbarConstraints to produce this particular loop. David is still working on the other exceptions when you disable all Java modules, and will monitor if the problem recurs.
verified in [nb3.5](200304072350)
Seems this patch worked well so far. However from time to time it can be reproduced with GTK Look and Feel with some themes. I've just tested it with Ubuntu 9.04 Beta and New Wave LAF. I'm going to fix this and sand a patch. Till then I reopen this on P5.
Reassigning this, as I haven't worked on the winsys in years. Most likely, with the 6.7 toolbar rewrite, the code in question no longer exists.
I can't reproduce it in NetBeans 6.7.