rustfmt_skip is ignored on expressions

[CraZySacX]

Before:

#[cfg_attr(rustfmt, rustfmt_skip)]
Box::new(self.inner.call(req).then(move |result| {
    match result {
       Ok(resp) => Box::new(future::done(Ok(resp))),
       Err(e) => {
           try_error!(clo_stderr, "{}", e);
           Box::new(future::err(e))
       }
   }
}))

Run cargo fmt

After:

#[cfg_attr(rustfmt, rustfmt_skip)]
Box::new(self.inner.call(req).then(move |result| match result {
                                       Ok(resp) => Box::new(future::done(Ok(resp))),
                                       Err(e) => {
    try_error!(clo_stderr, "{}", e);
    Box::new(future::err(e))
}
                                   }))

I would expect nothing to happen to the original structure.

Could you create a small working reproducer ?

[CraZySacX]

Will do.

Cool.

That being said, rustfmt_skip is not absolutely ignored, it usually works, but it seems there are still some cases which falls through the net :)

[nrc]

This is an attribute on an expression - it is an unstable Rust feature and may be deprecated. Rustfmt ignores these attributes. I'm not sure whether putting the attribute on a statement would work or not? Hopefully it does. I'm leaving this issue open for that.