]> git.lizzy.rs Git - rust.git/blobdiff - src/filemap.rs
Make children list in-order
[rust.git] / src / filemap.rs
index 27f5b9902014334f308215987758e90b15edd9ed..201403210328b31a87e4ced4cdc059001520fb24 100644 (file)
 // option. This file may not be copied, modified, or distributed
 // except according to those terms.
 
+use std::fs;
+use std::io::{self, Write};
 
-// TODO: add tests
+use checkstyle::output_checkstyle_file;
+use config::{Config, EmitMode, FileName, Verbosity};
+use rustfmt_diff::{make_diff, output_modified, print_diff};
 
-use strings::string_buffer::StringBuffer;
-
-use std::fs::{self, File};
-use std::io::{self, Write, Read, BufWriter};
-
-use config::{NewlineStyle, Config, WriteMode};
-use rustfmt_diff::{make_diff, print_diff, Mismatch};
-use checkstyle::{output_header, output_footer, output_checkstyle_file};
-
-// A map of the files of a crate, with their new content
-pub type FileMap = Vec<FileRecord>;
-
-pub type FileRecord = (String, StringBuffer);
+#[cfg(test)]
+use formatting::FileRecord;
 
 // Append a newline to the end of each file.
-pub fn append_newline(s: &mut StringBuffer) {
+pub fn append_newline(s: &mut String) {
     s.push_str("\n");
 }
 
-pub fn write_all_files<T>(file_map: &FileMap, out: &mut T, config: &Config) -> Result<(), io::Error>
-    where T: Write
+#[cfg(test)]
+pub(crate) fn write_all_files<T>(
+    file_map: &[FileRecord],
+    out: &mut T,
+    config: &Config,
+) -> Result<(), io::Error>
+where
+    T: Write,
 {
-    output_header(out, config.write_mode).ok();
+    if config.emit_mode() == EmitMode::Checkstyle {
+        write!(out, "{}", ::checkstyle::header())?;
+    }
     for &(ref filename, ref text) in file_map {
         write_file(text, filename, out, config)?;
     }
-    output_footer(out, config.write_mode).ok();
+    if config.emit_mode() == EmitMode::Checkstyle {
+        write!(out, "{}", ::checkstyle::footer())?;
+    }
 
     Ok(())
 }
 
-// Prints all newlines either as `\n` or as `\r\n`.
-pub fn write_system_newlines<T>(writer: T,
-                                text: &StringBuffer,
-                                config: &Config)
-                                -> Result<(), io::Error>
-    where T: Write
+pub fn write_file<T>(
+    formatted_text: &str,
+    filename: &FileName,
+    out: &mut T,
+    config: &Config,
+) -> Result<bool, io::Error>
+where
+    T: Write,
 {
-    // Buffer output, since we're writing a since char at a time.
-    let mut writer = BufWriter::new(writer);
-
-    let style = if config.newline_style == NewlineStyle::Native {
-        if cfg!(windows) {
-            NewlineStyle::Windows
-        } else {
-            NewlineStyle::Unix
-        }
-    } else {
-        config.newline_style
+    let filename_to_path = || match *filename {
+        FileName::Real(ref path) => path,
+        _ => panic!("cannot format `{}` and emit to files", filename),
     };
 
-    match style {
-        NewlineStyle::Unix => write!(writer, "{}", text),
-        NewlineStyle::Windows => {
-            for (c, _) in text.chars() {
-                match c {
-                    '\n' => write!(writer, "\r\n")?,
-                    '\r' => continue,
-                    c => write!(writer, "{}", c)?,
-                }
+    match config.emit_mode() {
+        EmitMode::Files if config.make_backup() => {
+            let filename = filename_to_path();
+            let ori = fs::read_to_string(filename)?;
+            if ori != formatted_text {
+                // Do a little dance to make writing safer - write to a temp file
+                // rename the original to a .bk, then rename the temp file to the
+                // original.
+                let tmp_name = filename.with_extension("tmp");
+                let bk_name = filename.with_extension("bk");
+
+                fs::write(&tmp_name, formatted_text)?;
+                fs::rename(filename, bk_name)?;
+                fs::rename(tmp_name, filename)?;
             }
-            Ok(())
         }
-        NewlineStyle::Native => unreachable!(),
-    }
-}
-
-pub fn write_file<T>(text: &StringBuffer,
-                     filename: &str,
-                     out: &mut T,
-                     config: &Config)
-                     -> Result<bool, io::Error>
-    where T: Write
-{
-
-    fn source_and_formatted_text(text: &StringBuffer,
-                                 filename: &str,
-                                 config: &Config)
-                                 -> Result<(String, String), io::Error> {
-        let mut f = File::open(filename)?;
-        let mut ori_text = String::new();
-        f.read_to_string(&mut ori_text)?;
-        let mut v = Vec::new();
-        write_system_newlines(&mut v, text, config)?;
-        let fmt_text = String::from_utf8(v).unwrap();
-        Ok((ori_text, fmt_text))
-    }
-
-    fn create_diff(filename: &str,
-                   text: &StringBuffer,
-                   config: &Config)
-                   -> Result<Vec<Mismatch>, io::Error> {
-        let (ori, fmt) = source_and_formatted_text(text, filename, config)?;
-        Ok(make_diff(&ori, &fmt, 3))
-    }
-
-    match config.write_mode {
-        WriteMode::Replace => {
-            if let Ok((ori, fmt)) = source_and_formatted_text(text, filename, config) {
-                if fmt != ori {
-                    // Do a little dance to make writing safer - write to a temp file
-                    // rename the original to a .bk, then rename the temp file to the
-                    // original.
-                    let tmp_name = filename.to_owned() + ".tmp";
-                    let bk_name = filename.to_owned() + ".bk";
-                    {
-                        // Write text to temp file
-                        let tmp_file = File::create(&tmp_name)?;
-                        write_system_newlines(tmp_file, text, config)?;
-                    }
-
-                    fs::rename(filename, bk_name)?;
-                    fs::rename(tmp_name, filename)?;
-                }
+        EmitMode::Files => {
+            // Write text directly over original file if there is a diff.
+            let filename = filename_to_path();
+            let ori = fs::read_to_string(filename)?;
+            if ori != formatted_text {
+                fs::write(filename, formatted_text)?;
             }
         }
-        WriteMode::Overwrite => {
-            // Write text directly over original file.
-            let file = File::create(filename)?;
-            write_system_newlines(file, text, config)?;
-        }
-        WriteMode::Plain => {
-            write_system_newlines(out, text, config)?;
-        }
-        WriteMode::Display | WriteMode::Coverage => {
-            println!("{}:\n", filename);
-            write_system_newlines(out, text, config)?;
-        }
-        WriteMode::Diff => {
-            if let Ok((ori, fmt)) = source_and_formatted_text(text, filename, config) {
-                let mismatch = make_diff(&ori, &fmt, 3);
-                let has_diff = !mismatch.is_empty();
-                print_diff(mismatch,
-                           |line_num| format!("Diff in {} at line {}:", filename, line_num));
-                return Ok(has_diff);
+        EmitMode::Stdout | EmitMode::Coverage => {
+            if config.verbose() != Verbosity::Quiet {
+                println!("{}:\n", filename);
             }
+            write!(out, "{}", formatted_text)?;
         }
-        WriteMode::Checkstyle => {
-            let diff = create_diff(filename, text, config)?;
+        EmitMode::ModifiedLines => {
+            let filename = filename_to_path();
+            let ori = fs::read_to_string(filename)?;
+            let mismatch = make_diff(&ori, formatted_text, 0);
+            let has_diff = !mismatch.is_empty();
+            output_modified(out, mismatch);
+            return Ok(has_diff);
+        }
+        EmitMode::Checkstyle => {
+            let filename = filename_to_path();
+            let ori = fs::read_to_string(filename)?;
+            let diff = make_diff(&ori, formatted_text, 3);
             output_checkstyle_file(out, filename, diff)?;
         }
+        EmitMode::Diff => {
+            let filename = filename_to_path();
+            let ori = fs::read_to_string(filename)?;
+            let mismatch = make_diff(&ori, formatted_text, 3);
+            let has_diff = !mismatch.is_empty();
+            print_diff(
+                mismatch,
+                |line_num| format!("Diff in {} at line {}:", filename.display(), line_num),
+                config,
+            );
+            return Ok(has_diff);
+        }
     }
 
     // when we are not in diff mode, don't indicate differing files