Rustdoc incorrectly fires redundant_explicit_link warning

[BD103]

We've recently begun testing Bevy against the beta toolchain, where it has incorrectly raised an rustdoc::redundant_explicit_links lint against this lint.

/// Determines how frequently an [`App`](bevy_app::App) should update.
///
/// **Note:** This setting is independent of VSync. VSync is controlled by a window's
/// [`PresentMode`](bevy_window::PresentMode) setting. If an app can update faster than the refresh
/// rate, but VSync is enabled, the update rate will be indirectly limited by the renderer.
#[derive(Debug, Clone, Copy)]
pub enum UpdateMode {
   // ...

bevy_app::App is not imported in this file, yet rustdoc believes that it is. (See the imports here.) As a temporary workaround, I've created bevyengine/bevy#12913 to ignore the error.

Here is the job that first failed. In case it expires, here's this archive. Look at 1_check-doc.txt.

I can reproduce it on this commit using rustc 1.78.0-beta.5 by running the following command:

$ cargo doc -p bevy_winit --no-deps

The --no-deps is optional, but it makes the command finish quicker. I'll continue testing to see if I can create a minimum reproducible example.

Meta

rustc --version --verbose:

rustc 1.78.0-beta.5 (9eff51035 2024-04-06)
binary: rustc
commit-hash: 9eff5103597e0ce73244028c96c75866dbcbad1d
commit-date: 2024-04-06
host: aarch64-apple-darwin
release: 1.78.0-beta.5
LLVM version: 18.1.2

[BD103]

I've created a minimal reproduction in my 123677-repro branch of my fork. Go to the bevy_winit folder and run cargo doc --no-deps.

I've managed to squish it down to this:

// lib.rs

mod winit_config;

pub use winit_config::*;

use bevy_app::App;

/// [`App::runner`]
/// 
/// [runner](bevy_app::App::runner)
pub fn winit_runner() {}

// winit_config.rs

/// Determines how frequently an [`App`](bevy_app::App) should update.
pub enum UpdateMode {}

[BD103]

I've created a reproduction that removes the dependency on Bevy here.

I've simplified it down to this:

mod bar {
    /// [`Rc`](std::rc::Rc)
    pub enum Baz {}
}

pub use bar::*;

use std::rc::Rc;

/// [`Rc::allocator`] [foo](std::rc::Rc)
pub fn winit_runner() {}

[BD103]

I bisected it, and this bug appears to first be introduced in beta-2024-03-19. I could not reproduce it in beta-2024-03-17.