closures and block formatting very verbose

[nikomatsakis]

@ChristopherDavenport was helpfully applying rustfmt to rayon, using the rfc-rustfmt settings. The new fn signatures look great, but closures result in quite a lot of indentation:

#[test]
fn future_rayon_wait_1_thread() {
    // run with only 1 worker thread; this would deadlock if we couldn't make progress
    let mut result = None;
    ThreadPool::new(Configuration::new().num_threads(1))
        .unwrap()
        .install(
            || {
                scope(
                    |s| {
                        use std::sync::mpsc::channel;
                        let (tx, rx) = channel();
                        let a = s.spawn_future(lazy(move || Ok::<usize, ()>(rx.recv().unwrap())));
                        //                          ^^^^ FIXME: why is this needed?
                        let b = s.spawn_future(a.map(|v| v + 1));
                        let c = s.spawn_future(b.map(|v| v + 1));
                        s.spawn(move |_| tx.send(20).unwrap());
                        result = Some(c.rayon_wait().unwrap());
                    },
                );
            },
        );
    assert_eq!(result, Some(22));
}

[vorner]

I think I see the same annoyance. I have code like this:

vec.into_iter()
    .map(|elemente| {
        do_stuff(element)
    })
    .collect();

And rustfmt tries to put newlines there, like this:

vec.into_iter()
    .map(
        |elemente| {
            do_stuff(element)
        }
    )
    .collect();