Remove last TODO comments.

diff --git a/src/cargo/core/compiler/context/unit_dependencies.rs b/src/cargo/core/compiler/context/unit_dependencies.rs
index b329de85aa973cadacec0fa4aa56a8ca93ad2f6b..75dbd24f1e69b0d59ec717d90194d34cf45b2d1c 100644 (file)
--- a/src/cargo/core/compiler/context/unit_dependencies.rs
+++ b/src/cargo/core/compiler/context/unit_dependencies.rs
@@ -172,8 +172,6 @@ fn compute_deps_custom_build<'a, 'cfg>(
         kind: unit.kind,
         mode: CompileMode::Build,
     };
-    // TODO: ProfileFor may need to be TestDependency if the random target we
-    // picked is a test and `panic` is set.  Need it investigate.
     let deps = deps_of(&tmp, cx, deps, ProfileFor::Any)?;
     Ok(deps.iter()
         .filter_map(|unit| {

diff --git a/src/cargo/core/compiler/job_queue.rs b/src/cargo/core/compiler/job_queue.rs
index b6202a90541c8460eb53c7054cc456c9f7574ad8..a713a6285cab66034255e137bd8e007e4c3fb275 100644 (file)
--- a/src/cargo/core/compiler/job_queue.rs
+++ b/src/cargo/core/compiler/job_queue.rs
@@ -256,7 +256,13 @@ impl<'a> JobQueue<'a> {
         }
 
         let build_type = if self.is_release { "release" } else { "dev" };
-        // TODO FIXME: We don't know which pkg to display this for!
+        // NOTE: This may be a bit inaccurate, since this may not display the
+        // profile for what was actually built.  Profile overrides can change
+        // these settings, and in some cases different targets are built with
+        // different profiles.  To be accurate, it would need to collect a
+        // list of Units built, and maybe display a list of the different
+        // profiles used.  However, to keep it simple and compatible with old
+        // behavior, we just display what the base profile is.
         let profile = cx.profiles.base_profile(self.is_release);
         let mut opt_type = String::from(if profile.opt_level.as_str() == "0" {
             "unoptimized"