user0/rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

minor: use &mut ref instead of by value argument

Browse source 
This also unifies `write_all_files` and `write_file` functions
This commit is contained in:
Aleksey Kladov 2016-04-02 19:36:56 +03:00
parent c3cef52cde
commit 294b463b5d
2 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

8
src/filemap.rs
View file

@ -31,14 +31,14 @@ pub fn append_newlines(file_map: &mut FileMap) {
}
}
pub fn write_all_files<T>(file_map: &FileMap, mut out: T, config: &Config) -> Result<(), io::Error>
pub fn write_all_files<T>(file_map: &FileMap, out: &mut T, config: &Config) -> Result<(), io::Error>
where T: Write
{
output_header(&mut out, config.write_mode).ok();
output_header(out, config.write_mode).ok();
for filename in file_map.keys() {
try!(write_file(&file_map[filename], filename, &mut out, config));
try!(write_file(&file_map[filename], filename, out, config));
}
output_footer(&mut out, config.write_mode).ok();
output_footer(out, config.write_mode).ok();
Ok(())
}

4
src/lib.rs
View file

@ -432,8 +432,8 @@ pub fn run(file: &Path, config: &Config) {
let mut result = format(file, config);
print!("{}", fmt_lines(&mut result, config));
let out = stdout();
let write_result = filemap::write_all_files(&result, out, config);
let mut out = stdout();
let write_result = filemap::write_all_files(&result, &mut out, config);
if let Err(msg) = write_result {
println!("Error writing files: {}", msg);