Minor improvements

Cargo.toml

@@ -20,12 +20,12 @@ rustc-args = ["--cfg", "os_str_bytes_docs_rs"]
 rustdoc-args = ["--cfg", "os_str_bytes_docs_rs"]
 
 [dependencies]
-memchr = { version = "2.4", optional = true }
+memchr = { version = "2.3.5", optional = true }
 
 [dev-dependencies]
 fastrand = "2.0"
-lazy_static = "1.4"
-tempfile = "3.2"
+lazy_static = "1.0.3"
+tempfile = "3.8"
 
 [features]
 default = ["memchr", "raw_os_str"]

src/ext.rs

@@ -61,17 +61,17 @@ pub(super) fn check_bound(string: &OsStr, index: usize) {
 
 macro_rules! r#impl {
     ( $($name:ident),+ ) => {
-        $(
-            #[cfg(feature = "memchr")]
-            use memchr::memmem::$name;
-
-            #[cfg(not(feature = "memchr"))]
-            fn $name(string: &[u8], pat: &[u8]) -> Option<usize> {
-                (pat.len()..=string.len())
-                    .$name(|&x| string[..x].ends_with(pat))
-                    .map(|x| x - pat.len())
-            }
-        )+
+    $(
+        #[cfg(feature = "memchr")]
+        use memchr::memmem::$name;
+
+        #[cfg(not(feature = "memchr"))]
+        fn $name(string: &[u8], pat: &[u8]) -> Option<usize> {
+            (pat.len()..=string.len())
+                .$name(|&x| string[..x].ends_with(pat))
+                .map(|x| x - pat.len())
+        }
+    )+
     };
 }
 r#impl!(find, rfind);

src/iter.rs

@@ -19,8 +19,8 @@ use super::RawOsStr;
 
 macro_rules! r#impl {
     (
-        $(#[$attr:meta])* $name:ident ,
-        $(#[$raw_attr:meta])* $raw_name:ident ,
+        $(#[ $attr:meta ])* $name:ident ,
+        $(#[ $raw_attr:meta ])* $raw_name:ident ,
         $split_method:ident ,
         $reverse:expr ,
     ) => {

src/lib.rs

@@ -69,8 +69,8 @@
 //!   - [`RawOsString::from_raw_vec`]
 //!
 //!   Because this feature should not be used in libraries, the
-//!   "OS_STR_BYTES_CHECKED_CONVERSIONS" environment variable must be defined
-//!   during compilation.
+//!   "OS\_STR\_BYTES\_CHECKED\_CONVERSIONS" environment variable must be
+//!   defined during compilation.
 //!
 //! - **conversions** -
 //!   Provides methods that require encoding conversion and may be expensive:
@@ -213,10 +213,10 @@ use std::path::PathBuf;
 
 macro_rules! if_checked_conversions {
     ( $($item:item)+ ) => {
-        $(
-            #[cfg(feature = "checked_conversions")]
-            $item
-        )+
+    $(
+        #[cfg(feature = "checked_conversions")]
+        $item
+    )+
     };
 }
 
@@ -248,10 +248,10 @@ const _: &str = env!(
 
 macro_rules! if_conversions {
     ( $($item:item)+ ) => {
-        $(
-            #[cfg(feature = "conversions")]
-            $item
-        )+
+    $(
+        #[cfg(feature = "conversions")]
+        $item
+    )+
     };
 }
 
@@ -265,10 +265,10 @@ if_conversions! {
 
 macro_rules! if_raw_str {
     ( $($item:item)+ ) => {
-        $(
-            #[cfg(feature = "raw_os_str")]
-            $item
-        )+
+    $(
+        #[cfg(feature = "raw_os_str")]
+        $item
+    )+
     };
 }
 

tests/common.rs

@@ -2,10 +2,10 @@
 
 macro_rules! if_checked_conversions {
     ( $($item:item)+ ) => {
-        $(
-            #[cfg(feature = "checked_conversions")]
-            $item
-        )+
+    $(
+        #[cfg(feature = "checked_conversions")]
+        $item
+    )+
     };
 }