clean up

graphhopper · Mar 28, 2020 · 1ebf236 · 1ebf236
1 parent a3a4e6f
commit 1ebf236
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/...ore/src/main/java/com/graphhopper/jsprit/core/algorithm/module/RuinAndRecreateModule.java b/...ore/src/main/java/com/graphhopper/jsprit/core/algorithm/module/RuinAndRecreateModule.java
@@ -80,8 +80,7 @@ public void setProportionOfUnassignedJobsToBeReinserted(double proportionOfUnass
     @Override
     public VehicleRoutingProblemSolution runAndGetSolution(VehicleRoutingProblemSolution previousVrpSolution) {
         Collection<Job> ruinedJobs = ruin.ruin(previousVrpSolution.getRoutes());
-        Set<Job> ruinedJobSet = new HashSet<>();
-        ruinedJobSet.addAll(ruinedJobs);
+        Set<Job> ruinedJobSet = new HashSet<>(ruinedJobs);
         List<Job> stillUnassignedInThisIteration = new ArrayList<>();
         if (previousVrpSolution.getUnassignedJobs().size() < minUnassignedJobsToBeReinserted) {
             ruinedJobSet.addAll(previousVrpSolution.getUnassignedJobs());
@@ -100,8 +99,7 @@ public VehicleRoutingProblemSolution runAndGetSolution(VehicleRoutingProblemSolu
         previousVrpSolution.getUnassignedJobs().clear();
         previousVrpSolution.getUnassignedJobs().addAll(unassignedJobs);
         previousVrpSolution.getUnassignedJobs().addAll(stillUnassignedInThisIteration);
-        VehicleRoutingProblemSolution newSolution = previousVrpSolution;
-        return newSolution;
+        return previousVrpSolution;
 
     }