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: | Go to symbol spends too much time computing symbol names for cymbols that are not visible | ||
---|---|---|---|
Product: | utilities | Reporter: | Vladimir Kvashin <vkvashin> |
Component: | Jump To | Assignee: | Tomas Zezula <tzezula> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | issues |
Priority: | P2 | ||
Version: | 8.1 | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Attachments: |
Proposed change
Improved patch |
Description
Vladimir Kvashin
2014-11-27 09:10:16 UTC
Created attachment 153552 [details]
Proposed change
I've added a proposed patch (not final). Production patch will be a bit more complicated. Current patch can update the name and icon however for java also removing already emitted SymbolDescriptor (when symbol is not resolvable) and forking of SymbolDescriptor to more SymbolDescriptors (for method overloadind, for every symbol name in a class one SymbolDescritor is emitted as method arguments are unknown, when the arguments are calculated there may be overloading and more SymbolDescriptors is needed). I also need to relax the SymbolComparator written by Victor Vasilyev not to use the SymbolDescriptor.getSymbolName which will cause calculation of args but SD.getSimpleName. This causes that the argument list is not included into sorting. Are you OK with it? Tomáš, thank you! Yes, I'm ok with not including argument list into sorting. Thanks Vladimir! I will continue on the work on the patch for NB 8.2. Sorry, I'm not aware of 8.2, only of upcoming 8.1 and 9; am I missing something? Sorry, it was a typo, I've wanted to write 8.1 Great! Thank you so much! Created attachment 153699 [details]
Improved patch
Fixed jet-main 36665e68fff8 Integrated into 'main-silver', will be available in build *201505210001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main-silver/rev/36665e68fff8 User: Tomas Zezula <tzezula@netbeans.org> Log: #248941:Go to symbol spends too much time computing symbol names for cymbols that are not visible |