mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Cargo fmt

Browse Source
This commit is contained in:
Seiichi Uchida 2018-01-22 13:05:18 +09:00
parent fa6892eb63
commit 154bf8e1af
18 changed files with 98 additions and 101 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/bin/rustfmt.rs
View File

@ -453,7 +453,7 @@ fn determine_operation(matches: &Matches) -> FmtResult<Operation> {
return Ok(Operation::Stdin {
input: buffer,
config_path: config_path,
config_path,
});
}
@ -469,8 +469,8 @@ fn determine_operation(matches: &Matches) -> FmtResult<Operation> {
.collect();
Ok(Operation::Format {
files: files,
config_path: config_path,
minimal_config_path: minimal_config_path,
files,
config_path,
minimal_config_path,
})
}

2
src/closures.rs
View File

@ -244,7 +244,7 @@ fn rewrite_closure_fn_decl(
};
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: SeparatorTactic::Never,
separator_place: SeparatorPlace::Back,

8
src/comment.rs
View File

@ -285,11 +285,11 @@ fn rewrite_comment_inner(
let mut fmt = StringFormat {
opener: "",
closer: "",
line_start: line_start,
line_start,
line_end: "",
shape: Shape::legacy(max_chars, fmt_indent),
trim_end: true,
config: config,
config,
};
let line_breaks = count_newlines(orig.trim_right());
@ -895,7 +895,7 @@ pub struct CommentCodeSlices<'a> {
impl<'a> CommentCodeSlices<'a> {
pub fn new(slice: &'a str) -> CommentCodeSlices<'a> {
CommentCodeSlices {
slice: slice,
slice,
last_slice_kind: CodeCharKind::Comment,
last_slice_end: 0,
}
@ -1019,7 +1019,7 @@ impl<'a> CommentReducer<'a> {
let is_block = comment.starts_with("/*");
let comment = remove_comment_header(comment);
CommentReducer {
is_block: is_block,
is_block,
at_start_line: false, // There are no supplementary '*' on the first line
iter: comment.chars(),
}

44
src/expr.rs
View File

@ -441,7 +441,7 @@ pub fn rewrite_array<T: Rewrite + Spanned + ToExpr>(
let ends_with_newline = tactic.ends_with_newline(context.config.indent_style());
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: if trailing_comma {
SeparatorTactic::Always
@ -462,7 +462,7 @@ pub fn rewrite_array<T: Rewrite + Spanned + ToExpr>(
},
separator_place: SeparatorPlace::Back,
shape: nested_shape,
ends_with_newline: ends_with_newline,
ends_with_newline,
preserve_newline: false,
config: context.config,
};
@ -779,19 +779,19 @@ impl<'a> ControlFlow<'a> {
) -> ControlFlow<'a> {
ControlFlow {
cond: Some(cond),
block: block,
else_block: else_block,
block,
else_block,
label: None,
pat: pat,
pat,
keyword: "if",
matcher: match pat {
Some(..) => "let",
None => "",
},
connector: " =",
allow_single_line: allow_single_line,
nested_if: nested_if,
span: span,
allow_single_line,
nested_if,
span,
}
}
@ -802,16 +802,16 @@ impl<'a> ControlFlow<'a> {
) -> ControlFlow<'a> {
ControlFlow {
cond: None,
block: block,
block,
else_block: None,
label: label,
label,
pat: None,
keyword: "loop",
matcher: "",
connector: "",
allow_single_line: false,
nested_if: false,
span: span,
span,
}
}
@ -824,10 +824,10 @@ impl<'a> ControlFlow<'a> {
) -> ControlFlow<'a> {
ControlFlow {
cond: Some(cond),
block: block,
block,
else_block: None,
label: label,
pat: pat,
label,
pat,
keyword: "while",
matcher: match pat {
Some(..) => "let",
@ -836,7 +836,7 @@ impl<'a> ControlFlow<'a> {
connector: " =",
allow_single_line: false,
nested_if: false,
span: span,
span,
}
}
@ -849,16 +849,16 @@ impl<'a> ControlFlow<'a> {
) -> ControlFlow<'a> {
ControlFlow {
cond: Some(cond),
block: block,
block,
else_block: None,
label: label,
label,
pat: Some(pat),
keyword: "for",
matcher: "",
connector: " in",
allow_single_line: false,
nested_if: false,
span: span,
span,
}
}
@ -1484,7 +1484,7 @@ fn rewrite_match_pattern(
)
};
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: " |",
trailing_separator: SeparatorTactic::Never,
separator_place: context.config.binop_separator(),
@ -1988,7 +1988,7 @@ where
);
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: if force_trailing_comma {
SeparatorTactic::Always
@ -2671,11 +2671,11 @@ where
nested_shape.width,
);
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: SeparatorTactic::Never,
separator_place: SeparatorPlace::Back,
shape: shape,
shape,
ends_with_newline: false,
preserve_newline: false,
config: context.config,

2
src/file_lines.rs
View File

@ -34,7 +34,7 @@ impl<'a> From<&'a LineRange> for Range {
impl Range {
pub fn new(lo: usize, hi: usize) -> Range {
Range { lo: lo, hi: hi }
Range { lo, hi }
}
fn is_empty(self) -> bool {

6
src/imports.rs
View File

@ -284,7 +284,7 @@ fn rewrite_imports(
separator: "",
trailing_separator: SeparatorTactic::Never,
separator_place: SeparatorPlace::Back,
shape: shape,
shape,
ends_with_newline: true,
preserve_newline: false,
config: context.config,
@ -537,7 +537,7 @@ fn rewrite_nested_use_tree(
&& tactic != DefinitiveListTactic::Horizontal;
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: if ends_with_newline {
context.config.trailing_comma()
@ -546,7 +546,7 @@ fn rewrite_nested_use_tree(
},
separator_place: SeparatorPlace::Back,
shape: nested_shape,
ends_with_newline: ends_with_newline,
ends_with_newline,
preserve_newline: true,
config: context.config,
};

13
src/issues.rs
View File

@ -90,8 +90,8 @@ impl BadIssueSeeker {
todo_idx: 0,
fixme_idx: 0,
},
report_todo: report_todo,
report_fixme: report_fixme,
report_todo,
report_fixme,
}
}
@ -169,8 +169,8 @@ impl BadIssueSeeker {
}
Seeking::Issue {
todo_idx: todo_idx,
fixme_idx: fixme_idx,
todo_idx,
fixme_idx,
}
}
@ -213,10 +213,7 @@ impl BadIssueSeeker {
NumberPart::CloseParen => {}
}
self.state = Seeking::Number {
part: part,
issue: issue,
};
self.state = Seeking::Number { part, issue };
IssueClassification::None
}

46
src/items.rs
View File

@ -138,7 +138,7 @@ impl<'a> Item<'a> {
.iter()
.map(|i| BodyElement::ForeignItem(i))
.collect(),
span: span,
span,
}
}
}
@ -169,8 +169,8 @@ impl<'a> FnSig<'a> {
vis: ast::Visibility,
) -> FnSig<'a> {
FnSig {
decl: decl,
generics: generics,
decl,
generics,
abi: abi::Abi::Rust,
constness: ast::Constness::NotConst,
defaultness: ast::Defaultness::Final,
@ -189,7 +189,7 @@ impl<'a> FnSig<'a> {
defaultness: ast::Defaultness::Final,
abi: method_sig.abi,
decl: &*method_sig.decl,
generics: generics,
generics,
visibility: ast::Visibility::Inherited,
}
}
@ -202,12 +202,12 @@ impl<'a> FnSig<'a> {
) -> FnSig<'a> {
match *fn_kind {
visit::FnKind::ItemFn(_, unsafety, constness, abi, visibility, _) => FnSig {
decl: decl,
generics: generics,
abi: abi,
decl,
generics,
abi,
constness: constness.node,
defaultness: defualtness,
unsafety: unsafety,
unsafety,
visibility: visibility.clone(),
},
visit::FnKind::Method(_, method_sig, vis, _) => {
@ -510,7 +510,7 @@ impl<'a> FmtVisitor<'a> {
separator: ",",
trailing_separator: self.config.trailing_comma(),
separator_place: SeparatorPlace::Back,
shape: shape,
shape,
ends_with_newline: true,
preserve_newline: true,
config: self.config,
@ -895,10 +895,10 @@ impl<'a> StructParts<'a> {
_ => unreachable!(),
};
StructParts {
prefix: prefix,
prefix,
ident: item.ident,
vis: &item.vis,
def: def,
def,
generics: Some(generics),
span: item.span,
}
@ -1509,11 +1509,11 @@ impl<'a> StaticParts<'a> {
_ => unreachable!(),
};
StaticParts {
prefix: prefix,
prefix,
vis: &item.vis,
ident: item.ident,
ty: ty,
mutability: mutability,
ty,
mutability,
expr_opt: Some(expr),
defaultness: None,
span: item.span,
@ -1529,7 +1529,7 @@ impl<'a> StaticParts<'a> {
prefix: "const",
vis: &ast::Visibility::Inherited,
ident: ti.ident,
ty: ty,
ty,
mutability: ast::Mutability::Immutable,
expr_opt: expr_opt.as_ref(),
defaultness: None,
@ -1546,7 +1546,7 @@ impl<'a> StaticParts<'a> {
prefix: "const",
vis: &ii.vis,
ident: ii.ident,
ty: ty,
ty,
mutability: ast::Mutability::Immutable,
expr_opt: Some(expr),
defaultness: Some(ii.defaultness),
@ -1818,7 +1818,7 @@ fn rewrite_fn_base(
let one_line_budget = context.budget(used_width + overhead);
let shape = Shape {
width: one_line_budget,
indent: indent,
indent,
offset: used_width,
};
let fd = fn_sig.decl;
@ -2085,8 +2085,8 @@ struct WhereClauseOption {
impl WhereClauseOption {
pub fn new(suppress_comma: bool, snuggle: bool) -> WhereClauseOption {
WhereClauseOption {
suppress_comma: suppress_comma,
snuggle: snuggle,
suppress_comma,
snuggle,
compress_where: false,
}
}
@ -2233,7 +2233,7 @@ fn rewrite_args(
debug!("rewrite_args: budget: {}, tactic: {:?}", budget, tactic);
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: if variadic {
SeparatorTactic::Never
@ -2404,7 +2404,7 @@ where
one_line_budget,
);
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: if context.config.indent_style() == IndentStyle::Visual {
SeparatorTactic::Never
@ -2412,7 +2412,7 @@ where
context.config.trailing_comma()
},
separator_place: SeparatorPlace::Back,
shape: shape,
shape,
ends_with_newline: tactic.ends_with_newline(context.config.indent_style()),
preserve_newline: true,
config: context.config,
@ -2637,7 +2637,7 @@ fn rewrite_where_clause(
}
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: comma_tactic,
separator_place: SeparatorPlace::Back,

2
src/lib.rs
View File

@ -448,7 +448,7 @@ fn format_lines(
line: cur_line,
kind: error_kind,
is_comment: kind.is_comment(),
is_string: is_string,
is_string,
line_buffer: line_buffer.clone(),
});
}

32
src/lists.rs
View File

@ -690,15 +690,15 @@ where
};
ListItem {
pre_comment: pre_comment,
pre_comment_style: pre_comment_style,
pre_comment,
pre_comment_style,
item: if self.inner.peek().is_none() && self.leave_last {
None
} else {
(self.get_item_string)(&item)
},
post_comment: post_comment,
new_lines: new_lines,
post_comment,
new_lines,
}
})
}
@ -724,16 +724,16 @@ where
F3: Fn(&T) -> Option<String>,
{
ListItems {
codemap: codemap,
codemap,
inner: inner.peekable(),
get_lo: get_lo,
get_hi: get_hi,
get_item_string: get_item_string,
prev_span_end: prev_span_end,
next_span_start: next_span_start,
terminator: terminator,
separator: separator,
leave_last: leave_last,
get_lo,
get_hi,
get_item_string,
prev_span_end,
next_span_start,
terminator,
separator,
leave_last,
}
}
@ -841,7 +841,7 @@ pub fn struct_lit_formatting<'a>(
let ends_with_newline = context.config.indent_style() != IndentStyle::Visual
&& tactic == DefinitiveListTactic::Vertical;
ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: if force_no_trailing_comma {
SeparatorTactic::Never
@ -849,8 +849,8 @@ pub fn struct_lit_formatting<'a>(
context.config.trailing_comma()
},
separator_place: SeparatorPlace::Back,
shape: shape,
ends_with_newline: ends_with_newline,
shape,
ends_with_newline,
preserve_newline: true,
config: context.config,
}

2
src/rustfmt_diff.rs
View File

@ -31,7 +31,7 @@ pub struct Mismatch {
impl Mismatch {
fn new(line_number: u32) -> Mismatch {
Mismatch {
line_number: line_number,
line_number,
lines: Vec::new(),
}
}

16
src/shape.rs
View File

@ -29,8 +29,8 @@ const INDENT_BUFFER: &str =
impl Indent {
pub fn new(block_indent: usize, alignment: usize) -> Indent {
Indent {
block_indent: block_indent,
alignment: alignment,
block_indent,
alignment,
}
}
@ -161,8 +161,8 @@ impl Shape {
// |<--->| width
pub fn legacy(width: usize, indent: Indent) -> Shape {
Shape {
width: width,
indent: indent,
width,
indent,
offset: indent.alignment,
}
}
@ -170,7 +170,7 @@ impl Shape {
pub fn indented(indent: Indent, config: &Config) -> Shape {
Shape {
width: config.max_width().checked_sub(indent.width()).unwrap_or(0),
indent: indent,
indent,
offset: indent.alignment,
}
}
@ -187,9 +187,9 @@ impl Shape {
pub fn offset(width: usize, indent: Indent, offset: usize) -> Shape {
Shape {
width: width,
indent: indent,
offset: offset,
width,
indent,
offset,
}
}

4
src/string.rs
View File

@ -36,9 +36,9 @@ impl<'a> StringFormat<'a> {
closer: "\"",
line_start: " ",
line_end: "\\",
shape: shape,
shape,
trim_end: false,
config: config,
config,
}
}
}

2
src/types.rs
View File

@ -352,7 +352,7 @@ where
);
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: if !context.use_block_indent() || variadic {
SeparatorTactic::Never

2
src/vertical.rs
View File

@ -247,7 +247,7 @@ fn rewrite_aligned_items_inner<T: AlignedItem>(
);
let fmt = ListFormatting {
tactic: tactic,
tactic,
separator: ",",
trailing_separator: context.config.trailing_comma(),
separator_place: SeparatorPlace::Back,

6
src/visitor.rs
View File

@ -576,14 +576,14 @@ impl<'b, 'a: 'b> FmtVisitor<'a> {
snippet_provider: &'a SnippetProvider,
) -> FmtVisitor<'a> {
FmtVisitor {
parse_session: parse_session,
parse_session,
codemap: parse_session.codemap(),
buffer: String::with_capacity(snippet_provider.big_snippet.len() * 2),
last_pos: BytePos(0),
block_indent: Indent::empty(),
config: config,
config,
is_if_else_block: false,
snippet_provider: snippet_provider,
snippet_provider,
line_number: 0,
skipped_range: vec![],
}

2
tests/target/closure.rs
View File

@ -147,7 +147,7 @@ fn issue470() {
hair::PatternRef::Hair(pattern),
&lvalue,
);
ArgDecl { ty: ty }
ArgDecl { ty }
},
);
}

2
tests/target/impl.rs
View File

@ -13,7 +13,7 @@ where
pub fn new(value: V) -> Self {
Test {
cloned_value: value.clone(),
value: value,
value,
}
}
}