perf: rm SourceMapLineChunk

Author: SyMind

src/helpers.rs

@@ -2,7 +2,7 @@ use arrayvec::ArrayVec;
 use rustc_hash::FxHashMap as HashMap;
 use std::{
   borrow::{BorrowMut, Cow},
-  cell::{OnceCell, RefCell},
+  cell::RefCell,
   rc::Rc,
 };
 
@@ -902,29 +902,7 @@ fn stream_chunks_of_source_map_lines_full<'a>(
 #[derive(Debug)]
 struct SourceMapLineData<'a> {
   pub mappings_data: Vec<i64>,
-  pub chunks: Vec<SourceMapLineChunk<'a>>,
-}
-
-#[derive(Debug)]
-struct SourceMapLineChunk<'a> {
-  content: Cow<'a, str>,
-  cached: OnceCell<WithIndices<Cow<'a, str>>>,
-}
-
-impl<'a> SourceMapLineChunk<'a> {
-  pub fn new(content: Cow<'a, str>) -> Self {
-    Self {
-      content,
-      cached: OnceCell::new(),
-    }
-  }
-
-  pub fn substring(&self, start_index: usize, end_index: usize) -> &str {
-    let cached = self
-      .cached
-      .get_or_init(|| WithIndices::new(self.content.clone()));
-    cached.substring(start_index, end_index)
-  }
+  pub chunks: Vec<WithIndices<Cow<'a, str>>>,
 }
 
 type InnerSourceIndexValueMapping<'a> =
@@ -1358,7 +1336,7 @@ pub fn stream_chunks_of_combined_source_map<'a>(
                 .unwrap_or(-1),
             );
             // SAFETY: final_source is false
-            let chunk = SourceMapLineChunk::new(chunk.unwrap());
+            let chunk = WithIndices::new(chunk.unwrap());
             data.chunks.push(chunk);
           },
           &mut |i, source, source_content| {

src/replace_source.rs

@@ -278,7 +278,8 @@ impl<'a, T: Source> StreamChunks<'a> for ReplaceSource<T> {
       RefCell::new(Vec::new());
     let name_mapping: RefCell<HashMap<Cow<str>, u32>> =
       RefCell::new(HashMap::default());
-    let name_index_mapping: RefCell<Vec<u32>> = RefCell::new(Vec::new());
+    let name_index_mapping: RefCell<HashMap<u32, u32>> =
+      RefCell::new(HashMap::default());
 
     // check if source_content[line][col] is equal to expect
     // Why this is needed?
@@ -404,10 +405,7 @@ impl<'a, T: Source> StreamChunks<'a> for ReplaceSource<T> {
                     original_line: original.original_line,
                     original_column: original.original_column,
                     name_index: original.name_index.and_then(|name_index| {
-                      name_index_mapping
-                        .borrow()
-                        .get(name_index as usize)
-                        .copied()
+                      name_index_mapping.borrow().get(&name_index).copied()
                     }),
                   }
                 }),
@@ -582,10 +580,7 @@ impl<'a, T: Source> StreamChunks<'a> for ReplaceSource<T> {
                   original_line: original.original_line,
                   original_column: original.original_column,
                   name_index: original.name_index.and_then(|name_index| {
-                    name_index_mapping
-                      .borrow()
-                      .get(name_index as usize)
-                      .copied()
+                    name_index_mapping.borrow().get(&name_index).copied()
                   }),
                 }
               }),
@@ -614,7 +609,7 @@ impl<'a, T: Source> StreamChunks<'a> for ReplaceSource<T> {
         }
         name_index_mapping
           .borrow_mut()
-          .insert(name_index as usize, global_index.unwrap());
+          .insert(name_index, global_index.unwrap());
       },
     );