moved type_alias, added additional comments, altered unstable defines

Author: ricci009

library/core/src/ffi/mod.rs

@@ -37,30 +37,14 @@ pub use self::va_list::{VaList, VaListImpl};
 )]
 pub mod va_list;
 
-macro_rules! type_alias {
-    {
-      $Docfile:tt, $Alias:ident = $Real:ty;
-      $( $Cfg:tt )*
-    } => {
-        #[doc = include_str!($Docfile)]
-        $( $Cfg )*
-        #[stable(feature = "core_ffi_c", since = "1.64.0")]
-        pub type $Alias = $Real;
-    }
-}
-
 mod primitives;
-#[unstable(feature = "c_ptrdiff_t", issue = "88345")]
-pub use self::primitives::c_ptrdiff_t;
-#[unstable(feature = "c_size_t", issue = "88345")]
-pub use self::primitives::c_size_t;
-#[unstable(feature = "c_ssize_t", issue = "88345")]
-pub use self::primitives::c_ssize_t;
 #[stable(feature = "core_ffi_c", since = "1.64.0")]
 pub use self::primitives::{
     c_char, c_double, c_float, c_int, c_long, c_longlong, c_schar, c_short, c_uchar, c_uint,
     c_ulong, c_ulonglong, c_ushort,
 };
+#[unstable(feature = "c_size_t", issue = "88345")]
+pub use self::primitives::{c_ptrdiff_t, c_size_t, c_ssize_t};
 
 // N.B., for LLVM to recognize the void pointer type and by extension
 //     functions like malloc(), we need to have it represented as i8* in

library/core/src/ffi/primitives.rs

@@ -1,4 +1,19 @@
-//! Provides C-compatible primitive types to ensure correct type matching when interfacing with C code.
+//! Defines primitive types that match C's type definitions for FFI compatibility.
+//!
+//! This module is intentionally standalone to facilitate parsing when retrieving
+//! core C types.
+
+macro_rules! type_alias {
+    {
+      $Docfile:tt, $Alias:ident = $Real:ty;
+      $( $Cfg:tt )*
+    } => {
+        #[doc = include_str!($Docfile)]
+        $( $Cfg )*
+        #[stable(feature = "core_ffi_c", since = "1.64.0")]
+        pub type $Alias = $Real;
+    }
+}
 
 type_alias! { "c_char.md", c_char = c_char_definition::c_char; #[doc(cfg(all()))] }
 

src/tools/tidy/src/pal.rs

@@ -34,7 +34,7 @@ use std::path::Path;
 
 use crate::walk::{filter_dirs, walk};
 
-// Paths that may contain platform-specific code.
+// Paths that may contain platform-specific code.Is there any comment needed here for this?
 const EXCEPTION_PATHS: &[&str] = &[
     "library/windows_targets",
     "library/panic_abort",
@@ -48,7 +48,7 @@ const EXCEPTION_PATHS: &[&str] = &[
     // we must use `#[cfg(windows)]` to conditionally compile the
     // correct `VaList` structure for windows.
     "library/core/src/ffi/va_list.rs",
-    // We placed a linkage against Windows libraries here
+    // core::ffi contains platform-specific type and linkage configuration
     "library/core/src/ffi/mod.rs",
     "library/core/src/ffi/primitives.rs",
     "library/std/src/sys", // Platform-specific code for std lives here.