Fix four_forward_slashes false positive on inner doc comments

The lint computes the item's end line from a span folded over the item's
doc-comment attributes, then scans the lines above for `////`. The fold
previously included inner doc comments (`//!`); an inner doc inside the body
lowers onto the item with `AttrStyle::Inner`, so folding its span pushed the
end line down into the body and the upward scan flagged a regular `////`
comment there — whose suggested fix (`///`) then fails to compile.

Fold in outer doc comments only, mirroring the pattern in
`doc/suspicious_doc_comments.rs`.

Author: zakrad

clippy_lints/src/four_forward_slashes.rs

@@ -1,8 +1,10 @@
 use clippy_utils::diagnostics::span_lint_and_then;
 use clippy_utils::source::SpanExt as _;
 use itertools::Itertools as _;
+use rustc_ast::AttrStyle;
 use rustc_errors::Applicability;
-use rustc_hir::Item;
+use rustc_hir::attrs::AttributeKind;
+use rustc_hir::{Attribute, Item};
 use rustc_lint::{LateContext, LateLintPass, LintContext as _};
 use rustc_session::declare_lint_pass;
 use rustc_span::Span;
@@ -48,7 +50,19 @@ impl<'tcx> LateLintPass<'tcx> for FourForwardSlashes {
             .tcx
             .hir_attrs(item.hir_id())
             .iter()
-            .filter(|i| i.is_doc_comment().is_some())
+            // Only fold in OUTER doc comments. An inner doc (`//!`) inside the item's
+            // body lowers onto the item with `AttrStyle::Inner`; folding its span drags
+            // `end_line` down into the body, so the upward `////` scan then flags a
+            // regular comment there (see #16168).
+            .filter(|i| {
+                matches!(
+                    i,
+                    Attribute::Parsed(AttributeKind::DocComment {
+                        style: AttrStyle::Outer,
+                        ..
+                    })
+                )
+            })
             .fold(item.span.shrink_to_lo(), |span, attr| span.to(attr.span()));
         let (Some(file), _, _, end_line, _) = sm.span_to_location_info(span) else {
             return;

tests/ui/four_forward_slashes.fixed

@@ -47,3 +47,10 @@ with_span! {
     //// don't lint me bozo
     fn f() {}
 }
+
+// Regression test for #16168: an inner doc comment (`//!`) inside the body must not
+// drag the `////` scan down into the body and flag a regular comment there.
+fn inner_doc() {
+    //// I am not a doc comment!
+    //! inner doc comment
+}

tests/ui/four_forward_slashes.rs

@@ -47,3 +47,10 @@ with_span! {
     //// don't lint me bozo
     fn f() {}
 }
+
+// Regression test for #16168: an inner doc comment (`//!`) inside the body must not
+// drag the `////` scan down into the body and flag a regular comment there.
+fn inner_doc() {
+    //// I am not a doc comment!
+    //! inner doc comment
+}