Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Required file resolver extension point #38

Open
wants to merge 3 commits into
base: testing
Choose a base branch
from

Conversation

frerich
Copy link

@frerich frerich commented Apr 11, 2017

This patch introduces a new extension point which permits expressing
custom algorithms for locating files specified with the 'require'
statement. Such handlers will be considered first by the 'Open
Declaration' functionality which traverses the set of included files
when trying to look for a declaration.

This allows implementing custom logic for expressions like

require customGlobalObject->myLibraryFile;

or similar: users can manually evaluate the variable name and return an
appropriate File object (or null, if the given string cannot be resolved
to any File).

Frerich Raabe added 3 commits April 7, 2017 09:23
This patch introduces a new extension point which permits expressing
custom algorithms for locating files specified with the 'require'
statement. Such handlers will be considered first by the 'Open
Declaration' functionality which traverses the set of included files
when trying to look for a declaration.

This allows implementing custom logic for expressions like

  require customGlobalObject->myLibraryFile;

or similar: users can manually evaluate the variable name and return an
appropriate File object (or null, if the given string cannot be resolved
to any File).
Avoids an 'import' in AbstractOpenDeclaration.
That way the errors are written to Eclipse' workspace log file and show up in
the Error Log View in Eclipse itself too.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant