Use a superpub field in MapsEntry

This may eliminate some of the overhead of a getter.
Also use a slightly more modern code style.

Author: workingjubilee

src/symbolize/gimli/libs_dl_iterate_phdr.rs

@@ -36,10 +36,9 @@ fn infer_current_exe(
     #[cfg(not(target_os = "hurd"))]
     if let Some(entries) = maps {
         let opt_path = entries
-            .iter()
-            .find(|e| e.ip_matches(base_addr) && e.pathname().len() > 0)
-            .map(|e| e.pathname())
-            .cloned();
+            .into_iter()
+            .find(|e| e.ip_matches(base_addr) && !e.pathname.is_empty())
+            .map(|e| e.pathname);
         if let Some(path) = opt_path {
             return path;
         }

src/symbolize/gimli/parse_running_mmaps_unix.rs

@@ -52,7 +52,7 @@ pub(super) struct MapsEntry {
     /// in general the pathname may be ambiguous. (I.e. you cannot tell if the
     /// denoted filename actually ended with the text "(deleted)", or if that
     /// was added by the maps rendering.
-    pathname: OsString,
+    pub(super) pathname: OsString,
 }
 
 pub(super) fn parse_maps() -> Result<Vec<MapsEntry>, &'static str> {
@@ -72,11 +72,6 @@ pub(super) fn parse_maps() -> Result<Vec<MapsEntry>, &'static str> {
 }
 
 impl MapsEntry {
-    #[inline]
-    pub(super) fn pathname(&self) -> &OsString {
-        &self.pathname
-    }
-
     #[inline]
     pub(super) fn ip_matches(&self, ip: usize) -> bool {
         self.address.0 <= ip && ip < self.address.1