Go to symbol spends too much time computing symbol names for cymbols that are not visible. This slows down a lot. It would be fine if it computed names on demand.
Created attachment 153552 [details]
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.
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]
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)
User: Tomas Zezula <email@example.com>
Log: #248941:Go to symbol spends too much time computing symbol names for cymbols that are not visible