(ml5717) Small improvements to release code compilation

Cargo.toml

@@ -8,7 +8,7 @@ edition = "2018"
 crate-type = ["cdylib", "rlib"]
 
 [features]
-default = ["console_error_panic_hook"]
+default = []
 
 [dependencies]
 wasm-bindgen = "0.2"
@@ -22,13 +22,7 @@ bincode = "1.2.1"
 fixedbitset = "0.2.0"
 ordered-float = { version = "1.0.2", features = ["serde"] }
 twox-hash = "1.1.1"
-web-sys = { version = "0.3.33", features = ["console"] }
-
-# The `console_error_panic_hook` crate provides better debugging of panics by
-# logging them with `console.error`. This is great for development, but requires
-# all the `std::fmt` and `std::panicking` infrastructure, so isn't great for
-# code size when deploying.
-console_error_panic_hook = { version = "0.1.1", optional = true }
+# web-sys = { version = "0.3.33", features = ["console"] }
 
 # `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
@@ -43,3 +37,4 @@ wasm-bindgen-test = "0.2"
 [profile.release]
 opt-level = 3
 lto = true
+# panic = "abort"

src/lib.rs

 #![feature(type_ascription)]
 
+#[cfg(feature = "wee_alloc")]
+#[global_allocator]
+static ALLOC: wee_alloc::WeeAlloc = wee_alloc::WeeAlloc::INIT;
+
 use wasm_bindgen::prelude::*;
 
 use js_sys::Error;
@@ -10,12 +14,6 @@ use std::ops::Deref;
 use std::option::Option;
 use vec_map::VecMap;
 
-#[cfg(feature = "wee_alloc")]
-#[global_allocator]
-static ALLOC: wee_alloc::WeeAlloc = wee_alloc::WeeAlloc::INIT;
-
-mod utils;
-
 mod crdt;
 use crdt::{DeltaVec, EventListener, IntervalUnion, Point, CRDT};
 
@@ -151,7 +149,20 @@ impl WasmCRDT {
     pub fn apply_deltas(&mut self, deltas: Box<[u8]>) -> Result<bool, JsValue> {
         let deltas = match bincode::deserialize(&packing::unpack(&deltas)) {
             Ok(deltas) => deltas,
-            Err(error) => return Err(Error::new(&format!("{:?}", error)).into()),
+            #[allow(unused_variables)]
+            Err(error) => {
+                return Err(Error::new({
+                    #[cfg(debug_assertions)]
+                    {
+                        &format!("{:?}", error)
+                    }
+                    #[cfg(not(debug_assertions))]
+                    {
+                        "apply_deltas deltas deserialise error"
+                    }
+                })
+                .into())
+            }
         };
 
         Ok(self.0.apply_deltas(deltas))
@@ -168,7 +179,20 @@ impl WasmCRDT {
     ) -> Result<(), JsValue> {
         let remote_state = match bincode::deserialize(&packing::unpack(&remote_state)) {
             Ok(remote_state) => remote_state,
-            Err(error) => return Err(Error::new(&format!("{:?}", error)).into()),
+            #[allow(unused_variables)]
+            Err(error) => {
+                return Err(Error::new({
+                    #[cfg(debug_assertions)]
+                    {
+                        &format!("{:?}", error)
+                    }
+                    #[cfg(not(debug_assertions))]
+                    {
+                        "fetch_deltas_from_state_vector state deserialise error"
+                    }
+                })
+                .into())
+            }
         };
 
         Ok(self.0.fetch_deltas_from_state_vector(user, &remote_state))

src/utils.rs

-#[allow(dead_code)]
-pub fn set_panic_hook() {
-    // When the `console_error_panic_hook` feature is enabled, we can call the
-    // `set_panic_hook` function at least once during initialization, and then
-    // we will get better error messages if our code ever panics.
-    //
-    // For more details see
-    // https://github.com/rustwasm/console_error_panic_hook#readme
-    #[cfg(feature = "console_error_panic_hook")]
-    console_error_panic_hook::set_once();
-}