fix(core): don't include a v8 dependency as this breaks docs.rs (deno…

…land#106)
hardfist · Jul 31, 2023 · d526874 · d526874
1 parent 2f9f8c9
commit d526874
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 7 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/ops/Cargo.toml b/ops/Cargo.toml
@@ -28,7 +28,6 @@ strum_macros.workspace = true
 syn.workspace = true
 syn2.workspace = true
 thiserror.workspace = true
-v8.workspace = true
 
 [dev-dependencies]
 pretty_assertions.workspace = true

diff --git a/ops/op2/dispatch_slow.rs b/ops/op2/dispatch_slow.rs
@@ -734,11 +734,7 @@ fn throw_type_error(
   message: String,
 ) -> Result<TokenStream, V8MappingError> {
   // Sanity check ASCII and a valid/reasonable message size
-  debug_assert!(
-    message.is_ascii()
-      && message.len() < v8::String::max_length()
-      && message.len() < 1024
-  );
+  debug_assert!(message.is_ascii() && message.len() < 1024);
 
   let maybe_scope = if generator_state.needs_scope {
     quote!()