Remove some (hopefully) unnecessary clippy and compiler directives

src/bin/fish_indent.rs

@@ -29,7 +29,6 @@ use fish::eprintf;
 use fish::expand::INTERNAL_SEPARATOR;
 use fish::fds::set_cloexec;
 use fish::fprintf;
-#[allow(unused_imports)]
 use fish::future::{IsSomeAnd, IsSorted};
 use fish::global_safety::RelaxedAtomicBool;
 use fish::highlight::{colorize, highlight_shell, HighlightRole, HighlightSpec};

src/editable_line.rs

@@ -1,4 +1,3 @@
-#[allow(unused_imports)]
 use crate::future::IsSomeAnd;
 use crate::highlight::HighlightSpec;
 use crate::wchar::prelude::*;

src/env/environment.rs

@@ -341,7 +341,6 @@ impl EnvStack {
     /// If `always` is set, perform synchronization even if there's no pending changes from this
     /// instance (that is, look for changes from other fish instances).
     /// Return a list of events for changed variables.
-    #[allow(clippy::vec_box)]
     pub fn universal_sync(&self, always: bool) -> Vec<Event> {
         if UVAR_SCOPE_IS_GLOBAL.load() {
             return Vec::new();

src/locale.rs

@@ -60,7 +60,6 @@ unsafe fn lconv_to_locale(lconv: &libc::lconv) -> Locale {
 
 /// Read the numeric locale, or None on any failure.
 #[cfg(localeconv_l)]
-#[allow(non_snake_case)]
 unsafe fn read_locale() -> Option<Locale> {
     extern "C" {
         fn localeconv_l(loc: libc::locale_t) -> *const libc::lconv;

src/screen.rs

@@ -26,7 +26,6 @@ use crate::curses::{term, tparm0, tparm1};
 use crate::env::{Environment, TERM_HAS_XN};
 use crate::fallback::fish_wcwidth;
 use crate::flog::FLOGF;
-#[allow(unused_imports)]
 use crate::future::IsSomeAnd;
 use crate::global_safety::RelaxedAtomicBool;
 use crate::highlight::HighlightColorResolver;