perf: add more

src/builder.rs

@@ -20,12 +20,12 @@ use crate::types::{RawToken, SourceMap, Token};
 /// objects is generally not very comfortable.  As a general aid this
 /// type can help.
 pub struct SourceMapBuilder {
-    file: Option<String>,
+    file: Option<Arc<str>>,
     name_map: HashMap<Arc<str>, u32>,
     names: Vec<Arc<str>>,
     tokens: Vec<RawToken>,
     source_map: HashMap<Arc<str>, u32>,
-    source_root: Option<String>,
+    source_root: Option<Arc<str>>,
     sources: Vec<Arc<str>>,
     source_contents: Vec<Option<Arc<str>>>,
     sources_mapping: Vec<u32>,
@@ -53,7 +53,7 @@ impl SourceMapBuilder {
     /// Creates a new source map builder and sets the file.
     pub fn new(file: Option<&str>) -> SourceMapBuilder {
         SourceMapBuilder {
-            file: file.map(str::to_owned),
+            file: file.map(Into::into),
             name_map: HashMap::new(),
             names: vec![],
             tokens: vec![],
@@ -72,7 +72,7 @@ impl SourceMapBuilder {
     }
 
     /// Sets the file for the sourcemap (optional)
-    pub fn set_file<T: Into<String>>(&mut self, value: Option<T>) {
+    pub fn set_file<T: Into<Arc<str>>>(&mut self, value: Option<T>) {
         self.file = value.map(Into::into);
     }
 
@@ -82,7 +82,7 @@ impl SourceMapBuilder {
     }
 
     /// Sets a new value for the source_root.
-    pub fn set_source_root<T: Into<String>>(&mut self, value: Option<T>) {
+    pub fn set_source_root<T: Into<Arc<str>>>(&mut self, value: Option<T>) {
         self.source_root = value.map(Into::into);
     }
 

src/decoder.rs

@@ -206,7 +206,7 @@ pub fn decode_regular(rsm: RawSourceMap) -> Result<SourceMap> {
 
     // file sometimes is not a string for unexplicable reasons
     let file = rsm.file.map(|val| match val {
-        Value::String(s) => s,
+        Value::String(s) => s.into(),
         _ => "<invalid>".into(),
     });
 

src/types.rs

@@ -458,11 +458,11 @@ pub struct SourceMapIndex {
 /// rejected with an error on reading.
 #[derive(Clone, Debug)]
 pub struct SourceMap {
-    pub(crate) file: Option<String>,
+    pub(crate) file: Option<Arc<str>>,
     pub(crate) tokens: Vec<RawToken>,
     pub(crate) index: Vec<(u32, u32, u32)>,
     pub(crate) names: Vec<Arc<str>>,
-    pub(crate) source_root: Option<String>,
+    pub(crate) source_root: Option<Arc<str>>,
     pub(crate) sources: Vec<Arc<str>>,
     pub(crate) sources_prefixed: Option<Vec<Arc<str>>>,
     pub(crate) sources_content: Vec<Option<SourceView>>,
@@ -574,7 +574,7 @@ impl SourceMap {
     /// - `sources` a vector of source filenames
     /// - `sources_content` optional source contents
     pub fn new(
-        file: Option<String>,
+        file: Option<Arc<str>>,
         tokens: Vec<RawToken>,
         names: Vec<Arc<str>>,
         sources: Vec<Arc<str>>,
@@ -619,7 +619,7 @@ impl SourceMap {
     }
 
     /// Sets a new value for the file.
-    pub fn set_file<T: Into<String>>(&mut self, value: Option<T>) {
+    pub fn set_file<T: Into<Arc<str>>>(&mut self, value: Option<T>) {
         self.file = value.map(Into::into);
     }
 
@@ -643,7 +643,7 @@ impl SourceMap {
     }
 
     /// Sets a new value for the source_root.
-    pub fn set_source_root<T: Into<String>>(&mut self, value: Option<T>) {
+    pub fn set_source_root<T: Into<Arc<str>>>(&mut self, value: Option<T>) {
         self.source_root = value.map(Into::into);
 
         match self.source_root.as_deref().filter(|rs| !rs.is_empty()) {