webAudio mistypo with --target no-modules

[simdax]

🐛 Bug description

When building the webaudio example, the main .js file instantiate a
new lAudioContext instead of a simple new AudioContext

🤔 Expected Behavior

It should call AudioContext, not lAudioContext

👟 Steps to reproduce

simply copy the webaudio example of wasm-bindgen package, build it with wasm-pack --target no-modules && grep -rn lAudio pkg and you'll see the error

🌍 Your environment

Include the relevant details of your environment.
wasm-pack version: 0.5.1
rustc version: 1.32.0-nightly

[ibaryshnikov]

After a short debug I found

[crates/cli-support/src/js/mod.rs:2105] name = "AudioContext"
[crates/cli-support/src/js/mod.rs:2141] class.to_string() = "lAudioContext"
[crates/cli-support/src/js/rust2js.rs:664] c = "lAudioContext"

https://github.com/rustwasm/wasm-bindgen/blob/master/crates/cli-support/src/js/mod.rs#L2105

what's the purpose of format!("l{}", name) ?

[alexcrichton]

Thanks for the report @simdax! This is definitely a bug in wasm-bindgen right now due to the internal branch we have on no-modules vs not. That's being refactored in #1295 and this issue is actually inadvertently fixed by that PR. I suspect that won't take too long to land, so we can fix this with that!