reorder_imports=false should still format the imports

[dtolnay]

Input file repro.rs:

use std :: fmt :: { self , Display } ;
use std :: collections :: HashMap ;

fn main() {}

Rustfmt invocation:

cargo run --bin rustfmt -- \
    path/to/repro.rs \
    --config-path <(echo reorder_imports=false) \
    --emit=stdout

Output as of current master branch, 34bf137:

use std :: fmt :: { self , Display } ;
use std :: collections :: HashMap ;

fn main() {}

Expected output, imports formatted but not reordered:

use std::fmt::{self, Display};
use std::collections::HashMap;

fn main() {}

I care about this for dtolnay/cargo-expand#56 in which I receive imports from a proc macro in TokenStream-style spaced out formatting and want to format them without reordering anything.

[dtolnay]

Note that reorder_modules=false seems to handle this correctly already. It turns:

mod b ;
mod a ;

into:

mod b;
mod a;

which is how I would expect.

[scampi]

@dtolnay the rustfmt output was the original one in your description.

use std::collections::HashMap;
use std::fmt::{self, Display};

fn main() {}

[dtolnay]

@scampi you may not have set reorder_imports=false, since what you show has imports reordered.

[scampi]

@dtolnay indeed... sorry ;oD