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: | CodeCompletion took 2228 ms. | ||
---|---|---|---|
Product: | platform | Reporter: | KhArtNJava |
Component: | -- Other -- | Assignee: | Tomas Zezula <tzezula> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | claucrespo, KhArtNJava, mfukala, tzezula, vercinstex |
Priority: | P1 | Keywords: | PERFORMANCE |
Version: | 8.0 | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | 206892 |
Attachments: |
nps snapshot
nps snapshot |
Description
KhArtNJava
2014-01-23 12:55:13 UTC
Created attachment 144293 [details]
nps snapshot
Created attachment 144798 [details]
nps snapshot
hes got a lot to think about...
In fact nothing to do with parsing.api. Caused by changes of Utilities.toURI() which needlessly accesses disk, before JDK 7 rewrite to use j.nio.Path it used j.io.File.toURI(). Also it's questionable what CSS|HTML code completion does with all the files. In my opinion one possible easy fix is to change CssIndex & friends like this: for(IndexResult result : results) { - FileObject resultFile = result.getFile(); //can be null as the file can be removed/moved + URI resultFile = uriOrNull(result.getURL); if(resultFile != null && resultFile.equals(fileURI)) { return factory.loadFromIndex(result); } } Which saves O(n) calls of Utilities.toURI() and O(n) non needed disk touches. I will try to find all the places which need to be changed like this. Fixed jet-main cafe8f1ef534 What a nice hg hash :-) Integrated into 'main-silver', will be available in build *201402200001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress) Changeset: http://hg.netbeans.org/main-silver/rev/cafe8f1ef534 User: Tomas Zezula <tzezula@netbeans.org> Log: #240880:CodeCompletion took 2228 ms. |