Clean up Windows comments

Author: workingjubilee

src/cli/job.rs

@@ -1,6 +1,6 @@
 // FIXME: stolen from cargo. Should be extracted into a common crate.
 
-//! Job management (mostly for windows)
+//! Job management (mostly for Windows)
 //!
 //! Most of the time when you're running cargo you expect Ctrl-C to actually
 //! terminate the entire tree of processes in play, not just the one at the top

src/cli/self_update/unix.rs

@@ -129,13 +129,7 @@ pub fn do_remove_from_programs() -> Result<()> {
 }
 
 /// Tell the upgrader to replace the rustup bins, then delete
-/// itself. Like with uninstallation, on Windows we're going to
-/// have to jump through hoops to make everything work right.
-///
-/// On windows we're not going to wait for it to finish before exiting
-/// successfully, so it should not do much, and it should try
-/// really hard to succeed, because at this point the upgrade is
-/// considered successful.
+/// itself.
 pub fn run_update(setup_path: &Path) -> Result<utils::ExitCode> {
     let status = Command::new(setup_path)
         .arg("--self-replace")
@@ -151,8 +145,7 @@ pub fn run_update(setup_path: &Path) -> Result<utils::ExitCode> {
 
 /// This function is as the final step of a self-upgrade. It replaces
 /// `CARGO_HOME`/bin/rustup with the running exe, and updates the the
-/// links to it. On windows this will run *after* the original
-/// rustup process exits.
+/// links to it.
 pub fn self_replace() -> Result<utils::ExitCode> {
     install_bins()?;
 
@@ -181,7 +174,7 @@ fn remove_legacy_source_command(source_cmd: String) -> Result<()> {
 
 fn remove_legacy_paths() -> Result<()> {
     // Before the work to support more kinds of shells, which was released in
-    // version 1.23.0 of rustup, we always inserted this line instead, which is
+    // version 1.23.0 of Rustup, we always inserted this line instead, which is
     // now considered legacy
     remove_legacy_source_command(format!(
         "export PATH=\"{}/bin:$PATH\"\n",

src/cli/self_update/windows.rs

@@ -391,7 +391,7 @@ pub fn self_replace() -> Result<utils::ExitCode> {
 //   This file is opened as inheritable so that subsequent
 //   processes created with the option to inherit handles
 //   will also keep them open.
-// - Run the gc exe, which waits for the original rustup
+// - Run the gc exe, which waits for the original rustup.exe
 //   process to close, then deletes CARGO_HOME. This process
 //   has inherited a FILE_FLAG_DELETE_ON_CLOSE handle to itself.
 // - Finally, spawn yet another system binary with the inherit handles