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.
Summary: | Default value for "Debugger" field in the Attach Debugger dialog is unpredictable | ||
---|---|---|---|
Product: | debugger | Reporter: | Alexander Zgursky <zgursky> |
Component: | Code | Assignee: | Martin Entlicher <mentlicher> |
Status: | VERIFIED FIXED | ||
Severity: | blocker | CC: | thp |
Priority: | P2 | ||
Version: | 5.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Attachments: | The textual diff of the fix. |
Description
Alexander Zgursky
2006-08-03 15:11:37 UTC
They are obtained from the lookup, which uses hash maps and does not guarantee any particular order. We can e.g. sort them alphabetically to have consistent UI. Fixed in trunk: /cvs/debuggercore/src/org/netbeans/modules/debugger/ui/actions/ConnectorPanel.java,v <-- ConnectorPanel.java new revision: 1.8; previous revision: 1.7 OK, the consistent order is the one part of the issue. Another part is that the user's choice is not remembered by the IDE. Should I post an enhancement on that? Well, there is some commented relict of the code that stores the default value. I'll find out why it was removed and if there's nothing wrong with it, I'll add it back... It was likely planned, but actually never implemented. I've made the storage of the last value in debugger settings. I've just JPDA debugger here, so please verify that it works as expected. Thanks. /cvs/debuggercore/src/org/netbeans/modules/debugger/ui/actions/ConnectorPanel.java,v <-- ConnectorPanel.java new revision: 1.9; previous revision: 1.8 Is it possible to integrate a fix into release55 branch? Well, it does not seem to be important enough, sorry. The policy for fixes into 5.5 are: * P1 regression over 5.0 * Blocker for Java EE 5 development * Highly user voted bug whose fix does not require update to many NetBeans parts This does not seem to be any of these. I see. OK then. Thanks to Ivan, we discovered a bug in the fix, the compareTo() method must be called with a String argument: /cvs/debuggercore/src/org/netbeans/modules/debugger/ui/actions/ConnectorPanel.java,v <-- ConnectorPanel.java new revision: 1.10; previous revision: 1.9 *** Issue 86965 has been marked as a duplicate of this issue. *** Created attachment 35338 [details]
The textual diff of the fix.
The fix of this issue is merged into release55_mars branch: /shared/data/ccvs/repository/debuggercore/src/org/netbeans/modules/debugger/ui/actions/ConnectorPanel.java,v <-- ConnectorPanel.java new revision: 1.5.8.1.2.1.18.1; previous revision: 1.5.8.1.2.1 This is very annoying issue. Please integrate the fix to release55_fixes branch as well in order to get this to HotFixes AUC. Thank you. The fix is merged into release55_fixes: /shared/data/ccvs/repository/debuggercore/src/org/netbeans/modules/debugger/ui/actions/ConnectorPanel.java,v <-- ConnectorPanel.java new revision: 1.5.8.1.2.1.20.1; previous revision: 1.5.8.1.2.1 And the module spec. version is increased: /shared/data/ccvs/repository/debuggercore/manifest.mf,v <-- manifest.mf new revision: 1.71.10.1.2.3.4.1; previous revision: 1.71.10.1.2.3 The fix is merged into release551: /shared/data/ccvs/repository/debuggercore/src/org/netbeans/modules/debugger/ui/actions/ConnectorPanel.java,v <-- ConnectorPanel.java new revision: 1.5.8.1.2.1.22.1; previous revision: 1.5.8.1.2.1 The fix works like a charm in NetBeans 5.5 FCS patched by NBM file built from sources checked out on December 01. Affected dialog always correctly remembers last selected debugger provider. It's a Go from QA to have the NBM staged on 5.5 HotFix AUC. |