debug

Author: Kohei316

compiler/rustc_hir_typeck/src/method/suggest.rs

@@ -3291,7 +3291,7 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> {
         }
     }
 
-    #[instrument(level = "debug", skip(self))]
+    #[instrument(level = "debug", skip(self, handle_candidates))]
     fn suggest_use_candidates<F>(&self, candidates: Vec<DefId>, handle_candidates: F)
     where
         F: FnOnce(Vec<String>, Vec<String>, Span),

compiler/rustc_middle/src/ty/print/mod.rs

@@ -6,6 +6,7 @@ use rustc_data_structures::sso::SsoHashSet;
 use rustc_hir as hir;
 use rustc_hir::def_id::{CrateNum, DefId, LocalDefId};
 use rustc_hir::definitions::{DefPathData, DisambiguatedDefPathData};
+#[allow(unused_imports)]
 use tracing::{debug, instrument, trace};
 
 use crate::ty::{self, GenericArg, ShortInstance, Ty, TyCtxt};
@@ -111,14 +112,14 @@ pub trait Printer<'tcx>: Sized {
 
     // Defaults (should not be overridden):
 
-    #[instrument(skip(self), level = "debug")]
+    // #[instrument(skip(self), level = "debug")]
     fn default_print_def_path(
         &mut self,
         def_id: DefId,
         args: &'tcx [GenericArg<'tcx>],
     ) -> Result<(), PrintError> {
         let key = self.tcx().def_key(def_id);
-        debug!(?key);
+        // debug!(?key);
 
         match key.disambiguated_data.data {
             DefPathData::CrateRoot => {
@@ -207,10 +208,10 @@ pub trait Printer<'tcx>: Sized {
         self_ty: Ty<'tcx>,
         impl_trait_ref: Option<ty::TraitRef<'tcx>>,
     ) -> Result<(), PrintError> {
-        debug!(
-            "default_print_impl_path: impl_def_id={:?}, self_ty={}, impl_trait_ref={:?}",
-            impl_def_id, self_ty, impl_trait_ref
-        );
+        // debug!(
+        //     "default_print_impl_path: impl_def_id={:?}, self_ty={}, impl_trait_ref={:?}",
+        //     impl_def_id, self_ty, impl_trait_ref
+        // );
 
         let key = self.tcx().def_key(impl_def_id);
         let parent_def_id = DefId { index: key.parent.unwrap(), ..impl_def_id };

compiler/rustc_middle/src/ty/print/pretty.rs

@@ -370,6 +370,7 @@ pub trait PrettyPrinter<'tcx>: Printer<'tcx> + fmt::Write {
     /// from at least one local module, and returns `true`. If the crate defining `def_id` is
     /// declared with an `extern crate`, the path is guaranteed to use the `extern crate`.
     fn try_print_visible_def_path(&mut self, def_id: DefId) -> Result<bool, PrintError> {
+        debug!("try_print_visible_def_path: def_id={:?}", def_id);
         if with_no_visible_paths() {
             return Ok(false);
         }
@@ -488,7 +489,7 @@ pub trait PrettyPrinter<'tcx>: Printer<'tcx> + fmt::Write {
         def_id: DefId,
         callers: &mut Vec<DefId>,
     ) -> Result<bool, PrintError> {
-        debug!("try_print_visible_def_path: def_id={:?}", def_id);
+        // debug!("try_print_visible_def_path: def_id={:?}", def_id);
 
         // If `def_id` is a direct or injected extern crate, return the
         // path to the crate followed by the path to the item within the crate.
@@ -551,7 +552,7 @@ pub trait PrettyPrinter<'tcx>: Printer<'tcx> + fmt::Write {
         let visible_parent_map = self.tcx().visible_parent_map(());
 
         let mut cur_def_key = self.tcx().def_key(def_id);
-        debug!("try_print_visible_def_path: cur_def_key={:?}", cur_def_key);
+        // debug!("try_print_visible_def_path: cur_def_key={:?}", cur_def_key);
 
         // For a constructor, we want the name of its parent rather than <unnamed>.
         if let DefPathData::Ctor = cur_def_key.disambiguated_data.data {
@@ -566,20 +567,21 @@ pub trait PrettyPrinter<'tcx>: Printer<'tcx> + fmt::Write {
         }
 
         let Some(visible_parent) = visible_parent_map.get(&def_id).cloned() else {
+            debug!("try_print_visible_def_path: no visible parent for {:?}", def_id);
             return Ok(false);
         };
 
         let actual_parent = self.tcx().opt_parent(def_id);
-        debug!(
-            "try_print_visible_def_path: visible_parent={:?} actual_parent={:?}",
-            visible_parent, actual_parent,
-        );
+        // debug!(
+        //     "try_print_visible_def_path: visible_parent={:?} actual_parent={:?}",
+        //     visible_parent, actual_parent,
+        // );
 
         let mut data = cur_def_key.disambiguated_data.data;
-        debug!(
-            "try_print_visible_def_path: data={:?} visible_parent={:?} actual_parent={:?}",
-            data, visible_parent, actual_parent,
-        );
+        // debug!(
+        //     "try_print_visible_def_path: data={:?} visible_parent={:?} actual_parent={:?}",
+        //     data, visible_parent, actual_parent,
+        // );
 
         match data {
             // In order to output a path that could actually be imported (valid and visible),
@@ -616,6 +618,10 @@ pub trait PrettyPrinter<'tcx>: Printer<'tcx> + fmt::Write {
             DefPathData::TypeNs(Some(ref mut name)) if Some(visible_parent) != actual_parent => {
                 // Item might be re-exported several times, but filter for the one
                 // that's public and whose identifier isn't `_`.
+                debug!(
+                    "try_print_visible_def_path: def_id={:?}, visible_parent={:?}, actual_parent={:?}",
+                    def_id, visible_parent, actual_parent
+                );
                 let reexport = self
                     .tcx()
                     // FIXME(typed_def_id): Further propagate ModDefId
@@ -624,6 +630,7 @@ pub trait PrettyPrinter<'tcx>: Printer<'tcx> + fmt::Write {
                     .filter(|child| child.res.opt_def_id() == Some(def_id))
                     .find(|child| child.vis.is_public() && child.ident.name != kw::Underscore)
                     .map(|child| child.ident.name);
+                debug!("try_print_visible_def_path: reexport={:?}", reexport);
 
                 if let Some(new_name) = reexport {
                     *name = new_name;
@@ -638,7 +645,7 @@ pub trait PrettyPrinter<'tcx>: Printer<'tcx> + fmt::Write {
             }
             _ => {}
         }
-        debug!("try_print_visible_def_path: data={:?}", data);
+        // debug!("try_print_visible_def_path: data={:?}", data);
 
         if callers.contains(&visible_parent) {
             return Ok(false);
@@ -2185,7 +2192,7 @@ impl<'t> TyCtxt<'t> {
     ) -> String {
         let def_id = def_id.into_query_param();
         let ns = guess_def_namespace(self, def_id);
-        debug!("def_path_str: def_id={:?}, ns={:?}", def_id, ns);
+        // debug!("def_path_str: def_id={:?}, ns={:?}", def_id, ns);
 
         FmtPrinter::print_string(self, ns, |cx| cx.print_def_path(def_id, args)).unwrap()
     }

tests/ui/issues/auxiliary/issue-127011-a-2.rs

@@ -0,0 +1,5 @@
+//@ edition: 2021
+
+pub trait Foo {
+    fn foo();
+}

tests/ui/issues/auxiliary/issue-127011-a.rs

@@ -0,0 +1,5 @@
+//@ edition: 2021
+
+pub trait Foo {
+    fn foo();
+}

tests/ui/issues/auxiliary/issue-127011-b-2.rs

@@ -0,0 +1,10 @@
+//@ edition: 2021
+//@ aux-crate:issue_127011_a_2=issue-127011-a-2.rs
+
+use issue_127011_a_2::Foo;
+
+pub struct Bar;
+
+impl Foo for Bar {
+    fn foo() {}
+}

tests/ui/issues/auxiliary/issue-127011-b.rs

@@ -0,0 +1,10 @@
+//@ edition: 2021
+//@ aux-crate:issue_127011_a=issue-127011-a.rs
+
+pub use issue_127011_a::Foo;
+
+pub struct Bar;
+
+impl Foo for Bar {
+    fn foo() {}
+}

tests/ui/issues/issue-127011-2.rs

@@ -0,0 +1,10 @@
+//@ edition: 2021
+//@ aux-crate:issue_127011_a_2=issue-127011-a-2.rs
+//@ aux-crate:issue_127011_b_2=issue-127011-b-2.rs
+
+use issue_127011_b_2::Bar;
+
+fn main() {
+    Bar::foo();
+    //~^ ERROR: no function or associated item named `foo` found for struct `Bar` in the current scope [E0599]
+}

tests/ui/issues/issue-127011-2.stderr

@@ -0,0 +1,15 @@
+error[E0599]: no function or associated item named `foo` found for struct `Bar` in the current scope
+  --> $DIR/issue-127011-2.rs:8:10
+   |
+LL |     Bar::foo();
+   |          ^^^ function or associated item not found in `Bar`
+   |
+   = help: items from traits can only be used if the trait is in scope
+help: trait `Foo` which provides `foo` is implemented but not in scope; perhaps you want to import it
+   |
+LL + use issue_127011_a_2::Foo;
+   |
+
+error: aborting due to 1 previous error
+
+For more information about this error, try `rustc --explain E0599`.

tests/ui/issues/issue-127011.rs

@@ -0,0 +1,10 @@
+//@ edition: 2021
+//@ aux-crate:issue_127011_a=issue-127011-a.rs
+//@ aux-crate:issue_127011_b=issue-127011-b.rs
+
+use issue_127011_b::Bar;
+
+fn main() {
+    Bar::foo();
+    //~^ ERROR: no function or associated item named `foo` found for struct `Bar` in the current scope [E0599]
+}