Skip to content

Commit 4096368

Browse files
committed
cleanup, add suggestions from PR
1 parent 180e5b8 commit 4096368

File tree

1 file changed

+6
-10
lines changed

1 file changed

+6
-10
lines changed

ui/org.eclipse.pde.bnd.ui/src/org/eclipse/pde/bnd/ui/tasks/AnalyseBundleResolutionJob.java

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,7 @@
2929
import java.util.Set;
3030
import java.util.function.Predicate;
3131

32-
import org.bndtools.api.ILogger;
33-
import org.bndtools.api.Logger;
32+
import org.eclipse.core.runtime.ILog;
3433
import org.eclipse.core.runtime.IProgressMonitor;
3534
import org.eclipse.core.runtime.IStatus;
3635
import org.eclipse.core.runtime.Status;
@@ -41,12 +40,9 @@
4140

4241
import aQute.bnd.build.model.EE;
4342
import aQute.bnd.osgi.resource.ResourceUtils;
44-
import aQute.lib.io.IO;
4543

4644
public class AnalyseBundleResolutionJob extends Job {
4745

48-
private static final ILogger logger = Logger.getLogger(AnalyseBundleResolutionJob.class);
49-
5046
private final Set<? extends CapReqLoader> loaders;
5147

5248
private Map<String, List<RequirementWrapper>> requirements;
@@ -86,17 +82,15 @@ protected IStatus run(IProgressMonitor monitor) {
8682
Map<String, List<Capability>> allCaps = new HashMap<>();
8783
Map<String, List<RequirementWrapper>> allReqs = new HashMap<>();
8884
for (CapReqLoader loader : loaders) {
89-
try {
85+
try (loader){
9086
Map<String, List<Capability>> caps = loader.loadCapabilities();
9187
mergeMaps(caps, allCaps);
9288

9389
Map<String, List<RequirementWrapper>> reqs = loader.loadRequirements();
9490
mergeMaps(reqs, allReqs);
9591
} catch (Exception e) {
96-
logger.logError("Error in Bnd resolution analysis.", e);
97-
} finally {
98-
IO.close(loader);
99-
}
92+
ILog.get().error("Error in Bnd resolution analysis.", e);
93+
}
10094
}
10195

10296
// Check for resolved requirements
@@ -139,6 +133,8 @@ protected IStatus run(IProgressMonitor monitor) {
139133

140134
// showResults(resultFileArray, importResults, exportResults);
141135
return Status.OK_STATUS;
136+
} catch (RuntimeException e) {
137+
throw e;
142138
} catch (Exception e) {
143139
throw new RuntimeException(e);
144140
}

0 commit comments

Comments
 (0)