Simplify some lifetimes

Author: camsteffen

src/modules.rs

@@ -95,16 +95,16 @@ pub(crate) enum ModuleResolutionErrorKind {
 }
 
 #[derive(Clone)]
-enum SubModKind<'a, 'ast> {
+enum SubModKind<'a> {
     /// `mod foo;`
-    External(PathBuf, DirectoryOwnership, Module<'ast>),
+    External(PathBuf, DirectoryOwnership, Module<'static>),
     /// `mod foo;` with multiple sources.
-    MultiExternal(Vec<(PathBuf, DirectoryOwnership, Module<'ast>)>),
+    MultiExternal(Vec<(PathBuf, DirectoryOwnership, Module<'static>)>),
     /// `mod foo {}`
     Internal(&'a ast::Item),
 }
 
-impl<'ast, 'sess, 'c> ModResolver<'ast, 'sess> {
+impl<'ast, 'sess> ModResolver<'ast, 'sess> {
     /// Creates a new `ModResolver`.
     pub(crate) fn new(
         parse_sess: &'sess ParseSess,
@@ -204,7 +204,7 @@ impl<'ast, 'sess, 'c> ModResolver<'ast, 'sess> {
 
     fn visit_sub_mod(
         &mut self,
-        item: &'c ast::Item,
+        item: &ast::Item,
         sub_mod: Module<'_>,
         from_ast: bool,
     ) -> Result<(), ModuleResolutionError> {
@@ -217,11 +217,11 @@ impl<'ast, 'sess, 'c> ModResolver<'ast, 'sess> {
     }
 
     /// Inspect the given sub-module which we are about to visit and returns its kind.
-    fn peek_sub_mod(
+    fn peek_sub_mod<'a>(
         &self,
-        item: &'c ast::Item,
+        item: &'a ast::Item,
         sub_mod: &Module<'_>,
-    ) -> Result<Option<SubModKind<'c, 'ast>>, ModuleResolutionError> {
+    ) -> Result<Option<SubModKind<'a>>, ModuleResolutionError> {
         if contains_skip(&item.attrs) {
             return Ok(None);
         }
@@ -238,7 +238,7 @@ impl<'ast, 'sess, 'c> ModResolver<'ast, 'sess> {
 
     fn insert_sub_mod(
         &mut self,
-        sub_mod_kind: SubModKind<'c, 'ast>,
+        sub_mod_kind: SubModKind<'_>,
     ) -> Result<(), ModuleResolutionError> {
         match sub_mod_kind {
             SubModKind::External(mod_path, _, sub_mod) => {
@@ -261,7 +261,7 @@ impl<'ast, 'sess, 'c> ModResolver<'ast, 'sess> {
     fn visit_sub_mod_inner(
         &mut self,
         sub_mod: Module<'_>,
-        sub_mod_kind: SubModKind<'c, 'ast>,
+        sub_mod_kind: SubModKind<'_>,
         from_ast: bool,
     ) -> Result<(), ModuleResolutionError> {
         match sub_mod_kind {
@@ -303,7 +303,7 @@ impl<'ast, 'sess, 'c> ModResolver<'ast, 'sess> {
         mod_name: symbol::Ident,
         attrs: &[ast::Attribute],
         sub_mod: &Module<'_>,
-    ) -> Result<Option<SubModKind<'c, 'ast>>, ModuleResolutionError> {
+    ) -> Result<Option<SubModKind<'static>>, ModuleResolutionError> {
         let relative = match self.directory.ownership {
             DirectoryOwnership::Owned { relative } => relative,
             DirectoryOwnership::UnownedViaBlock => None,
@@ -468,7 +468,7 @@ impl<'ast, 'sess, 'c> ModResolver<'ast, 'sess> {
         &self,
         attrs: &[ast::Attribute],
         sub_mod: &Module<'_>,
-    ) -> Vec<(PathBuf, DirectoryOwnership, Module<'ast>)> {
+    ) -> Vec<(PathBuf, DirectoryOwnership, Module<'static>)> {
         // Filter nested path, like `#[cfg_attr(feature = "foo", path = "bar.rs")]`.
         let mut path_visitor = visitor::PathVisitor::default();
         for attr in attrs.iter() {