mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-06-08 02:06:01 +02:00
Code Issues Wiki Activity

Corrected logic for determining if a URI points to a workspace file.

Browse source
This commit is contained in:
Sergey Prigogin 2010-05-19 06:59:01 +00:00
parent 8f07ba0efc
commit bd4da84e73
1 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/editor/AddIncludeOnSelectionAction.java
View file

@ -25,6 +25,7 @@ import java.util.LinkedList;
import java.util.List;
import java.util.Map;
import org.eclipse.core.resources.IFile;
import org.eclipse.core.resources.IProject;
import org.eclipse.core.runtime.CoreException;
import org.eclipse.core.runtime.IPath;
@ -442,7 +443,7 @@ public class AddIncludeOnSelectionAction extends TextEditorAction {
*/
private IIndexFile getRepresentativeFile(IIndexFile headerFile, IIndex index) {
try {
if (ResourceLookup.findFilesForLocationURI(headerFile.getLocation().getURI()).length > 0) {
if (isWorkspaceFile(headerFile.getLocation().getURI())) {
return headerFile;
}
// TODO(sprigogin): Change to ArrayDeque when Java 5 support is no longer needed.
@ -461,7 +462,7 @@ public class AddIncludeOnSelectionAction extends TextEditorAction {
IIndexFile includer = include.getIncludedBy();
if (!processed.contains(includer)) {
URI uri = includer.getLocation().getURI();
if (isSource(uri.getPath()) || ResourceLookup.findFilesForLocationURI(uri).length > 0) {
if (isSource(uri.getPath()) || isWorkspaceFile(uri)) {
return file;
}
front.add(includer);
@ -475,6 +476,15 @@ public class AddIncludeOnSelectionAction extends TextEditorAction {
return headerFile;
}
private boolean isWorkspaceFile(URI uri) {
for (IFile file : ResourceLookup.findFilesForLocationURI(uri)) {
if (file.exists()) {
return true;
}
}
return false;
}
private boolean hasExtension(String path) {
return path.indexOf('.', path.lastIndexOf('/') + 1) >= 0;
}