If you do make the change of moving the `'self` definition this could/should be come `<'a>(crate_map:&'a CrateMap<'a>) -> (&'a [ModEntry], &'a [&'a CrateMap<'a>])` (and similarly for anywhere than mentions `'static`: it'll make testing nicer if we don't have the assumption of `'static`.)