chore(examples): remove wee_alloc from wasm-demo and ensure-no_std (#…

…1337) * Remove wee_alloc from wasm-demo The allocator wasn't being used anyways (it was an optional dependency). Also, using it results in a 1.2 MB size, compared to the original 1.1 MB. * Remove wee_alloc from ensure-no_std * Update changelog
lambdaclass · Jul 18, 2023 · 2a1f732 · 2a1f732 · github-actions · Jul 18, 2023
1 parent e1bfb73
commit 2a1f732
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 54 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,8 @@
 
 #### Upcoming Changes
 
+* chore(examples): remove _wee_alloc_ dependency from _wasm-demo_ example and _ensure-no_std_ dummy crate [#1337](https://github.com/lambdaclass/cairo-vm/pull/1337)
+
 * docs: improved crate documentation [#1334](https://github.com/lambdaclass/cairo-vm/pull/1334)
 
 * chore!: made `deserialize_utils` module private [#1334](https://github.com/lambdaclass/cairo-vm/pull/1334)

diff --git a/Cargo.lock b/Cargo.lock
diff --git a/ensure-no_std/Cargo.toml b/ensure-no_std/Cargo.toml
@@ -9,7 +9,7 @@ cairo-felt = { path = "../felt", default-features = false, features = [
 ] }
 cairo-vm = { path = "../vm", default-features = false }
 
-wee_alloc = "0.4.5"
+esp-alloc = "0.3.0"
 
 [features]
 default = ["lambdaworks-felt"]

diff --git a/ensure-no_std/src/main.rs b/ensure-no_std/src/main.rs
@@ -15,7 +15,8 @@ pub extern "C" fn _start() -> ! {
 }
 
 #[global_allocator]
-static ALLOC: wee_alloc::WeeAlloc = wee_alloc::WeeAlloc::INIT;
+// NOTE: this should be initialized before use
+static ALLOC: esp_alloc::EspHeap = esp_alloc::EspHeap::empty();
 
 #[allow(unused_imports)]
 use {cairo_felt, cairo_vm};
diff --git a/examples/wasm-demo/Cargo.toml b/examples/wasm-demo/Cargo.toml
@@ -25,11 +25,6 @@ wasm-bindgen = "0.2.87"
 # code size when deploying.
 console_error_panic_hook = { version = "0.1.6", optional = true }
 
-# `wee_alloc` is a tiny allocator for wasm that is only ~1K in code size
-# compared to the default allocator's ~10K. It is slower than the default
-# allocator, however.
-wee_alloc = { version = "0.4.5", optional = true }
-
 cairo-vm = { workspace = true }
 
 [dev-dependencies]

diff --git a/examples/wasm-demo/src/lib.rs b/examples/wasm-demo/src/lib.rs
@@ -6,12 +6,6 @@ use cairo_vm::{
 };
 use wasm_bindgen::prelude::*;
 
-// When the `wee_alloc` feature is enabled, use `wee_alloc` as the global
-// allocator.
-#[cfg(feature = "wee_alloc")]
-#[global_allocator]
-static ALLOC: wee_alloc::WeeAlloc = wee_alloc::WeeAlloc::INIT;
-
 #[wasm_bindgen]
 extern "C" {
     #[wasm_bindgen(js_namespace = console)]