bench: to json

Author: SyMind

benches/bench.rs

@@ -19,6 +19,7 @@ use rspack_sources::{
 use bench_complex_replace_source::benchmark_complex_replace_source;
 use bench_source_map::{
  benchmark_parse_source_map_from_json, benchmark_source_map_clone,
+ benchmark_stringify_source_map_to_json,
 };
 
 const HELLOWORLD_JS: &str = include_str!(concat!(
@@ -254,6 +255,11 @@ fn bench_rspack_sources(criterion: &mut Criterion) {
 
  group.bench_function("source_map_clone", benchmark_source_map_clone);
 
+ group.bench_function(
+ "stringify_source_map_to_json",
+ benchmark_stringify_source_map_to_json,
+ );
+
  group.finish();
 }
 

benches/bench_source_map.rs

@@ -10,18 +10,25 @@ use rspack_sources::SourceMap;
 
 const ANTD_MIN_JS_MAP: &str = include_str!(concat!(
  env!("CARGO_MANIFEST_DIR"),
- "/benches/fixtures/antd-mini/antd.min.js"
+ "/benches/fixtures/antd-mini/antd.min.js.map"
 ));
 
 pub fn benchmark_parse_source_map_from_json(b: &mut Bencher) {
  b.iter(|| {
- let _ = black_box(|| SourceMap::from_json(ANTD_MIN_JS_MAP).unwrap());
+ black_box(SourceMap::from_json(black_box(ANTD_MIN_JS_MAP)).unwrap())
  })
 }
 
 pub fn benchmark_source_map_clone(b: &mut Bencher) {
  let source = SourceMap::from_json(ANTD_MIN_JS_MAP).unwrap();
  b.iter(|| {
- let _ = black_box(|| source.clone());
+ let _ = black_box(source.clone());
+ })
+}
+
+pub fn benchmark_stringify_source_map_to_json(b: &mut Bencher) {
+ let source = SourceMap::from_json(ANTD_MIN_JS_MAP).unwrap();
+ b.iter(|| {
+ let _ = black_box(source.to_json().unwrap());
  })
 }

src/source.rs

@@ -403,7 +403,7 @@ impl SourceMap {
  }
 
  /// Generate source map to a json string.
- pub fn to_json(self) -> Result<String> {
+ pub fn to_json(&self) -> Result<String> {
  let json = simd_json::serde::to_string(&self)?;
  Ok(json)
  }