Makes brace behavior consistent with empty bodies for traits and impls

Adds a newline before the last brace on impls without a body if the first brace is also on a newline
This commit is contained in:
André Luis Leal Cardoso Junior 2018-09-25 08:53:42 -03:00
parent ac6871f376
commit 2727d41ddb
3 changed files with 12 additions and 8 deletions

View file

@ -755,11 +755,10 @@ pub fn format_impl(
result.push_str(&inner_indent_str);
result.push_str(visitor.buffer.to_string().trim());
result.push_str(&outer_indent_str);
}
if result.ends_with('{') && !context.config.empty_item_single_line() {
} else if need_newline || !context.config.empty_item_single_line() {
result.push_str(&sep);
}
result.push('}');
Some(result)

View file

@ -78,4 +78,5 @@ where
S: event::Stream,
F: for<'t> FnMut(transform::Api<'t, Stream<ContentStream<S>>>) -> transform::Api<'t, X>,
X: event::Stream,
{}
{
}

View file

@ -134,11 +134,13 @@ mod m {
impl<BorrowType, K, V, NodeType, HandleType>
Handle<NodeRef<BorrowType, K, V, NodeType>, HandleType>
{}
{
}
impl<BorrowType, K, V, NodeType, HandleType> PartialEq
for Handle<NodeRef<BorrowType, K, V, NodeType>, HandleType>
{}
{
}
mod x {
impl<A, B, C, D> Foo
@ -147,7 +149,8 @@ mod x {
B: 'static,
C: 'static,
D: 'static,
{}
{
}
}
impl<ConcreteThreadSafeLayoutNode: ThreadSafeLayoutNodeFoo>
@ -229,4 +232,5 @@ impl<'seq1, 'seq2, 'body, 'scope, Channel>
>
where
Channel: DmaChannel,
{}
{
}