fluencelabs/wasm-bindgen
1
0
Fork 0
mirror of https://github.com/fluencelabs/wasm-bindgen synced 2025-03-28 07:51:07 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #869 from alexcrichton/cleanup-classes

Browse Source 
Only generate JS null checks in debug mode
This commit is contained in:
Alex Crichton 2018-09-21 16:36:02 -07:00 committed by GitHub
commit 616b27457d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 14 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
crates/backend/src/codegen.rs
View File

@ -985,7 +985,7 @@ impl ToTokens for ast::Enum {
_extra: &mut ::wasm_bindgen::convert::Stack, _extra: &mut ::wasm_bindgen::convert::Stack,
) -> Self { ) -> Self {
#(#cast_clauses else)* { #(#cast_clauses else)* {
::wasm_bindgen::throw("invalid enum value passed") ::wasm_bindgen::throw_str("invalid enum value passed")
} }
} }
} }

12
crates/cli-support/src/js/js2rust.rs
View File

@ -68,11 +68,13 @@ impl<'a, 'b> Js2Rust<'a, 'b> {
/// passed should be `this.ptr`. /// passed should be `this.ptr`.
pub fn method(&mut self, method: bool, consumed: bool) -> &mut Self { pub fn method(&mut self, method: bool, consumed: bool) -> &mut Self {
if method { if method {
self.prelude( if self.cx.config.debug {
"if (this.ptr === 0) { self.prelude(
throw new Error('Attempt to use a moved value'); "if (this.ptr === 0) {
}", throw new Error('Attempt to use a moved value');
); }",
);
}
if consumed { if consumed {
self.prelude( self.prelude(
"\ "\

3
tests/wasm/imports.js
View File

@ -85,7 +85,8 @@ exports.custom_type_return_2 = function() {
}; };
exports.touch_custom_type = function() { exports.touch_custom_type = function() {
assert.throws(() => CUSTOM_TYPE.touch(), /Attempt to use a moved value/); assert.throws(() => CUSTOM_TYPE.touch(),
/Attempt to use a moved value|null pointer passed to rust/);
}; };
exports.interpret_2_as_custom_type = function() { exports.interpret_2_as_custom_type = function() {

6
tests/wasm/validate_prt.js
View File

@ -5,14 +5,16 @@ const useMoved = () => {
const apple = new wasm.Fruit('apple'); const apple = new wasm.Fruit('apple');
apple.name(); apple.name();
wasm.eat(apple); wasm.eat(apple);
assert.throws(() => apple.name(), /Attempt to use a moved value/); assert.throws(() => apple.name(),
/Attempt to use a moved value|null pointer passed to rust/);
}; };
const moveMoved = () => { const moveMoved = () => {
const pear = new wasm.Fruit('pear'); const pear = new wasm.Fruit('pear');
pear.name(); pear.name();
wasm.eat(pear); wasm.eat(pear);
assert.throws(() => wasm.eat(pear), /Attempt to use a moved value/); assert.throws(() => wasm.eat(pear),
/Attempt to use a moved value|null pointer passed to rust/);
}; };
exports.js_works = () => { exports.js_works = () => {