// 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::collections::HashMap;
-use std::fs::{self, File};
-use std::io::{self, Write, Read, stdout, 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 = HashMap<String, StringBuffer>;
+#[cfg(test)]
+use formatting::FileRecord;
// Append a newline to the end of each file.
-pub fn append_newlines(file_map: &mut FileMap) {
- for (_, s) in file_map.iter_mut() {
- s.push_str("\n");
- }
+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();
- for filename in file_map.keys() {
- try!(write_file(&file_map[filename], filename, out, config));
+ if config.emit_mode() == EmitMode::Checkstyle {
+ write!(out, "{}", ::checkstyle::header())?;
+ }
+ for &(ref filename, ref text) in file_map {
+ write_file(text, filename, out, config)?;
+ }
+ if config.emit_mode() == EmitMode::Checkstyle {
+ write!(out, "{}", ::checkstyle::footer())?;
}
- output_footer(out, config.write_mode).ok();
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' => try!(write!(writer, "\r\n")),
- '\r' => continue,
- c => try!(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!(),
- }
-}
-
-fn write_file<T>(text: &StringBuffer,
- filename: &str,
- out: &mut T,
- config: &Config)
- -> Result<Option<String>, io::Error>
- where T: Write
-{
-
- fn source_and_formatted_text(text: &StringBuffer,
- filename: &str,
- config: &Config)
- -> Result<(String, String), io::Error> {
- let mut f = try!(File::open(filename));
- let mut ori_text = String::new();
- try!(f.read_to_string(&mut ori_text));
- let mut v = Vec::new();
- try!(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) = try!(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 = try!(File::create(&tmp_name));
- try!(write_system_newlines(tmp_file, text, config));
- }
-
- try!(fs::rename(filename, bk_name));
- try!(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 = try!(File::create(filename));
- try!(write_system_newlines(file, text, config));
- }
- WriteMode::Plain => {
- let stdout = stdout();
- let stdout = stdout.lock();
- try!(write_system_newlines(stdout, text, config));
+ EmitMode::Stdout | EmitMode::Coverage => {
+ if config.verbose() != Verbosity::Quiet {
+ println!("{}:\n", filename);
+ }
+ write!(out, "{}", formatted_text)?;
}
- WriteMode::Display | WriteMode::Coverage => {
- println!("{}:\n", filename);
- let stdout = stdout();
- let stdout = stdout.lock();
- try!(write_system_newlines(stdout, 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);
}
- WriteMode::Diff => {
- println!("Diff of {}:\n", filename);
- if let Ok((ori, fmt)) = source_and_formatted_text(text, filename, config) {
- print_diff(make_diff(&ori, &fmt, 3),
- |line_num| format!("\nDiff at line {}:", line_num));
- }
+ 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)?;
}
- WriteMode::Checkstyle => {
- let diff = try!(create_diff(filename, text, config));
- try!(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);
}
}
- Ok(None)
+ // when we are not in diff mode, don't indicate differing files
+ Ok(false)
}
projects / rust.git / blobdiff