adhere to conventions + fix feature check

Author: pepinns

src/socket.rs

@@ -884,41 +884,6 @@ impl Socket {
         }
     }
 
-    /// IP_TRANSPARENT (since Linux 2.6.24)
-    /// Setting this boolean option enables transparent proxying
-    /// on this socket.  This socket option allows the calling
-    /// application to bind to a nonlocal IP address and operate
-    /// both as a client and a server with the foreign address as
-    /// the local endpoint.  NOTE: this requires that routing be
-    /// set up in a way that packets going to the foreign address
-    /// are routed through the TProxy box (i.e., the system
-    /// hosting the application that employs the IP_TRANSPARENT
-    /// socket option).  Enabling this socket option requires
-    /// superuser privileges (the CAP_NET_ADMIN capability).
-    ///
-    /// TProxy redirection with the iptables TPROXY target also
-    /// requires that this option be set on the redirected socket.
-    /// this feature is only available on linux
-    #[cfg(any(target_os = "linux"))]
-    pub fn set_ip_transparent(&self, transparent: bool) -> io::Result<()> {
-        unsafe {
-            setsockopt(
-                self.inner,
-                sys::IPPROTO_IP,
-                libc::IP_TRANSPARENT,
-                transparent as c_int,
-            )
-        }
-    }
-    /// Get whether the IP_TRANSPARENT is set for this socket.
-    #[cfg(any(target_os = "linux"))]
-    pub fn ip_transparent(&self) -> io::Result<bool> {
-        unsafe {
-            getsockopt::<c_int>(self.inner, sys::IPPROTO_IP, libc::IP_TRANSPARENT)
-                .map(|transparent| transparent != 0)
-        }
-    }
-
     /// Get the value of the `SO_SNDBUF` option on this socket.
     ///
     /// For more information about this option, see [`set_send_buffer_size`].
@@ -990,6 +955,46 @@ fn into_linger(duration: Option<Duration>) -> sys::linger {
 /// * Linux: <https://man7.org/linux/man-pages/man7/ip.7.html>
 /// * Windows: <https://docs.microsoft.com/en-us/windows/win32/winsock/ipproto-ip-socket-options>
 impl Socket {
+    /// Get the value of the `IP_TRANSPARENTF` option on this socket.
+    ///
+    /// For more information about this option, see [`set_ip_transparent`].
+    ///
+    /// [`ip_transparent`]: Socket::set_ip_transparent
+    #[cfg(all(feature = "all", target_os = "linux"))]
+    pub fn ip_transparent(&self) -> io::Result<bool> {
+        unsafe {
+            getsockopt::<c_int>(self.inner, sys::IPPROTO_IP, libc::IP_TRANSPARENT)
+                .map(|transparent| transparent != 0)
+        }
+    }
+
+    /// IP_TRANSPARENT (since Linux 2.6.24)
+    /// Setting this boolean option enables transparent proxying
+    /// on this socket.  This socket option allows the calling
+    /// application to bind to a nonlocal IP address and operate
+    /// both as a client and a server with the foreign address as
+    /// the local endpoint.  NOTE: this requires that routing be
+    /// set up in a way that packets going to the foreign address
+    /// are routed through the TProxy box (i.e., the system
+    /// hosting the application that employs the IP_TRANSPARENT
+    /// socket option).  Enabling this socket option requires
+    /// superuser privileges (the CAP_NET_ADMIN capability).
+    ///
+    /// TProxy redirection with the iptables TPROXY target also
+    /// requires that this option be set on the redirected socket.
+    /// this feature is only available on linux
+    #[cfg(all(feature = "all", target_os = "linux"))]
+    pub fn set_ip_transparent(&self, transparent: bool) -> io::Result<()> {
+        unsafe {
+            setsockopt(
+                self.inner,
+                sys::IPPROTO_IP,
+                libc::IP_TRANSPARENT,
+                transparent as c_int,
+            )
+        }
+    }
+
     /// Join a multicast group using `IP_ADD_MEMBERSHIP` option on this socket.
     ///
     /// This function specifies a new multicast group for this socket to join.

tests/socket.rs

@@ -1047,7 +1047,7 @@ test!(
     mss,
     set_mss(256)
 );
-#[cfg(any(target_os = "linux"))]
+#[cfg(all(feature = "all", target_os = "linux"))]
 test!(
     #[ignore = "setting `IP_TRANSPARENT` requires the `CAP_NET_ADMIN` capability (works when running as root)"]
     ip_transparent,