Changes to rustfmt source
This commit is contained in:
parent
3a9e4f0540
commit
617eed353c
6 changed files with 62 additions and 15 deletions
|
|
@ -212,7 +212,10 @@ enum CodeCharKind {
|
|||
|
||||
impl<T> CharClasses<T> where T: Iterator, T::Item: RichChar {
|
||||
fn new(base: T) -> CharClasses<T> {
|
||||
CharClasses { base: base.peekable(), status: CharClassesStatus::Normal }
|
||||
CharClasses {
|
||||
base: base.peekable(),
|
||||
status: CharClassesStatus::Normal,
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -101,7 +101,10 @@ pub struct BadIssueSeeker {
|
|||
impl BadIssueSeeker {
|
||||
pub fn new(report_todo: ReportTactic, report_fixme: ReportTactic) -> BadIssueSeeker {
|
||||
BadIssueSeeker {
|
||||
state: Seeking::Issue { todo_idx: 0, fixme_idx: 0 },
|
||||
state: Seeking::Issue {
|
||||
todo_idx: 0,
|
||||
fixme_idx: 0,
|
||||
},
|
||||
report_todo: report_todo,
|
||||
report_fixme: report_fixme,
|
||||
}
|
||||
|
|
@ -121,7 +124,10 @@ impl BadIssueSeeker {
|
|||
return None;
|
||||
}
|
||||
|
||||
self.state = Seeking::Issue { todo_idx: 0, fixme_idx: 0 };
|
||||
self.state = Seeking::Issue {
|
||||
todo_idx: 0,
|
||||
fixme_idx: 0,
|
||||
};
|
||||
|
||||
if let IssueClassification::Bad(issue) = result {
|
||||
return Some(issue);
|
||||
|
|
@ -173,7 +179,10 @@ impl BadIssueSeeker {
|
|||
fixme_idx = 0;
|
||||
}
|
||||
|
||||
Seeking::Issue { todo_idx: todo_idx, fixme_idx: fixme_idx }
|
||||
Seeking::Issue {
|
||||
todo_idx: todo_idx,
|
||||
fixme_idx: fixme_idx,
|
||||
}
|
||||
}
|
||||
|
||||
fn inspect_number(&mut self,
|
||||
|
|
@ -214,7 +223,10 @@ impl BadIssueSeeker {
|
|||
NumberPart::CloseParen => {}
|
||||
}
|
||||
|
||||
self.state = Seeking::Number { part: part, issue: issue };
|
||||
self.state = Seeking::Number {
|
||||
part: part,
|
||||
issue: issue,
|
||||
};
|
||||
|
||||
IssueClassification::None
|
||||
}
|
||||
|
|
@ -274,7 +286,10 @@ fn find_issue() {
|
|||
#[test]
|
||||
fn issue_type() {
|
||||
let mut seeker = BadIssueSeeker::new(ReportTactic::Always, ReportTactic::Never);
|
||||
let expected = Some(Issue { issue_type: IssueType::Todo, missing_number: false });
|
||||
let expected = Some(Issue {
|
||||
issue_type: IssueType::Todo,
|
||||
missing_number: false,
|
||||
});
|
||||
|
||||
assert_eq!(expected,
|
||||
"TODO(#100): more awesomeness"
|
||||
|
|
@ -284,7 +299,10 @@ fn issue_type() {
|
|||
.unwrap());
|
||||
|
||||
let mut seeker = BadIssueSeeker::new(ReportTactic::Never, ReportTactic::Unnumbered);
|
||||
let expected = Some(Issue { issue_type: IssueType::Fixme, missing_number: true });
|
||||
let expected = Some(Issue {
|
||||
issue_type: IssueType::Fixme,
|
||||
missing_number: true,
|
||||
});
|
||||
|
||||
assert_eq!(expected,
|
||||
"Test. FIXME: bad, bad, not good"
|
||||
|
|
|
|||
25
src/lib.rs
25
src/lib.rs
|
|
@ -92,7 +92,10 @@ pub struct Indent {
|
|||
|
||||
impl Indent {
|
||||
pub fn new(block_indent: usize, alignment: usize) -> Indent {
|
||||
Indent { block_indent: block_indent, alignment: alignment }
|
||||
Indent {
|
||||
block_indent: block_indent,
|
||||
alignment: alignment,
|
||||
}
|
||||
}
|
||||
|
||||
pub fn empty() -> Indent {
|
||||
|
|
@ -304,7 +307,10 @@ pub fn fmt_lines(file_map: &mut FileMap, config: &Config) -> FormatReport {
|
|||
|
||||
// Add warnings for bad todos/ fixmes
|
||||
if let Some(issue) = issue_seeker.inspect(c) {
|
||||
errors.push(FormattingError { line: cur_line, kind: ErrorKind::BadIssue(issue) });
|
||||
errors.push(FormattingError {
|
||||
line: cur_line,
|
||||
kind: ErrorKind::BadIssue(issue),
|
||||
});
|
||||
}
|
||||
|
||||
if c == '\n' {
|
||||
|
|
@ -315,7 +321,10 @@ pub fn fmt_lines(file_map: &mut FileMap, config: &Config) -> FormatReport {
|
|||
}
|
||||
// Check for any line width errors we couldn't correct.
|
||||
if line_len > config.max_width {
|
||||
errors.push(FormattingError { line: cur_line, kind: ErrorKind::LineOverflow });
|
||||
errors.push(FormattingError {
|
||||
line: cur_line,
|
||||
kind: ErrorKind::LineOverflow,
|
||||
});
|
||||
}
|
||||
line_len = 0;
|
||||
cur_line += 1;
|
||||
|
|
@ -340,7 +349,10 @@ pub fn fmt_lines(file_map: &mut FileMap, config: &Config) -> FormatReport {
|
|||
}
|
||||
|
||||
for &(l, _, _) in &trims {
|
||||
errors.push(FormattingError { line: l, kind: ErrorKind::TrailingWhitespace });
|
||||
errors.push(FormattingError {
|
||||
line: l,
|
||||
kind: ErrorKind::TrailingWhitespace,
|
||||
});
|
||||
}
|
||||
|
||||
report.file_error_map.insert(f.to_owned(), errors);
|
||||
|
|
@ -395,7 +407,10 @@ pub fn format(args: Vec<String>, config: &Config) -> FileMap {
|
|||
|
||||
{
|
||||
let config = Rc::new(config.clone());
|
||||
let mut call_ctxt = RustFmtCalls { config: config, result: result.clone() };
|
||||
let mut call_ctxt = RustFmtCalls {
|
||||
config: config,
|
||||
result: result.clone(),
|
||||
};
|
||||
rustc_driver::run_compiler(&args, &mut call_ctxt);
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -107,7 +107,12 @@ impl ListItem {
|
|||
}
|
||||
|
||||
pub fn from_str<S: Into<String>>(s: S) -> ListItem {
|
||||
ListItem { pre_comment: None, item: s.into(), post_comment: None, new_lines: false }
|
||||
ListItem {
|
||||
pre_comment: None,
|
||||
item: s.into(),
|
||||
post_comment: None,
|
||||
new_lines: false,
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -15,7 +15,10 @@ pub struct Mismatch {
|
|||
|
||||
impl Mismatch {
|
||||
fn new(line_number: u32) -> Mismatch {
|
||||
Mismatch { line_number: line_number, lines: Vec::new() }
|
||||
Mismatch {
|
||||
line_number: line_number,
|
||||
lines: Vec::new(),
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -275,7 +275,10 @@ impl<'a> FmtVisitor<'a> {
|
|||
codemap: codemap,
|
||||
buffer: StringBuffer::new(),
|
||||
last_pos: BytePos(0),
|
||||
block_indent: Indent { block_indent: 0, alignment: 0 },
|
||||
block_indent: Indent {
|
||||
block_indent: 0,
|
||||
alignment: 0,
|
||||
},
|
||||
config: config,
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue