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.

Bug 32328 - [TTV] infinite loop when adding debugger 'type' column
Summary: [TTV] infinite loop when adding debugger 'type' column
Status: VERIFIED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Explorer (show other bugs)
Version: 3.x
Hardware: All All
: P1 blocker (vote)
Assignee: Jiri Rechtacek
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-03-26 01:23 UTC by _ jrichard
Modified: 2008-12-22 21:54 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
a proposed patch (3.22 KB, patch)
2003-03-26 17:20 UTC, Jiri Rechtacek
Details | Diff
a binary patch to lib/patches (50.06 KB, application/octet-stream)
2003-03-26 17:21 UTC, Jiri Rechtacek
Details

Note You need to log in before you can comment on or make changes to this bug.
Description _ jrichard 2003-03-26 01:23:20 UTC
I was using the debugger with the 21-Mar-2003 dev
build and
tried to add the 'type' column to the list of
variables.  This
froze the IDE.  A stack dump showed sortSubNode
was repeated many 
times:

[sortSubNode repeated MANY times]
       at
org.openide.explorer.view.TreeTableView$SortedNodeTreeModel.sortSubNo
des(TreeTableView.java:1074)
       at
org.openide.explorer.view.TreeTableView$SortedNodeTreeModel.sortSubNo
des(TreeTableView.java:1074)
       at
org.openide.explorer.view.TreeTableView$SortedNodeTreeModel.sortingCh
anged(TreeTableView.java:1057)
       at
org.openide.explorer.view.TreeTableView$SortedNodeTreeModel.setSorted
ByProperty(TreeTableView.java:878)
       at
org.openide.explorer.view.TreeTableView.setSortingColumn(TreeTableVie
w.java:667)
       at
org.openide.explorer.view.TreeTableView.clickOnColumnAction(TreeTable
View.java:204)
       at
org.openide.explorer.view.TreeTableView.access$100(TreeTableView.java
:49)
       at
org.openide.explorer.view.TreeTableView$2.mouseClicked(TreeTableView.
java:169)
       at
java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:20
8)
       at
java.awt.Component.processMouseEvent(Component.java:5137)
Comment 1 Jan Jancura 2003-03-26 08:22:46 UTC
Looks like problem in openide
Comment 2 Marian Mirilovic 2003-03-26 09:56:57 UTC
Hanzi, please don't forget to reassigne issue if you change
subcomponent/component!
Comment 3 Marian Mirilovic 2003-03-26 11:09:56 UTC
I cannot reproduce it on [s1s5](030325) , neither
[nb_release35](200303252350)
Comment 4 Marian Mirilovic 2003-03-26 14:31:08 UTC
It must be fixed for release 35.
Comment 5 Jiri Rechtacek 2003-03-26 16:31:07 UTC
Evaluated: there is a problem the sorting follows a hierarchy of being
sorted node from the root deep to leaf. If a hierarchy is unlimited
then sorting of sub nodes is infinite. I have a fix prepared, I'm
testing it now, I'll attach it ASAP.
Comment 6 Jiri Rechtacek 2003-03-26 17:20:31 UTC
Created attachment 9549 [details]
a proposed patch
Comment 7 Jiri Rechtacek 2003-03-26 17:21:13 UTC
Created attachment 9550 [details]
a binary patch to lib/patches
Comment 8 Jiri Rechtacek 2003-03-26 17:35:08 UTC
fixed in maintrunk. The folders for sort are not from whole root's
hierarchy not. The folder comes from folders which are exactly
expanded in tree. It sure prevent a infinite look during getting
children still deeply w/o any leaf.
John, Marian: pls. assure it fix correctly works. Thanks
Comment 9 Marian Mirilovic 2003-03-27 09:04:19 UTC
patch verified
Comment 10 Jiri Rechtacek 2003-03-27 10:55:52 UTC
fix merged in release35-QBE200303252350 branch
Comment 11 _ ttran 2003-03-27 13:02:12 UTC
Code review for release35?
Comment 12 Petr Hrebejk 2003-03-27 13:51:36 UTC
Should be OK. Hrebejk
Comment 13 _ ttran 2003-03-27 13:59:07 UTC
approved by release coordinator for release35
Comment 14 Jiri Rechtacek 2003-03-27 14:08:46 UTC
integrated to release35 branch.
Comment 15 _ jrichard 2003-03-28 01:47:32 UTC
I'm not exactly sure what I did to cause this, beyond what
I entered above.  So as far as verifying the fix goes, 
it seemed OK...



Comment 16 Marian Mirilovic 2003-04-03 12:19:48 UTC
verified in [nb3.5](200304022350)