Bug 438087 - Reduce AST lookup by DLTK IImportDeclaration
Summary: Reduce AST lookup by DLTK IImportDeclaration
Status: NEW
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: PDT (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: PHP Core CLA
QA Contact:
URL:
Whiteboard:
Keywords: performance
Depends on: 455754 472740 437856
Blocks: 440965
  Show dependency tree
 
Reported: 2014-06-24 14:53 EDT by Dawid Pakula CLA
Modified: 2020-05-14 10:15 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dawid Pakula CLA 2014-06-24 14:53:12 EDT 
In theory PDT is registering in model all imports. But, while resolving references (for example in TI or CA), AST is required to detect aliases. Since 5.6 problem will grow up, because will be possible to import functions and constants.

If bug 437856 cannot be resolved, we have to find way to put these informations into type metadata.
Comment 1 Michal Niewrzal CLA 2015-05-11 05:45:39 EDT 
@Dawid, do you plan to work with this or I can give it a shot?
Comment 2 Dawid Pakula CLA 2015-05-11 05:46:39 EDT 
I'm working on refactoring now.
Comment 3 Eclipse Genie CLA 2016-07-13 07:41:51 EDT 
New Gerrit change created: https://git.eclipse.org/r/77234
Comment 5 Michal Niewrzal CLA 2016-07-18 06:39:23 EDT 
Merged patch is reducing AST lookup for use statements only around Outline view.