Apply suggestions from code review

Co-authored-by: Trevor Gross <[email protected]>
rust-lang · Aug 21, 2024 · f74db2b · f74db2b
1 parent b087c94
commit f74db2b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/libc-test/build.rs b/libc-test/build.rs
@@ -3759,10 +3759,10 @@ fn test_linux(target: &str) {
         }
         if musl {
             // FIXME: Requires >= 5.0 kernel headers
-            if name == "SECCOMP_GET_NOTIF_SIZES"
+            if name == "PTP_SYS_OFFSET_EXTENDED"
+               || name == "SECCOMP_GET_NOTIF_SIZES"
                || name == "SECCOMP_FILTER_FLAG_NEW_LISTENER"
                || name == "SECCOMP_FILTER_FLAG_TSYNC_ESRCH"
-               || name == "PTP_SYS_OFFSET_EXTENDED"
                || name == "SECCOMP_USER_NOTIF_FLAG_CONTINUE"  // requires >= 5.5
                || name == "SECCOMP_ADDFD_FLAG_SETFD"  // requires >= 5.9
                || name == "SECCOMP_ADDFD_FLAG_SEND"   // requires >= 5.9
@@ -3776,12 +3776,12 @@ fn test_linux(target: &str) {
                 || name.starts_with("SO_J1939")
                 || name.starts_with("SCM_J1939")
                 || name == "PTP_CLOCK_GETCAPS2"
+                || name == "PTP_ENABLE_PPS2"
                 || name == "PTP_EXTTS_REQUEST2"
                 || name == "PTP_PEROUT_REQUEST2"
-                || name == "PTP_ENABLE_PPS2"
-                || name == "PTP_SYS_OFFSET2"
                 || name == "PTP_PIN_GETFUNC2"
                 || name == "PTP_PIN_SETFUNC2"
+                || name == "PTP_SYS_OFFSET2"
                 || name == "PTP_SYS_OFFSET_PRECISE2"
                 || name == "PTP_SYS_OFFSET_EXTENDED2"
             {

diff --git a/src/unix/linux_like/linux/mod.rs b/src/unix/linux_like/linux/mod.rs
@@ -3835,6 +3835,7 @@ cfg_if! {
     }
 }
 
+// enum ptp_pin_function
 pub const PTP_PF_NONE: ::c_uint = 0;
 pub const PTP_PF_EXTTS: ::c_uint = 1;
 pub const PTP_PF_PEROUT: ::c_uint = 2;