Improve "rustup show" output order

Author: majaha

src/cli/rustup_mode.rs

@@ -1108,8 +1108,8 @@ fn show(cfg: &Cfg, m: &ArgMatches) -> Result<utils::ExitCode> {
     };
 
     let show_installed_toolchains = installed_toolchains.len() > 1;
-    let show_active_targets = active_targets.len() > 1;
     let show_active_toolchain = true;
+    let show_active_targets = active_targets.len() > 1;
 
     // Only need to display headers if we have multiple sections
     let show_headers = [
@@ -1150,27 +1150,6 @@ fn show(cfg: &Cfg, m: &ArgMatches) -> Result<utils::ExitCode> {
         };
     }
 
-    if show_active_targets {
-        let mut t = process().stdout().terminal();
-
-        if show_headers {
-            print_header::<Error>(&mut t, "installed targets for active toolchain")?;
-        }
-        for at in active_targets {
-            writeln!(
-                t.lock(),
-                "{}",
-                at.component
-                    .target
-                    .as_ref()
-                    .expect("rust-std should have a target")
-            )?;
-        }
-        if show_headers {
-            writeln!(t.lock())?;
-        };
-    }
-
     if show_active_toolchain {
         let mut t = process().stdout().terminal();
 
@@ -1208,6 +1187,27 @@ fn show(cfg: &Cfg, m: &ArgMatches) -> Result<utils::ExitCode> {
         }
     }
 
+    if show_active_targets {
+        let mut t = process().stdout().terminal();
+
+        if show_headers {
+            print_header::<Error>(&mut t, "installed targets for active toolchain")?;
+        }
+        for at in active_targets {
+            writeln!(
+                t.lock(),
+                "{}",
+                at.component
+                    .target
+                    .as_ref()
+                    .expect("rust-std should have a target")
+            )?;
+        }
+        if show_headers {
+            writeln!(t.lock())?;
+        };
+    }
+
     fn print_header<E>(t: &mut ColorableTerminal, s: &str) -> std::result::Result<(), E>
     where
         E: From<std::io::Error>,

tests/suite/cli_rustup.rs

@@ -709,18 +709,18 @@ fn show_multiple_targets() {
                     r"Default host: {2}
 rustup home:  {3}
 
-installed targets for active toolchain
---------------------------------------
-
-{1}
-{0}
-
 active toolchain
 ----------------
 
 nightly-{0} (default)
 1.3.0 (xxxx-nightly-2)
 
+installed targets for active toolchain
+--------------------------------------
+
+{1}
+{0}
+
 ",
                     clitools::MULTI_ARCH1,
                     clitools::CROSS_ARCH2,
@@ -764,18 +764,18 @@ installed toolchains
 stable-{0}
 nightly-{0} (default)
 
-installed targets for active toolchain
---------------------------------------
-
-{1}
-{0}
-
 active toolchain
 ----------------
 
 nightly-{0} (default)
 1.3.0 (xxxx-nightly-2)
 
+installed targets for active toolchain
+--------------------------------------
+
+{1}
+{0}
+
 ",
                     clitools::MULTI_ARCH1,
                     clitools::CROSS_ARCH2,