/*
 * Licensed to GraphHopper GmbH under one or more contributor
 * license agreements. See the NOTICE file distributed with this work for
 * additional information regarding copyright ownership.
 *
 * GraphHopper GmbH licenses this file to you under the Apache License,
 * Version 2.0 (the "License"); you may not use this file except in
 * compliance with the License. You may obtain a copy of the License at
 *
 *       http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package com.graphhopper.jsprit.core.algorithm;

import com.graphhopper.jsprit.core.algorithm.recreate.listener.InsertionEndsListener;
import com.graphhopper.jsprit.core.problem.job.Job;
import com.graphhopper.jsprit.core.problem.solution.route.VehicleRoute;
import com.graphhopper.jsprit.core.problem.vehicle.VehicleFleetManager;

import java.util.ArrayList;
import java.util.Collection;
import java.util.List;


public class RemoveEmptyVehicles implements InsertionEndsListener {

    private VehicleFleetManager fleetManager;

    public RemoveEmptyVehicles(VehicleFleetManager fleetManager) {
        super();
        this.fleetManager = fleetManager;
    }

    @Override
    public String toString() {
        return "[name=removeEmptyVehicles]";
    }

    @Override
    public void informInsertionEnds(Collection<VehicleRoute> vehicleRoutes, Collection<Job> badJobs) {
        List<VehicleRoute> routes = new ArrayList<>(vehicleRoutes);
        for (VehicleRoute route : routes) {
            if (route.isEmpty()) {
                fleetManager.unlock(route.getVehicle());
                vehicleRoutes.remove(route);
            }
        }
    }
}

@@ -1122,36 +1122,8 @@
 r;%0A%0A
-import java.util.ArrayList;%0A
 impo
@@ -1164,20 +1164,24 @@
 va.util.
-List
+Iterator
 ;%0A%0A%0Apubl
@@ -1647,20 +1647,29 @@
         
-List
+for (Iterator
 %3CVehicle
@@ -1679,48 +1679,68 @@
 te%3E 
-routes = new ArrayList%3C%3E(vehicleRoutes);
+iterator = vehicleRoutes.iterator(); iterator.hasNext(); ) %7B
 %0A   
@@ -1744,21 +1744,20 @@
         
-for (
+    
 VehicleR
@@ -1771,19 +1771,26 @@
 ute 
-: routes) %7B
+= iterator.next();
 %0A   
@@ -1898,29 +1898,24 @@
     
-vehicleRoutes
+iterator
 .remove(
 rout
@@ -1910,21 +1910,16 @@
 .remove(
-route
 );%0A     

fix #452