chore: add labels for FIXMEs in repo

lvllvl · lvllvl · commit 8cbc185ba625 · 2025-01-13T01:06:43.000Z
diff --git a/libc-test/build.rs b/libc-test/build.rs
@@ -255,7 +255,7 @@ fn test_apple(target: &str) {
         "os/clock.h",
         "os/lock.h",
         "os/signpost.h",
-        // FIXME: Requires the macOS 14.4 SDK.
+        // FIXME(macos): Requires the macOS 14.4 SDK.
         //"os/os_sync_wait_on_address.h",
         "poll.h",
         "pthread.h",
@@ -325,15 +325,15 @@ fn test_apple(target: &str) {
             return true;
         }
         match ty {
-            // FIXME: actually a union
+            // FIXME(union): actually a union
             "sigval" => true,
 
-            // FIXME: The size is changed in recent macOSes.
+            // FIXME(macos): The size is changed in recent macOSes.
             "malloc_zone_t" => true,
             // it is a moving target, changing through versions
             // also contains bitfields members
             "tcp_connection_info" => true,
-            // FIXME: The size is changed in recent macOSes.
+            // FIXME(macos): The size is changed in recent macOSes.
             "malloc_introspection_t" => true,
             // sonoma changes the padding `rmx_filler` field.
             "rt_metrics" => true,
@@ -347,10 +347,10 @@ fn test_apple(target: &str) {
             return true;
         }
         match ty {
-            // FIXME: Requires the macOS 14.4 SDK.
+            // FIXME(macos): Requires the macOS 14.4 SDK.
             "os_sync_wake_by_address_flags_t" | "os_sync_wait_on_address_flags_t" => true,
 
-            // FIXME: "'__uint128' undeclared" in C
+            // FIXME(macos): "'__uint128' undeclared" in C
             "__uint128" => true,
 
             _ => false,
@@ -362,13 +362,13 @@ fn test_apple(target: &str) {
             // These OSX constants are removed in Sierra.
             // https://developer.apple.com/library/content/releasenotes/General/APIDiffsMacOS10_12/Swift/Darwin.html
             "KERN_KDENABLE_BG_TRACE" | "KERN_KDDISABLE_BG_TRACE" => true,
-            // FIXME: the value has been changed since Catalina (0xffff0000 -> 0x3fff0000).
+            // FIXME(macos): the value has been changed since Catalina (0xffff0000 -> 0x3fff0000).
             "SF_SETTABLE" => true,
 
-            // FIXME: XCode 13.1 doesn't have it.
+            // FIXME(macos): XCode 13.1 doesn't have it.
             "TIOCREMOTE" => true,
 
-            // FIXME: Requires the macOS 14.4 SDK.
+            // FIXME(macos): Requires the macOS 14.4 SDK.
             "OS_SYNC_WAKE_BY_ADDRESS_NONE"
             | "OS_SYNC_WAKE_BY_ADDRESS_SHARED"
             | "OS_SYNC_WAIT_ON_ADDRESS_NONE"
@@ -384,19 +384,19 @@ fn test_apple(target: &str) {
             // close calls the close_nocancel system call
             "close" => true,
 
-            // FIXME: std removed libresolv support: https://github.com/rust-lang/rust/pull/102766
+            // FIXME(1.0): std removed libresolv support: https://github.com/rust-lang/rust/pull/102766
             "res_init" => true,
 
-            // FIXME: remove once the target in CI is updated
+            // FIXME(macos): remove once the target in CI is updated
             "pthread_jit_write_freeze_callbacks_np" => true,
 
-            // FIXME: ABI has been changed on recent macOSes.
+            // FIXME(macos): ABI has been changed on recent macOSes.
             "os_unfair_lock_assert_owner" | "os_unfair_lock_assert_not_owner" => true,
 
-            // FIXME: Once the SDK get updated to Ventura's level
+            // FIXME(macos): Once the SDK get updated to Ventura's level
             "freadlink" | "mknodat" | "mkfifoat" => true,
 
-            // FIXME: Requires the macOS 14.4 SDK.
+            // FIXME(macos): Requires the macOS 14.4 SDK.
             "os_sync_wake_by_address_any"
             | "os_sync_wake_by_address_all"
             | "os_sync_wake_by_address_flags_t"
@@ -411,7 +411,7 @@ fn test_apple(target: &str) {
 
     cfg.skip_field(move |struct_, field| {
         match (struct_, field) {
-            // FIXME: the array size has been changed since macOS 10.15 ([8] -> [7]).
+            // FIXME(macos): the array size has been changed since macOS 10.15 ([8] -> [7]).
             ("statfs", "f_reserved") => true,
             ("__darwin_arm_neon_state64", "__v") => true,
 
@@ -425,7 +425,7 @@ fn test_apple(target: &str) {
 
     cfg.skip_field_type(move |struct_, field| {
         match (struct_, field) {
-            // FIXME: actually a union
+            // FIXME(union): actually a union
             ("sigevent", "sigev_value") => true,
             _ => false,
         }
@@ -459,7 +459,7 @@ fn test_apple(target: &str) {
             s if s.ends_with("_nsec") && struct_.starts_with("stat") => {
                 s.replace("e_nsec", "espec.tv_nsec")
             }
-            // FIXME: sigaction actually contains a union with two variants:
+            // FIXME(macos): sigaction actually contains a union with two variants:
             // a sa_sigaction with type: (*)(int, struct __siginfo *, void *)
             // a sa_handler with type sig_t
             "sa_sigaction" if struct_ == "sigaction" => "sa_handler".to_string(),
@@ -468,7 +468,7 @@ fn test_apple(target: &str) {
     });
 
     cfg.skip_roundtrip(move |s| match s {
-        // FIXME: this type has the wrong ABI
+        // FIXME(macos): this type has the wrong ABI
         "max_align_t" if i686 => true,
         // Can't return an array from a C function.
         "uuid_t" | "vol_capabilities_set_t" => true,
@@ -575,7 +575,7 @@ fn test_openbsd(target: &str) {
             return true;
         }
         match ty {
-            // FIXME: actually a union
+            // FIXME(union): actually a union
             "sigval" => true,
 
             _ => false,
diff --git a/src/fuchsia/mod.rs b/src/fuchsia/mod.rs
@@ -91,7 +91,7 @@ pub type rlim_t = c_ulonglong;
 pub type c_long = i64;
 pub type c_ulong = u64;
 
-// FIXME: why are these uninhabited types? that seems... wrong?
+// FIXME(fuchsia): why are these uninhabited types? that seems... wrong?
 // Presumably these should be `()` or an `extern type` (when that stabilizes).
 #[cfg_attr(feature = "extra_traits", derive(Debug))]
 pub enum timezone {}
@@ -111,7 +111,7 @@ impl Clone for DIR {
 }
 
 #[cfg_attr(feature = "extra_traits", derive(Debug))]
-pub enum fpos64_t {} // FIXME: fill this out with a struct
+pub enum fpos64_t {} // FIXME(fuchsia): fill this out with a struct
 impl Copy for fpos64_t {}
 impl Clone for fpos64_t {
     fn clone(&self) -> fpos64_t {
@@ -144,7 +144,7 @@ s! {
         pub tv_nsec: c_long,
     }
 
-    // FIXME: the rlimit and rusage related functions and types don't exist
+    // FIXME(fuchsia): the rlimit and rusage related functions and types don't exist
     // within zircon. Are there reasons for keeping them around?
     pub struct rlimit {
         pub rlim_cur: rlim_t,
@@ -478,7 +478,7 @@ s! {
         pub ifa_flags: c_uint,
         pub ifa_addr: *mut crate::sockaddr,
         pub ifa_netmask: *mut crate::sockaddr,
-        pub ifa_ifu: *mut crate::sockaddr, // FIXME This should be a union
+        pub ifa_ifu: *mut crate::sockaddr, // FIXME(union) This should be a union
         pub ifa_data: *mut c_void,
     }
 
@@ -1097,7 +1097,7 @@ cfg_if! {
                     .field("totalhigh", &self.totalhigh)
                     .field("freehigh", &self.freehigh)
                     .field("mem_unit", &self.mem_unit)
-                    // FIXME: .field("__reserved", &self.__reserved)
+                    // FIXME(debug): .field("__reserved", &self.__reserved)
                     .finish()
             }
         }
@@ -1135,7 +1135,7 @@ cfg_if! {
             fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
                 f.debug_struct("sockaddr_un")
                     .field("sun_family", &self.sun_family)
-                    // FIXME: .field("sun_path", &self.sun_path)
+                    // FIXME(debug): .field("sun_path", &self.sun_path)
                     .finish()
             }
         }
@@ -1163,7 +1163,7 @@ cfg_if! {
                 f.debug_struct("sockaddr_storage")
                     .field("ss_family", &self.ss_family)
                     .field("__ss_align", &self.__ss_align)
-                    // FIXME: .field("__ss_pad2", &self.__ss_pad2)
+                    // FIXME(debug): .field("__ss_pad2", &self.__ss_pad2)
                     .finish()
             }
         }
@@ -1207,11 +1207,11 @@ cfg_if! {
         impl fmt::Debug for utsname {
             fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
                 f.debug_struct("utsname")
-                    // FIXME: .field("sysname", &self.sysname)
-                    // FIXME: .field("nodename", &self.nodename)
-                    // FIXME: .field("release", &self.release)
-                    // FIXME: .field("version", &self.version)
-                    // FIXME: .field("machine", &self.machine)
+                    // FIXME(debug): .field("sysname", &self.sysname)
+                    // FIXME(debug): .field("nodename", &self.nodename)
+                    // FIXME(debug): .field("release", &self.release)
+                    // FIXME(debug): .field("version", &self.version)
+                    // FIXME(debug): .field("machine", &self.machine)
                     .finish()
             }
         }
@@ -1246,7 +1246,7 @@ cfg_if! {
                     .field("d_off", &self.d_off)
                     .field("d_reclen", &self.d_reclen)
                     .field("d_type", &self.d_type)
-                    // FIXME: .field("d_name", &self.d_name)
+                    // FIXME(debug): .field("d_name", &self.d_name)
                     .finish()
             }
         }
@@ -1281,7 +1281,7 @@ cfg_if! {
                     .field("d_off", &self.d_off)
                     .field("d_reclen", &self.d_reclen)
                     .field("d_type", &self.d_type)
-                    // FIXME: .field("d_name", &self.d_name)
+                    // FIXME(debug): .field("d_name", &self.d_name)
                     .finish()
             }
         }
@@ -1390,7 +1390,7 @@ cfg_if! {
         impl fmt::Debug for pthread_cond_t {
             fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
                 f.debug_struct("pthread_cond_t")
-                    // FIXME: .field("size", &self.size)
+                    // FIXME(debug): .field("size", &self.size)
                     .finish()
             }
         }
@@ -1409,7 +1409,7 @@ cfg_if! {
         impl fmt::Debug for pthread_mutex_t {
             fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
                 f.debug_struct("pthread_mutex_t")
-                    // FIXME: .field("size", &self.size)
+                    // FIXME(debug): .field("size", &self.size)
                     .finish()
             }
         }
@@ -1428,7 +1428,7 @@ cfg_if! {
         impl fmt::Debug for pthread_rwlock_t {
             fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
                 f.debug_struct("pthread_rwlock_t")
-                    // FIXME: .field("size", &self.size)
+                    // FIXME(debug): .field("size", &self.size)
                     .finish()
             }
         }
@@ -3562,7 +3562,7 @@ impl Clone for FILE {
     }
 }
 #[cfg_attr(feature = "extra_traits", derive(Debug))]
-pub enum fpos_t {} // FIXME: fill this out with a struct
+pub enum fpos_t {} // FIXME(fuchsia): fill this out with a struct
 impl Copy for fpos_t {}
 impl Clone for fpos_t {
     fn clone(&self) -> fpos_t {
diff --git a/src/lib.rs b/src/lib.rs
@@ -12,7 +12,7 @@
     redundant_semicolons,
     unused_macros,
     unused_macro_rules,
-    // FIXME: temporarily allow dead_code to fix CI:
+    // FIXME(1.0): temporarily allow dead_code to fix CI:
     // - https://github.com/rust-lang/libc/issues/3740
     // - https://github.com/rust-lang/rust/pull/126456
     dead_code,
diff --git a/src/vxworks/mod.rs b/src/vxworks/mod.rs
@@ -570,7 +570,7 @@ pub const EAI_SERVICE: c_int = 9;
 pub const EAI_SOCKTYPE: c_int = 10;
 pub const EAI_SYSTEM: c_int = 11;
 
-// FIXME: This is not defined in vxWorks, but we have to define it here
+// FIXME(vxworks): This is not defined in vxWorks, but we have to define it here
 // to make the building pass for getrandom and std
 pub const RTLD_DEFAULT: *mut c_void = 0i64 as *mut c_void;
 
@@ -733,7 +733,7 @@ pub const S_taskLib_TASK_HOOK_TABLE_FULL: c_int = taskErrorBase + 0x0066;
 pub const S_taskLib_TASK_HOOK_NOT_FOUND: c_int = taskErrorBase + 0x0067;
 pub const S_taskLib_ILLEGAL_PRIORITY: c_int = taskErrorBase + 0x0068;
 
-// FIXME: could also be useful for TASK_DESC type
+// FIXME(vxworks): could also be useful for TASK_DESC type
 pub const VX_TASK_NAME_LENGTH: c_int = 31;
 
 // semLibCommon.h
@@ -1077,7 +1077,7 @@ impl Clone for FILE {
     }
 }
 #[cfg_attr(feature = "extra_traits", derive(Debug))]
-pub enum fpos_t {} // FIXME: fill this out with a struct
+pub enum fpos_t {} // FIXME(vxworks): fill this out with a struct
 impl Copy for fpos_t {}
 impl Clone for fpos_t {
     fn clone(&self) -> fpos_t {

Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ pub type rlim_t = c_ulonglong;`
`91`	`91`	`pub type c_long = i64;`
`92`	`92`	`pub type c_ulong = u64;`
`93`	`93`
`94`		`-// FIXME: why are these uninhabited types? that seems... wrong?`
	`94`	`+// FIXME(fuchsia): why are these uninhabited types? that seems... wrong?`
`95`	`95`	// Presumably these should be `()` or an `extern type` (when that stabilizes).
`96`	`96`	`#[cfg_attr(feature = "extra_traits", derive(Debug))]`
`97`	`97`	`pub enum timezone {}`
`@@ -111,7 +111,7 @@ impl Clone for DIR {`
`111`	`111`	`}`
`112`	`112`
`113`	`113`	`#[cfg_attr(feature = "extra_traits", derive(Debug))]`
`114`		`-pub enum fpos64_t {} // FIXME: fill this out with a struct`
	`114`	`+pub enum fpos64_t {} // FIXME(fuchsia): fill this out with a struct`
`115`	`115`	`impl Copy for fpos64_t {}`
`116`	`116`	`impl Clone for fpos64_t {`
`117`	`117`	`fn clone(&self) -> fpos64_t {`
`@@ -144,7 +144,7 @@ s! {`
`144`	`144`	`pub tv_nsec: c_long,`
`145`	`145`	`}`
`146`	`146`
`147`		`- // FIXME: the rlimit and rusage related functions and types don't exist`
	`147`	`+ // FIXME(fuchsia): the rlimit and rusage related functions and types don't exist`
`148`	`148`	`// within zircon. Are there reasons for keeping them around?`
`149`	`149`	`pub struct rlimit {`
`150`	`150`	`pub rlim_cur: rlim_t,`
`@@ -478,7 +478,7 @@ s! {`
`478`	`478`	`pub ifa_flags: c_uint,`
`479`	`479`	`pub ifa_addr: *mut crate::sockaddr,`
`480`	`480`	`pub ifa_netmask: *mut crate::sockaddr,`
`481`		`- pub ifa_ifu: *mut crate::sockaddr, // FIXME This should be a union`
	`481`	`+ pub ifa_ifu: *mut crate::sockaddr, // FIXME(union) This should be a union`
`482`	`482`	`pub ifa_data: *mut c_void,`
`483`	`483`	`}`
`484`	`484`
`@@ -1097,7 +1097,7 @@ cfg_if! {`
`1097`	`1097`	`.field("totalhigh", &self.totalhigh)`
`1098`	`1098`	`.field("freehigh", &self.freehigh)`
`1099`	`1099`	`.field("mem_unit", &self.mem_unit)`
`1100`		`- // FIXME: .field("__reserved", &self.__reserved)`
	`1100`	`+ // FIXME(debug): .field("__reserved", &self.__reserved)`
`1101`	`1101`	`.finish()`
`1102`	`1102`	`}`
`1103`	`1103`	`}`
`@@ -1135,7 +1135,7 @@ cfg_if! {`
`1135`	`1135`	`fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {`
`1136`	`1136`	`f.debug_struct("sockaddr_un")`
`1137`	`1137`	`.field("sun_family", &self.sun_family)`
`1138`		`- // FIXME: .field("sun_path", &self.sun_path)`
	`1138`	`+ // FIXME(debug): .field("sun_path", &self.sun_path)`
`1139`	`1139`	`.finish()`
`1140`	`1140`	`}`
`1141`	`1141`	`}`
`@@ -1163,7 +1163,7 @@ cfg_if! {`
`1163`	`1163`	`f.debug_struct("sockaddr_storage")`
`1164`	`1164`	`.field("ss_family", &self.ss_family)`
`1165`	`1165`	`.field("__ss_align", &self.__ss_align)`
`1166`		`- // FIXME: .field("__ss_pad2", &self.__ss_pad2)`
	`1166`	`+ // FIXME(debug): .field("__ss_pad2", &self.__ss_pad2)`
`1167`	`1167`	`.finish()`
`1168`	`1168`	`}`
`1169`	`1169`	`}`
`@@ -1207,11 +1207,11 @@ cfg_if! {`
`1207`	`1207`	`impl fmt::Debug for utsname {`
`1208`	`1208`	`fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {`
`1209`	`1209`	`f.debug_struct("utsname")`
`1210`		`- // FIXME: .field("sysname", &self.sysname)`
`1211`		`- // FIXME: .field("nodename", &self.nodename)`
`1212`		`- // FIXME: .field("release", &self.release)`
`1213`		`- // FIXME: .field("version", &self.version)`
`1214`		`- // FIXME: .field("machine", &self.machine)`
	`1210`	`+ // FIXME(debug): .field("sysname", &self.sysname)`
	`1211`	`+ // FIXME(debug): .field("nodename", &self.nodename)`
	`1212`	`+ // FIXME(debug): .field("release", &self.release)`
	`1213`	`+ // FIXME(debug): .field("version", &self.version)`
	`1214`	`+ // FIXME(debug): .field("machine", &self.machine)`
`1215`	`1215`	`.finish()`
`1216`	`1216`	`}`
`1217`	`1217`	`}`
`@@ -1246,7 +1246,7 @@ cfg_if! {`
`1246`	`1246`	`.field("d_off", &self.d_off)`
`1247`	`1247`	`.field("d_reclen", &self.d_reclen)`
`1248`	`1248`	`.field("d_type", &self.d_type)`
`1249`		`- // FIXME: .field("d_name", &self.d_name)`
	`1249`	`+ // FIXME(debug): .field("d_name", &self.d_name)`
`1250`	`1250`	`.finish()`
`1251`	`1251`	`}`
`1252`	`1252`	`}`
`@@ -1281,7 +1281,7 @@ cfg_if! {`
`1281`	`1281`	`.field("d_off", &self.d_off)`
`1282`	`1282`	`.field("d_reclen", &self.d_reclen)`
`1283`	`1283`	`.field("d_type", &self.d_type)`
`1284`		`- // FIXME: .field("d_name", &self.d_name)`
	`1284`	`+ // FIXME(debug): .field("d_name", &self.d_name)`
`1285`	`1285`	`.finish()`
`1286`	`1286`	`}`
`1287`	`1287`	`}`
`@@ -1390,7 +1390,7 @@ cfg_if! {`
`1390`	`1390`	`impl fmt::Debug for pthread_cond_t {`
`1391`	`1391`	`fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {`
`1392`	`1392`	`f.debug_struct("pthread_cond_t")`
`1393`		`- // FIXME: .field("size", &self.size)`
	`1393`	`+ // FIXME(debug): .field("size", &self.size)`
`1394`	`1394`	`.finish()`
`1395`	`1395`	`}`
`1396`	`1396`	`}`
`@@ -1409,7 +1409,7 @@ cfg_if! {`
`1409`	`1409`	`impl fmt::Debug for pthread_mutex_t {`
`1410`	`1410`	`fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {`
`1411`	`1411`	`f.debug_struct("pthread_mutex_t")`
`1412`		`- // FIXME: .field("size", &self.size)`
	`1412`	`+ // FIXME(debug): .field("size", &self.size)`
`1413`	`1413`	`.finish()`
`1414`	`1414`	`}`
`1415`	`1415`	`}`
`@@ -1428,7 +1428,7 @@ cfg_if! {`
`1428`	`1428`	`impl fmt::Debug for pthread_rwlock_t {`
`1429`	`1429`	`fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {`
`1430`	`1430`	`f.debug_struct("pthread_rwlock_t")`
`1431`		`- // FIXME: .field("size", &self.size)`
	`1431`	`+ // FIXME(debug): .field("size", &self.size)`
`1432`	`1432`	`.finish()`
`1433`	`1433`	`}`
`1434`	`1434`	`}`
`@@ -3562,7 +3562,7 @@ impl Clone for FILE {`
`3562`	`3562`	`}`
`3563`	`3563`	`}`
`3564`	`3564`	`#[cfg_attr(feature = "extra_traits", derive(Debug))]`
`3565`		`-pub enum fpos_t {} // FIXME: fill this out with a struct`
	`3565`	`+pub enum fpos_t {} // FIXME(fuchsia): fill this out with a struct`
`3566`	`3566`	`impl Copy for fpos_t {}`
`3567`	`3567`	`impl Clone for fpos_t {`
`3568`	`3568`	`fn clone(&self) -> fpos_t {`