diff options
Diffstat (limited to 'core')
| -rw-r--r-- | core/extensions.rs | 41 | ||||
| -rw-r--r-- | core/modules.rs | 30 | ||||
| -rw-r--r-- | core/runtime.rs | 4 |
3 files changed, 39 insertions, 36 deletions
diff --git a/core/extensions.rs b/core/extensions.rs index 801f5f1a6..2a578429b 100644 --- a/core/extensions.rs +++ b/core/extensions.rs @@ -37,7 +37,7 @@ impl ExtensionFileSourceCode { #[derive(Clone, Debug)] pub struct ExtensionFileSource { - pub specifier: String, + pub specifier: &'static str, pub code: ExtensionFileSourceCode, } pub type OpFnRef = v8::FunctionCallback; @@ -190,11 +190,11 @@ macro_rules! extension { #[allow(unused_variables)] fn with_js(ext: &mut $crate::ExtensionBuilder) { $( ext.esm( - $crate::include_js_files!( $( dir $dir_esm , )? $( $esm , )* ) + $crate::include_js_files!( $name $( dir $dir_esm , )? $( $esm , )* ) ); )? $( ext.esm(vec![ExtensionFileSource { - specifier: "ext:setup".to_string(), + specifier: "ext:setup", code: ExtensionFileSourceCode::IncludedInBinary($esm_setup_script), }]); )? @@ -202,7 +202,7 @@ macro_rules! extension { ext.esm_entry_point($esm_entry_point); )? $( ext.js( - $crate::include_js_files!( $( dir $dir_js , )? $( $js , )* ) + $crate::include_js_files!( $name $( dir $dir_js , )? $( $js , )* ) ); )? } @@ -456,28 +456,11 @@ pub struct ExtensionBuilder { impl ExtensionBuilder { pub fn js(&mut self, js_files: Vec<ExtensionFileSource>) -> &mut Self { - let js_files = - // TODO(bartlomieju): if we're automatically remapping here, then we should - // use a different result struct that `ExtensionFileSource` as it's confusing - // when (and why) the remapping happens. - js_files.into_iter().map(|file_source| ExtensionFileSource { - specifier: format!("ext:{}/{}", self.name, file_source.specifier), - code: file_source.code, - }); self.js.extend(js_files); self } pub fn esm(&mut self, esm_files: Vec<ExtensionFileSource>) -> &mut Self { - let esm_files = esm_files - .into_iter() - // TODO(bartlomieju): if we're automatically remapping here, then we should - // use a different result struct that `ExtensionFileSource` as it's confusing - // when (and why) the remapping happens. - .map(|file_source| ExtensionFileSource { - specifier: format!("ext:{}/{}", self.name, file_source.specifier), - code: file_source.code, - }); self.esm.extend(esm_files); self } @@ -600,10 +583,10 @@ impl ExtensionBuilder { #[cfg(not(feature = "include_js_files_for_snapshotting"))] #[macro_export] macro_rules! include_js_files { - (dir $dir:literal, $($file:literal,)+) => { + ($name:ident dir $dir:literal, $($file:literal,)+) => { vec![ $($crate::ExtensionFileSource { - specifier: concat!($file).to_string(), + specifier: concat!("ext:", stringify!($name), "/", $file), code: $crate::ExtensionFileSourceCode::IncludedInBinary( include_str!(concat!($dir, "/", $file) )), @@ -611,10 +594,10 @@ macro_rules! include_js_files { ] }; - ($($file:literal,)+) => { + ($name:ident $($file:literal,)+) => { vec![ $($crate::ExtensionFileSource { - specifier: $file.to_string(), + specifier: concat!("ext:", stringify!($name), "/", $file), code: $crate::ExtensionFileSourceCode::IncludedInBinary( include_str!($file) ), @@ -626,10 +609,10 @@ macro_rules! include_js_files { #[cfg(feature = "include_js_files_for_snapshotting")] #[macro_export] macro_rules! include_js_files { - (dir $dir:literal, $($file:literal,)+) => { + ($name:ident dir $dir:literal, $($file:literal,)+) => { vec![ $($crate::ExtensionFileSource { - specifier: concat!($file).to_string(), + specifier: concat!("ext:", stringify!($name), "/", $file), code: $crate::ExtensionFileSourceCode::LoadedFromFsDuringSnapshot( std::path::PathBuf::from(env!("CARGO_MANIFEST_DIR")).join($dir).join($file) ), @@ -637,10 +620,10 @@ macro_rules! include_js_files { ] }; - ($($file:literal,)+) => { + ($name:ident $($file:literal,)+) => { vec![ $($crate::ExtensionFileSource { - specifier: $file.to_string(), + specifier: concat!("ext:", stringify!($name), "/", $file), code: $crate::ExtensionFileSourceCode::LoadedFromFsDuringSnapshot( std::path::PathBuf::from(env!("CARGO_MANIFEST_DIR")).join($file) ), diff --git a/core/modules.rs b/core/modules.rs index ddd55199b..2d80071aa 100644 --- a/core/modules.rs +++ b/core/modules.rs @@ -432,7 +432,7 @@ impl ModuleLoader for ExtModuleLoader { if let Some(file_source) = maybe_file_source { { let mut used_esm_sources = self.used_esm_sources.borrow_mut(); - let used = used_esm_sources.get_mut(&file_source.specifier).unwrap(); + let used = used_esm_sources.get_mut(file_source.specifier).unwrap(); *used = true; } @@ -1047,13 +1047,23 @@ impl ModuleMap { let main = v8::Boolean::new(scope, info.main); module_info_arr.set_index(scope, 1, main.into()); - let name = v8::String::new(scope, &info.name).unwrap(); + let name = v8::String::new_from_one_byte( + scope, + info.name.as_bytes(), + v8::NewStringType::Normal, + ) + .unwrap(); module_info_arr.set_index(scope, 2, name.into()); let array_len = 2 * info.requests.len() as i32; let requests_arr = v8::Array::new(scope, array_len); for (i, request) in info.requests.iter().enumerate() { - let specifier = v8::String::new(scope, &request.specifier).unwrap(); + let specifier = v8::String::new_from_one_byte( + scope, + request.specifier.as_bytes(), + v8::NewStringType::Normal, + ) + .unwrap(); requests_arr.set_index(scope, 2 * i as u32, specifier.into()); let asserted_module_type = @@ -1079,7 +1089,12 @@ impl ModuleMap { let arr = v8::Array::new(scope, 3); let (specifier, asserted_module_type) = elem.0; - let specifier = v8::String::new(scope, specifier).unwrap(); + let specifier = v8::String::new_from_one_byte( + scope, + specifier.as_bytes(), + v8::NewStringType::Normal, + ) + .unwrap(); arr.set_index(scope, 0, specifier.into()); let asserted_module_type = @@ -1088,7 +1103,12 @@ impl ModuleMap { let symbolic_module: v8::Local<v8::Value> = match &elem.1 { SymbolicModule::Alias(alias) => { - let alias = v8::String::new(scope, alias).unwrap(); + let alias = v8::String::new_from_one_byte( + scope, + alias.as_bytes(), + v8::NewStringType::Normal, + ) + .unwrap(); alias.into() } SymbolicModule::Mod(id) => { diff --git a/core/runtime.rs b/core/runtime.rs index 054c55470..f1aa63f23 100644 --- a/core/runtime.rs +++ b/core/runtime.rs @@ -712,7 +712,7 @@ impl JsRuntime { futures::executor::block_on(async { let id = runtime .load_side_module( - &ModuleSpecifier::parse(&file_source.specifier)?, + &ModuleSpecifier::parse(file_source.specifier)?, None, ) .await?; @@ -748,7 +748,7 @@ impl JsRuntime { // TODO(@AaronO): use JsRuntime::execute_static() here to move src off heap realm.execute_script( self.v8_isolate(), - &file_source.specifier, + file_source.specifier, &file_source.code.load()?, )?; } |