make all crates build (sim.save became mut)

gtfs/src/lib.rs

@@ -50,7 +50,6 @@ pub fn load(dir_path: &str) -> Vec<Route> {
     for (key, group) in
         GTFSIterator::<_, transitfeed::StopTime>::from_path(&format!("{}/stop_times.txt", dir_path))
             .unwrap()
-            .into_iter()
             .map(|rec| rec.unwrap())
             .group_by(|rec| rec.trip_id.clone())
             .into_iter()

sim/src/sim.rs

@@ -632,7 +632,7 @@ impl Sim {
         self.run_until_done(map, |_, _| {}, time_limit);
     }
 
-    pub fn run_until_done<F: Fn(&Sim, &Map)>(
+    pub fn run_until_done<F: Fn(&mut Sim, &Map)>(
         &mut self,
         map: &Map,
         callback: F,

tests/src/runner.rs

@@ -248,7 +248,7 @@ pub struct TestHelper {
 }
 
 impl TestHelper {
-    pub fn setup_done(&mut self, sim: &Sim) {
+    pub fn setup_done(&mut self, sim: &mut Sim) {
         if self.debug_with_savestate.is_some() {
             panic!("Can't call setup_done twice in one test");
         }

tests/src/sim_completion.rs

@@ -9,7 +9,7 @@ pub fn run(t: &mut TestRunner) {
         flags.opts.savestate_every = Some(Duration::seconds(30.0));
         let (map, mut sim, mut rng) = flags.load(&mut Timer::throwaway());
         Scenario::small_run(&map).instantiate(&mut sim, &map, &mut rng, &mut Timer::throwaway());
-        h.setup_done(&sim);
+        h.setup_done(&mut sim);
         sim.just_run_until_done(&map, Some(Duration::minutes(70)));
     });
 }

tests/src/sim_determinism.rs

@@ -98,7 +98,7 @@ pub fn run(t: &mut TestRunner) {
         }
 
         let mut sim3: Sim =
-            Sim::load_savestate(sim1_save.clone(), &mut Timer::throwaway()).unwrap();
+            Sim::load_savestate(sim1_save.clone(), &map, &mut Timer::throwaway()).unwrap();
         sim3.set_name("with_savestating_3".to_string());
         if sim3 != sim2 {
             panic!(

tests/src/transit.rs

@@ -11,7 +11,7 @@ pub fn run(t: &mut TestRunner) {
         let route = map.get_bus_route("49").unwrap();
         let buses = sim.seed_bus_route(route, &map, &mut Timer::throwaway());
         let bus = buses[0];
-        h.setup_done(&sim);
+        h.setup_done(&mut sim);
 
         let mut expectations: Vec<Event> = Vec::new();
         // TODO assert stuff about other buses as well, although the timing is a little unclear
@@ -61,7 +61,7 @@ pub fn run(t: &mut TestRunner) {
             .0
             .unwrap();
         sim.spawn_all_trips(&map, &mut Timer::throwaway(), false);
-        h.setup_done(&sim);
+        h.setup_done(&mut sim);
 
         sim.run_until_expectations_met(
             &map,

tests/src/trips.rs

@@ -22,7 +22,7 @@ pub fn run(t: &mut TestRunner) {
             &map,
         );
         sim.spawn_all_trips(&map, &mut Timer::throwaway(), false);
-        h.setup_done(&sim);
+        h.setup_done(&mut sim);
 
         sim.run_until_expectations_met(
             &map,