Merge pull request #1181 from cgwalters/gstring-as-str

glib/GStringPtr: Add `as_str()` and `Deref<Target=&str>`
gtk-rs · Sep 25, 2023 · 9b61929 · 9b61929
2 parents 153420b + dc856e5
commit 9b61929
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 10 deletions.
diff --git a/glib/src/collections/strv.rs b/glib/src/collections/strv.rs
@@ -409,7 +409,7 @@ impl Clone for StrV {
         unsafe {
             let mut s = Self::with_capacity(self.len());
             for (i, item) in self.iter().enumerate() {
-                *s.ptr.as_ptr().add(i) = GString::from(item.to_str()).into_glib_ptr();
+                *s.ptr.as_ptr().add(i) = GString::from(item.as_str()).into_glib_ptr();
             }
             s.len = self.len();
             *s.ptr.as_ptr().add(s.len) = ptr::null_mut();

diff --git a/glib/src/gstring.rs b/glib/src/gstring.rs
@@ -694,10 +694,18 @@ impl GStringPtr {
     // rustdoc-stripper-ignore-next
     /// Returns the corresponding [`&str`].
     #[inline]
-    pub fn to_str(&self) -> &str {
+    pub fn as_str(&self) -> &str {
         self.to_gstr().as_str()
     }
 
+    // rustdoc-stripper-ignore-next
+    /// This is just an alias for [`as_str`].
+    #[inline]
+    #[deprecated = "Use as_str instead"]
+    pub fn to_str(&self) -> &str {
+        self
+    }
+
     // rustdoc-stripper-ignore-next
     /// Returns the string's C pointer.
     #[inline]
@@ -724,6 +732,15 @@ impl Clone for GStringPtr {
     }
 }
 
+impl Deref for GStringPtr {
+    type Target = str;
+
+    #[inline]
+    fn deref(&self) -> &str {
+        self.as_str()
+    }
+}
+
 impl IntoGlibPtr<*mut c_char> for GStringPtr {
     #[inline]
     unsafe fn into_glib_ptr(self) -> *mut c_char {
@@ -749,7 +766,7 @@ impl fmt::Debug for GStringPtr {
 impl fmt::Display for GStringPtr {
     #[inline]
     fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
-        f.write_str(self.to_str())
+        f.write_str(self.as_str())
     }
 }
 
@@ -863,7 +880,7 @@ impl Ord for GStringPtr {
 impl PartialOrd<GStringPtr> for String {
     #[inline]
     fn partial_cmp(&self, other: &GStringPtr) -> Option<std::cmp::Ordering> {
-        Some(self.as_str().cmp(other.to_str()))
+        Some(self.as_str().cmp(other))
     }
 }
 
@@ -877,7 +894,7 @@ impl PartialOrd<GStringPtr> for GString {
 impl PartialOrd<String> for GStringPtr {
     #[inline]
     fn partial_cmp(&self, other: &String) -> Option<std::cmp::Ordering> {
-        Some(self.to_str().cmp(other))
+        Some(self.as_str().cmp(other))
     }
 }
 
@@ -891,7 +908,7 @@ impl PartialOrd<GString> for GStringPtr {
 impl PartialOrd<GStringPtr> for str {
     #[inline]
     fn partial_cmp(&self, other: &GStringPtr) -> Option<std::cmp::Ordering> {
-        Some(self.cmp(other.to_str()))
+        Some(self.cmp(other.as_str()))
     }
 }
 
@@ -905,14 +922,14 @@ impl PartialOrd<GStringPtr> for GStr {
 impl PartialOrd<str> for GStringPtr {
     #[inline]
     fn partial_cmp(&self, other: &str) -> Option<std::cmp::Ordering> {
-        Some(self.to_str().cmp(other))
+        Some(self.as_str().cmp(other))
     }
 }
 
 impl PartialOrd<&str> for GStringPtr {
     #[inline]
     fn partial_cmp(&self, other: &&str) -> Option<std::cmp::Ordering> {
-        Some(self.to_str().cmp(other))
+        Some(self.as_str().cmp(other))
     }
 }
 
@@ -933,7 +950,7 @@ impl PartialOrd<&GStr> for GStringPtr {
 impl PartialOrd<GStringPtr> for &str {
     #[inline]
     fn partial_cmp(&self, other: &GStringPtr) -> Option<std::cmp::Ordering> {
-        Some(self.cmp(&other.to_str()))
+        Some(self.cmp(&other.as_str()))
     }
 }
 
@@ -954,7 +971,7 @@ impl AsRef<GStringPtr> for GStringPtr {
 impl std::hash::Hash for GStringPtr {
     #[inline]
     fn hash<H: std::hash::Hasher>(&self, state: &mut H) {
-        self.to_str().hash(state);
+        self.as_str().hash(state);
     }
 }