Skip to content

Wrong indentation with match for struct field #1456

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
leoschwarz opened this issue Apr 16, 2017 · 3 comments
Closed

Wrong indentation with match for struct field #1456

leoschwarz opened this issue Apr 16, 2017 · 3 comments

Comments

@leoschwarz
Copy link

leoschwarz commented Apr 16, 2017

I use rustfmt to keep my code tidy and found this error.

Here's a full minimal failing example contrasted with a non-failing example.

I'm not sure if this is a duplicate of #1371 ?

rustfmt 0.8.3 / b4833a8

@leoschwarz
Copy link
Author

leoschwarz commented Apr 16, 2017

Ok I just saw #1454 and my minimal example is fixed by it, but my actual code is not it ends up looking like this. (Which is a bit better than before but still wrong.)

From the diff it seems like what's essentially missing is indentation for the lines with the chained method (L40-L44).

@topecongiro
Copy link
Contributor

@leoschwarz Hi, I added a commit to the PR to fix this issue. I appreciate if you could confirm that nothing is going wrong with your code. Thanks!

@leoschwarz
Copy link
Author

@topecongiro Thanks, your new commit fixes all issues for me.

@nrc nrc closed this as completed May 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants