chore: fix compiler warnings

Author: calebcartwright

Diff for: src/formatting.rs

@@ -100,7 +100,7 @@ fn format_project(
         None
     };
 
-    let krate = match Parser::parse_crate(config, input, directory_ownership, &parse_session) {
+    let krate = match Parser::parse_crate(input, &parse_session) {
         Ok(krate) => krate,
         Err(e) => {
             return Err(OperationError::ParseError {

Diff for: src/formatting/comment.rs

@@ -1422,7 +1422,7 @@ impl<'a> Iterator for LineClasses<'a> {
             None => unreachable!(),
         };
 
-        let mut prev_kind = FullCodeCharKind::Normal;
+        let mut prev_kind: FullCodeCharKind;
         while let Some((kind, c)) = self.base.next() {
             prev_kind = self.kind;
             // needed to set the kind of the ending character on the last line
@@ -1723,7 +1723,8 @@ fn remove_comment_header(comment: &str) -> &str {
     } else {
         assert!(
             comment.starts_with("/*"),
-            format!("string '{}' is not a comment", comment)
+            "string '{}' is not a comment",
+            comment,
         );
         &comment[2..comment.len() - 2]
     }

Diff for: src/formatting/syntux/parser.rs

@@ -12,7 +12,7 @@ use rustc_span::{sym, symbol::kw, Span};
 
 use crate::formatting::attr::first_attr_value_str_by_name;
 use crate::formatting::syntux::session::ParseSess;
-use crate::{Config, Input};
+use crate::Input;
 
 pub(crate) type DirectoryOwnership = rustc_expand::module::DirOwnership;
 pub(crate) type ModulePathSuccess = rustc_expand::module::ModulePathSuccess;
@@ -32,10 +32,8 @@ pub(crate) struct Parser<'a> {
 /// A builder for the `Parser`.
 #[derive(Default)]
 pub(crate) struct ParserBuilder<'a> {
-    config: Option<&'a Config>,
     sess: Option<&'a ParseSess>,
     input: Option<Input>,
-    directory_ownership: Option<DirectoryOwnership>,
 }
 
 impl<'a> ParserBuilder<'a> {
@@ -49,19 +47,6 @@ impl<'a> ParserBuilder<'a> {
         self
     }
 
-    pub(crate) fn config(mut self, config: &'a Config) -> ParserBuilder<'a> {
-        self.config = Some(config);
-        self
-    }
-
-    pub(crate) fn directory_ownership(
-        mut self,
-        directory_ownership: Option<DirectoryOwnership>,
-    ) -> ParserBuilder<'a> {
-        self.directory_ownership = directory_ownership;
-        self
-    }
-
     pub(crate) fn build(self) -> Result<Parser<'a>, ParserError> {
         let sess = self.sess.ok_or(ParserError::NoParseSess)?;
         let input = self.input.ok_or(ParserError::NoInput)?;
@@ -158,12 +143,10 @@ impl<'a> Parser<'a> {
     }
 
     pub(crate) fn parse_crate(
-        config: &'a Config,
         input: Input,
-        directory_ownership: Option<DirectoryOwnership>,
         sess: &'a ParseSess,
     ) -> Result<ast::Crate, ParserError> {
-        let krate = Parser::parse_crate_inner(config, input, directory_ownership, sess)?;
+        let krate = Parser::parse_crate_inner(input, sess)?;
         if !sess.has_errors() {
             return Ok(krate);
         }
@@ -176,19 +159,8 @@ impl<'a> Parser<'a> {
         Err(ParserError::ParseError)
     }
 
-    fn parse_crate_inner(
-        config: &'a Config,
-        input: Input,
-        directory_ownership: Option<DirectoryOwnership>,
-        sess: &'a ParseSess,
-    ) -> Result<ast::Crate, ParserError> {
-        let mut parser = ParserBuilder::default()
-            .config(config)
-            .input(input)
-            .directory_ownership(directory_ownership)
-            .sess(sess)
-            .build()?;
-
+    fn parse_crate_inner(input: Input, sess: &'a ParseSess) -> Result<ast::Crate, ParserError> {
+        let mut parser = ParserBuilder::default().input(input).sess(sess).build()?;
         parser.parse_crate_mod()
     }
 

Diff for: src/rustfmt/main.rs

@@ -662,6 +662,7 @@ mod test {
                 me.push("rustfmt");
                 assert!(
                     me.is_file() || me.with_extension("exe").is_file(),
+                    "{}",
                     if cfg!(release) {
                         "no rustfmt bin, try running `cargo build --release` before testing"
                     } else {

Diff for: src/test/mod.rs

@@ -136,10 +136,8 @@ fn verify_config_used(path: &Path, config_name: &str) {
                     .map(Result::unwrap)
                     .take_while(|l| l.starts_with("//"))
                     .any(|l| l.starts_with(&format!("// rustfmt-{}", config_name))),
-                format!(
-                    "config option file {} does not contain expected config name",
-                    path.display()
-                )
+                "config option file {} does not contain expected config name",
+                path.display(),
             );
         }
     }