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

Pretty-printer is still adding extra whitespace in places #850

Closed
brson opened this issue Aug 19, 2011 · 5 comments
Closed

Pretty-printer is still adding extra whitespace in places #850

brson opened this issue Aug 19, 2011 · 5 comments
Labels
A-pretty Area: Pretty printing (including `-Z unpretty`)

Comments

@brson
Copy link
Contributor

brson commented Aug 19, 2011

Diffs from a recent reformat:

     item_obj(_obj, [ty_param], /* constructor id */node_id);
     item_res(_fn,
-              /* dtor */
+
+             /* dtor */
              node_id,
-              /* dtor id */
+
+             /* dtor id */
              [ty_param],
     noreturn; // functions with return type _|_ that always
               // raise an error or exit (i.e. never return to the caller)

+
     alt c {


+
       // One-byte tokens.
       '?' {
         rdr.bump();
@@ -400,6 +401,7 @@ fn next_token_inner(rdr: &reader) -> token::token {
       }


+
       // Multi-byte tokens.
       '=' {
@brson
Copy link
Contributor Author

brson commented Aug 21, 2011

At least one of these is the same issue as #751

@catamorphism
Copy link
Contributor

Reproduced with this test case:

type _obj = int;
type ty_param = int;
type node_id = int;
type _fn = int;

enum foo {
  item_obj(_obj, ~[ty_param], /* constructor id */node_id),
  item_res(_fn,
              /* dtor */
              node_id,
              /* dtor id */
              ~[ty_param]
           )
}

I suspect these are all the same issue as #751, as all of them involve extra blank lines getting added before comments.

@pnkfelix
Copy link
Member

linking to the metabug #3195 so that this ticket gets an entry (with status) in its comments area.

@flaper87
Copy link
Contributor

still reproducible. Triaged

@brson
Copy link
Contributor Author

brson commented Jan 13, 2015

Nobody is ever going to fix the current pretty-printer. Closing.

@brson brson closed this as completed Jan 13, 2015
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this issue Dec 12, 2017
celinval pushed a commit to celinval/rust-dev that referenced this issue Jun 4, 2024
* Moved atomic intrinsics tests, added description

* Add support for `atomic_nand*`

* Support for `atomic_singlethreadfence` and variants

* Add missing test for `atomic_cxchgweak`

* Add negative tests for min & max atomics

* Update doc with missing intrinsics
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-pretty Area: Pretty printing (including `-Z unpretty`)
Projects
None yet
Development

No branches or pull requests

4 participants