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

@@ -20,10 +20,9 @@ pub(super) fn native_libraries() -> Vec<Library> {
 fn infer_current_exe(base_addr: usize) -> OsString {
     if let Ok(entries) = super::parse_running_mmaps::parse_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

@@ -50,7 +50,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> {
@@ -70,11 +70,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