Some comments re Rhai packages

[schungx]

https://github.com/alvinhochun/wasm-project-template/blob/rhai-demo/wasm-module/src/scripting.rs#L10-11

You don't need these lines:

engine.load_package(ArithmeticPackage::new().get());
engine.load_package(LogicPackage::new().get());

Engine::new() already includes these packages. You don't have to load them once again and it'll be better on speed.

If you want to only include these packages and nothing else, you should use Engine::new_raw(). This way, the WASM size will be significantly reduced because code from non-loaded packages will not be included.

[schungx]

Not sure if these in Cargo.toml will reduce the WASM size:

lto = "fat"
codegen-units = 1

[alvinhochun]

Fat LTO does cut down about 12KB in raw size, but it is currently a bit broken on Rust stable.

[schungx]

Well, how about lto = true?

[alvinhochun]

Same thing.

[schungx]

I've been using LTO for quite a while now (well, actually months), and it has been stable so far...

Really needed this for dead-code removal... without it the binaries tend to be huge.

[alvinhochun]

I've been using LTO for quite a while now (well, actually months), and it has been stable so far...

No, it's an issue specific to Wasm: rust-lang/rust#72758