optional nit: I think it's cleaner to put this code below in the struct constructor, e.g.  ```rust  Resolver {      ...      // `#![feature(proc_macro)]` implies `#[feature(extern_macros)]`      use_extern_macros: features.use_extern_macros || proc_macro_enabled;  }  ```