Spring/spring-tools
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix NPE in resolveClasspath

Browse Source 
Caused by unusual projects that do not use
JRE classpath container to point to
system libraries.
This commit is contained in:
Kris De Volder
2018-05-22 14:45:24 -07:00
parent 2fee63d35e
commit 394a7b5543
1 changed files with 13 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
headless-services/jdt-ls-extension/org.springframework.tooling.jdt.ls.commons/src/org/springframework/tooling/jdt/ls/commons/classpath/ClasspathUtil.java
View File

@@ -36,12 +36,14 @@ public class ClasspathUtil {
private static Set<String> getSystemLibraryPaths(IJavaProject javaProject) {
try {
IClasspathEntry jreContainer = getJreContainer(javaProject.getRawClasspath());
IClasspathEntry[] resolvedJreEntries = ((JavaProject)javaProject).resolveClasspath(new IClasspathEntry[] {jreContainer});
Set<String> paths = new HashSet<>();
for (IClasspathEntry systemEntry : resolvedJreEntries) {
paths.add(systemEntry.getPath().toString());
if (jreContainer!=null) {
IClasspathEntry[] resolvedJreEntries = ((JavaProject)javaProject).resolveClasspath(new IClasspathEntry[] {jreContainer});
Set<String> paths = new HashSet<>();
for (IClasspathEntry systemEntry : resolvedJreEntries) {
paths.add(systemEntry.getPath().toString());
}
return paths;
}
return paths;
} catch (Exception e) {
e.printStackTrace();
}
@@ -49,10 +51,12 @@ public class ClasspathUtil {
}
private static IClasspathEntry getJreContainer(IClasspathEntry[] rawClasspath) {
for (IClasspathEntry cpe : rawClasspath) {
if (cpe.getEntryKind()==IClasspathEntry.CPE_CONTAINER) {
if (cpe.getPath().segment(0).equals(JRE_CONTAINER_ID)) {
return cpe;
if (rawClasspath!=null) {
for (IClasspathEntry cpe : rawClasspath) {
if (cpe.getEntryKind()==IClasspathEntry.CPE_CONTAINER) {
if (cpe.getPath().segment(0).equals(JRE_CONTAINER_ID)) {
return cpe;
}
}
}
}