Skip to content
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

internal error: left behind trailing whitespace #2916

Closed
bjorn3 opened this issue Aug 14, 2018 · 3 comments · Fixed by #3768
Closed

internal error: left behind trailing whitespace #2916

bjorn3 opened this issue Aug 14, 2018 · 3 comments · Fixed by #3768
Labels
bug Panic, non-idempotency, invalid code, etc.

Comments

@bjorn3
Copy link
Member

bjorn3 commented Aug 14, 2018

Source: https://github.com/bjorn3/rustc_codegen_cranelift/tree/rustfmt_error

$ cargo fmt
internal error: left behind trailing whitespace
   --> /home/bjorn/Documenten/rustc_codegen_cranelift/src/lib.rs:416
    |
416 |                     
    | ^^^^^^^^^^^^^^^^^^^^

warning: rustfmt may have failed to format. See previous 1 errors.
@nrc nrc added the bug Panic, non-idempotency, invalid code, etc. label Aug 14, 2018
@minno726
Copy link

Same issue for me. The minimal reproduction I have is two lines:

a_macro!(name<Param1, Param2>, 
);

The error points to the trailing space on the first line. Removing one or both of the generic parameters removes the error.

@topecongiro
Copy link
Contributor

@bjorn3 This is likely due to the comment in the match arm. #1087 is a similar issue.

kszucs pushed a commit to apache/arrow that referenced this issue Oct 10, 2018
…nally

I also updated a lint issue in buffer.rs that was caught by nightly rustfmt.  It's the same issue that was addressed in #2658 so I don't know how it got reverted.

There may be other lint issues that I will address as CI flags them (I hope not) as locally I am running into [this issue](rust-lang/rustfmt#2916) after running `rustup update`

cc @kszucs @sunchao @andygrove

Author: Paddy Horan <[email protected]>

Closes #2700 from paddyhoran/ARROW-3398 and squashes the following commits:

4e05ff0 <Paddy Horan> Re-phrased old doc-comment
c1bff9e <Paddy Horan> Addressed review
c670d07 <Paddy Horan> Added tests for `write_bytes`
8dc5ed3 <Paddy Horan> Fixed lint issues
4cf5e37 <Paddy Horan> Updated `Builder` to use `MutableBuffer` internally
@theduke
Copy link

theduke commented Jan 22, 2019

I have run into the same issue.

Sadly it occurs all over the place for me with using a macro to build HTML.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Panic, non-idempotency, invalid code, etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants